fix: add ARM support, by switching to Debian as bases image#142
Open
Michiel Piscaer (mpiscaer) wants to merge 4 commits intovexxhost:mainfrom
Open
fix: add ARM support, by switching to Debian as bases image#142Michiel Piscaer (mpiscaer) wants to merge 4 commits intovexxhost:mainfrom
Michiel Piscaer (mpiscaer) wants to merge 4 commits intovexxhost:mainfrom
Conversation
Member
|
At this point, all the dependent images should be ready at this point :) Going to try and see what this might look like soon. |
Michiel Piscaer (mpiscaer)
force-pushed
the
armsupport
branch
3 times, most recently
from
65a028f to
15166e6
Compare
Signed-off-by: Michiel Piscaer <michiel@piscaer.com>
Michiel Piscaer (mpiscaer)
force-pushed
the
armsupport
branch
from
15166e6 to
319e3e9
Compare
Michiel Piscaer (mpiscaer)
changed the title
Author
|
I did a new build based on the new structure, also I run the new images on an ARM base architecture, it looks good. |
Signed-off-by: Michiel Piscaer <michiel@piscaer.com>
There was a problem hiding this comment.
Pull request overview
This PR updates the container base images to Debian variants to improve multi-architecture compatibility (notably ARM) and simplifies Apache OpenID Connect module installation by using Debian packages.
Changes:
- Switch build stage to
openstack-venv-builder-debianand runtime stage topython-base-debian. - Install
libapache2-mod-auth-openidcviaapt-get(removing the prior architecture-gated manual.debdownload/install flow).
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Michiel Piscaer (mpiscaer)
force-pushed
the
armsupport
branch
2 times, most recently
from
be36f9a to
289e958
Compare
Signed-off-by: Michiel Piscaer <michiel@piscaer.com>
Author
|
Mohammed Naser (@mnaser) Do you think this can merge? |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.