Skip to content

Commit 536ec5b

Browse files
authored
Merge pull request #115 from TomasTomecek/fix-114
podman i exists does not work in buildah unshare
2 parents 109daa9 + 838bdca commit 536ec5b

File tree

2 files changed

+9
-3
lines changed

2 files changed

+9
-3
lines changed

README.md

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -104,8 +104,11 @@ privileged container.
104104

105105
#### Setting up buildah and podman
106106

107-
If you run ansible-bender as root, you don't need to do anything. Just install
108-
the packages and you are good to go.
107+
If you run ansible-bender as root, you don't need to set up anything. Just
108+
install the packages and you are good to go. This is the preferred way —
109+
buildah and podman are much more efficient when using the in-kernel overlay
110+
filesystem and you will encounter [less issues than with the rootless
111+
mode](https://github.com/containers/libpod/blob/master/rootless.md).
109112

110113
On the other hand, if you want to utilize [the rootless
111114
mode](https://github.com/containers/libpod/blob/master/docs/podman-create.1.md#rootless-containers),

ansible_bender/builders/buildah_builder.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,10 @@ def pull_buildah_image(container_image):
3636

3737

3838
def does_image_exist(container_image):
39-
cmd = ["podman", "image", "exists", container_image]
39+
# cmd = ["podman", "image", "exists", container_image]
40+
# https://github.com/containers/libpod/issues/2924
41+
# https://github.com/ansible-community/ansible-bender/issues/114
42+
cmd = ["buildah", "inspect", "-t", "image", container_image]
4043
run_cmd(cmd, print_output=True)
4144

4245

0 commit comments

Comments
 (0)