🐛📦 Fix the OCI manylinux image platform tag #692
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
webknjaz
added
bug
This was caused by a mistake during the initial migration from docker to podman in ansible#181. The original command looked like `docker buildx build --platform linux/arm64` but the `redhat-actions/buildah-build` action has an input called `arch:` that we started using. And we've prefixed the passed value with `linux/`. It would've been fine if we used the `platform:` input but we didn't. The consequence was that the images we were making were tagged with an additional leading `linux/` prefix in the platform metadata which made it look like `linux/linux/arm64`. The issue became evident in ansible#648 that attempted to bump the version of `cibuildwheel`. And the container interaction started failing loudly as this tool started using the `--platform` option when working with OCI images in v2.21 [[1]]. [1]: pypa/cibuildwheel#1961
webknjaz
force-pushed
the
maintenance/manylinux-cache-platform-tags
branch
from
56fbac6 to
4a1047b
Compare
psf-chronographer
bot
added
the
bot:chronographer:provided
|
|
Congratulations! One of the builds has completed. 🍾 You can install the built RPMs by following these steps:
Please note that the RPMs should be used only in a testing environment. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This was caused by a mistake during the initial migration from docker to podman in #181. The original command looked like
docker buildx build --platform linux/arm64but theredhat-actions/buildah-buildaction has an input calledarch:that we started using. And we've prefixed the passed value withlinux/. It would've been fine if we used theplatform:input but we didn't. The consequence was that the images we were making were tagged with an additional leadinglinux/prefix in the platform metadata which made it look likelinux/linux/arm64.The issue became evident in #648 that attempted to bump the version of
cibuildwheel. And the container interaction started failing loudly as this tool started using the--platformoption when working with OCI images in v2.21 [1].ISSUE TYPE
ADDITIONAL INFORMATION
Here's what the crash looks like: