Skip to content
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

Release Metal3 0.9.1 #173

Merged
merged 2 commits into from
Dec 6, 2024
Merged

Conversation

alknopfler
Copy link
Contributor

@alknopfler alknopfler commented Dec 2, 2024

This version updates the ironic and ipa-downloader images - some fixes landed in both to enable builds for aarch64, so we can enable initial testing of this architecture (preview/beta status)

Note that currently the only scenario expected to work is downstream cluster deployments of the same architecture as the management cluster - mixed architecture scenarios are not yet enabled and will be addressed via a future chart update.

@hardys hardys changed the title Fix add arm support Release Metal3 0.9.1 Dec 2, 2024
@hardys hardys requested a review from fdegir December 2, 2024 16:16
@hardys
Copy link
Contributor

hardys commented Dec 3, 2024

Thanks @alknopfler - not yet tested but lgtm, two small requests re the commit messages (so we can rebase merge the PR without squashing), can you rename the commit messages with something like:

  • metal3: Update ironic images to enable arm testing
  • make charts/html

These will be a bit more descriptive in the history if we want to merge the commits directly without squashing, thanks!

@fdegir please could we regression test with SV to ensure the x86 scenario is still working as expected?

Copy link
Contributor

@hardys hardys left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm - also locally tested (x86 VM only) and works for me

Copy link

@fdegir fdegir left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Validation passed!

@hardys hardys marked this pull request as ready for review December 6, 2024 16:33
@hardys hardys merged commit 5d82785 into suse-edge:main Dec 6, 2024
2 checks passed
@alknopfler alknopfler deleted the fix-add-arm-support branch December 9, 2024 08:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants