Support podman in tasks/build.rake

[gregorynisbet-google]

Support podman in addition to docker in tasks/build.rake via the DOCKER environment variable.

Pull Request (PR) description

I did the simplest thing I could think of to make DOCKER=podman rake vox:build succeed.

There are a few differences between --format json for Docker and podman, but I still think
that that's the most robust way to get the commands to emit structured output.

I'm curious whether folks think this is worth doing at all, and whether an environment variable
is the best way of achieving this (and also whether shell substitution is better than doing the
conditional at the ruby level)

This Pull Request (PR) fixes the following issues

n/a

[Sharpie]

I opened #228 before I noticed this PR. The approach there is even simpler, but requires the use of the podman-docker shim package to re-direct the docker CLI to podman.

Building other OpenVox components, such as openvox-agent and openbolt, under Podman requires the use of podman-docker unless some enhancement is done to Vanagon.

[gregorynisbet-google]

No worries. Any interest in me reworking this to remove the dependency on the shim?

[Sharpie]

Using environment variables to pick a CLI implementation seems reasonable to me. Perhaps DOCKER_BIN or DOCKER_CLI instead of DOCKER as the shorter variable is sometimes used to flag whether or not a process is running in a containerized environment.

[Sharpie]

Looks good to me 👍

Thanks for the contribution!

[gregorynisbet-google]

Hi, so I can keep the PR up to date with respect to main (which I did just now with a force push), but I don't know the procedure for actually merging the PR.

Auto-merge was enabled prior to my last push, but I wasn't able to merge it at that time. Is two committers' approval needed for a PR to be merged?