Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dockerctl: use 'podman' if present and 'docker version' fails #11

Merged
merged 2 commits into from
May 1, 2019

Conversation

wjt
Copy link
Contributor

@wjt wjt commented Mar 15, 2019

podman is an almost-CLI-compatible replacement for docker, which
allows unprivileged users to run containers, with no daemon. On some
systems, docker is a symbolic link to podman.

I say "almost" because one command that is not compatible is docker version --format '{{ json . }}', used by Karton to detect whether
docker is present and usable.
containers/podman#2671

To work around this, if _try_docker() fails, but running podman version succeeds, use podman instead. Otherwise, behave as before.

wjt added 2 commits April 30, 2019 12:52
`podman` is an almost-CLI-compatible replacement for `docker`, which
allows unprivileged users to run containers, with no daemon. On some
systems, `docker` is a symbolic link to `podman`.

I say "almost" because one command that is not compatible is `docker
version --format '{{ json . }}'`, used by Karton to detect whether
docker is present and usable.
containers/podman#2671

To work around this, if _try_docker() fails, but running `podman
version` succeeds, use `podman` instead. Otherwise, behave as before.
@wjt wjt force-pushed the use-podman-if-docker-fails branch from 0771135 to 23c5a9c Compare April 30, 2019 11:53
@wjt
Copy link
Contributor Author

wjt commented Apr 30, 2019

Updated to fix some warnings caught by a linter .

@barisione barisione merged commit 6909fd6 into karton:master May 1, 2019
@wjt wjt deleted the use-podman-if-docker-fails branch May 1, 2019 15:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants