-
Notifications
You must be signed in to change notification settings - Fork 112
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
Small fix for release.sh so that it will work with podman's docker cli #1688
Comments
Thanks for raising this @ThorbenJ Btw had a quick look and seems podman does support .tar.gz? |
I am using Debian Stable my podman does not like it, its entirely possible that the latest now does; or that its the lack of explicit file extension. I figured I'd share this for anyone else in my boat, as it was a simple compatibility fix. FYI
|
So I tried this on my box and seems to work, though my podman is more recent: ❯ podman load <$(nix-build -A images.mayastor-io-engine) ─╯
WARN[0000] Using cgroups-v1 which is deprecated in favor of cgroups-v2 with Podman v5 and will be removed in a future version. Set environment variable `PODMAN_IGNORE_CGROUPSV1_WARNING` to hide this warning.
Getting image source signatures
Copying blob 252ebb56fa0a done |
Copying config 0b3387f3d1 done |
Writing manifest to image destination
Loaded image: localhost/openebs/mayastor-io-engine:5fa8b486b675
❯ podman --version ─╯
podman version 5.0.3 |
Ss Debian Stable appears to be a major version behind. This issue documents a work around for users of that version of podman; as such that might just be enough and therefore the issue could be closed. |
Also this seems to work btw:
|
I've raised #1703 on the dataplane repo, it checks if docker or podman are available, and makes use of it. |
Release script now can pick up docker or podman.
|
A bug was introduced, fixing this... |
Alright, re-fixed and also now, if using podman and load of the tar.gz fails, we use zcat and load the tar instead. Also you can opt in from the start with --tar. |
Describe the bug
Although podman pretends to provide a compatible docker cli, it's not 100% compatible.
I made a small change to release.sh that allowed it to work, so that that I could build for arm64
To Reproduce
Install podman instead of docker, and then run release.sh to build to container images
Expected behavior
All images built and loaded
Screenshots
** OS info (please complete the following information): **
Additional context
It doesn't like gz tar files apparently, and I'm not sure about the lack of extention.
I changed
to this
(Lines before and after remain the same.
i.e. I needed gunzip the file, and (I believe) keep the .tar extention - and it now works fine. It generated and loaded all the images from both mayastor repos.
The text was updated successfully, but these errors were encountered: