-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
placeholder: reason for emergency podman-build --arch exception #9915
Comments
@vrothberg This looks like containers/image is not catching the localhost case? |
Not quite. There are a couple of things to unpack:
I think it's a Buildah bug. Added to my todo. containers/image is entirely unaware of the "localhost" prefixing that Podman and Buildah do. |
Update. It's a podman-side bug. Cannot reproduce with |
Move the platform check into `resolveLocalImage` and inspect each of the candidates in the given order. This preserves the local lookup order and fixes a bug reported in containers/podman/issues/9915. Signed-off-by: Valentin Rothberg <[email protected]>
Move the platform check into `resolveLocalImage` and inspect each of the candidates in the given order. This preserves the local lookup order and fixes a bug reported in containers/podman/issues/9915. Signed-off-by: Valentin Rothberg <[email protected]>
I opened containers/buildah#3129 to fix it. It worked with Buildah since it's using a different default pull policy. |
containers/buildah#3129 merged. @rhatdan, do you want to backport or shall we wait for the next release? |
Wait for next release. which might be coming up quickly. |
A friendly reminder that this issue had no activity for 30 days. |
This looks like it is fixed in the main branch now. |
The following bugs seem to be fixed: * containers#6510 (e2e tests) - podman rmi gives "layer not known" * containers#9915 (buildah-bud tests) - podman build --arch * containers#10248 - spurious warning from first-ever invocation of rootless podman Signed-off-by: Ed Santiago <[email protected]>
PR #9821 vendored in buildah v1.20.0. That brought in a new test of the
--arch
flag:https://github.com/containers/buildah/blob/b405ca6551576a98fbc79ae8fe701246d1c7a4f0/tests/bud.bats#L2596
This test failed under podman CI, requiring an emergency
skip
to get CI passing:The reason for this has not been understood. It now seems like it's related to the
prefetch
helper. In particular, here is a reproducer:The image does include
arm64
and other arches:I wonder if this is related to #3761, in which podman save/load changes digests?
Anyhow, sorry for the non-helpful issue; I'm out of time for today. Hope someone can run with this.
The text was updated successfully, but these errors were encountered: