Skip to content

Conversation

@cgwalters
Copy link
Member

compose: Split off a containers_storage module

Prep for further work.

Signed-off-by: Colin Walters walters@verbum.org


compose: Support buildah too

It's helpful for us to support build environments that only have podman,
as well as those that only have buildah. They're mostly doing the same
things; an abstraction over them is trivial. From this code, the only
difference really is podman create vs buildah from.

Signed-off-by: Colin Walters walters@verbum.org


compose: Add some error prefixing

Turned out I was missing skopeo.

Signed-off-by: Colin Walters walters@verbum.org


@cgwalters
Copy link
Member Author

OK for me this now works inside the stock buildah container to go from built container (i.e. in containers-storage: -> OCI. Trying to copy the image back via skopeo fails on user namespace mapping issues, but doing a buildah pull does.

Copy link
Member

@jmarrero jmarrero left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@cgwalters
Copy link
Member Author

unshare: unrecognized option '--map-users=all'

Nooo util-linux in c9s is too old for that...ugh

Prep for further work.

Signed-off-by: Colin Walters <walters@verbum.org>
It's helpful for us to support build environments that only have podman,
as well as those that only have buildah. They're mostly doing the same
things; an abstraction over them is trivial. From this code, the only
difference really is `podman create` vs `buildah from`.

Signed-off-by: Colin Walters <walters@verbum.org>
Turned out I was missing skopeo.

Signed-off-by: Colin Walters <walters@verbum.org>
@cgwalters cgwalters force-pushed the chunk-via-buildah branch 4 times, most recently from 44cf92b to 88d2ae5 Compare February 7, 2025 15:50
Signed-off-by: Colin Walters <walters@verbum.org>
@cgwalters
Copy link
Member Author

Whee, finally got all the GHA tests to work!

@cgwalters cgwalters disabled auto-merge February 7, 2025 18:38
@cgwalters cgwalters merged commit 3ac642e into coreos:main Feb 7, 2025
17 of 18 checks passed
@cgwalters
Copy link
Member Author

Jenkins failure was longstanding autoupdate flake

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.

3 participants