-
Notifications
You must be signed in to change notification settings - Fork 55
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
Enable multi-arch builds again #546
Conversation
2a394f5
to
4d75810
Compare
This pull request has been marked 'stale' due to lack of recent activity. If there is no further activity, the PR will be closed in another 30 days. Thank you for your contribution! |
bump |
This issue is no longer marked for closure. |
|
This pull request has been marked 'stale' due to lack of recent activity. If there is no further activity, the PR will be closed in another 30 days. Thank you for your contribution! |
Bumping this again, would still like to see mult-arch builds working on here. |
This pull request is no longer marked for closure. |
@StopMotionCuber Sorry for the delay. I was out the month of September and this fell through the cracks. Merging now! |
Building oci-env fails after this change... (on amd64)
(also, https://github.com/ansible/ansible-hub-ui/actions/runs/6924023217/job/18832694190?pr=4531#step:9:223) (This looks quite close to the errors seen in pulp/oci_env#124, referencing #369 ... might be why this was previously reverted? Should we do so again?) |
I'd favor not directly closing it, but rather solving the root cause of that issue |
I'm currently emulating the building steps manually on my workstation (takes quite a while with |
I was looking for some sort of override that would be forcing oci-env to build a non-default arch, but I think what's happening is, the multi-archness of the image is infectious and causes any images built from it to also try building for all the archs, as opposed to just the current one. After pruning everything, setting |
@himdel I'm not completely sure about your investigation. What I've discovered so far was that no multi-arch manifest is pushed to the registry and I'm not sure which architecture is pushed to the registry or whether that is random (which might lead to some weird effects). Unfortunately with Anyway, a followup is ready in #562, which does the manifest stuff properly |
This is the same as for #367 and should fix #366. For some reason that commit got reverted and I'm not sure why.