Skip to content

🐛 Fix CentOS Stream 10 build#734

Merged
metal3-io-bot merged 1 commit intometal3-io:mainfrom
elfosardo:fix-cs10-build
Aug 26, 2025
Merged

🐛 Fix CentOS Stream 10 build#734
metal3-io-bot merged 1 commit intometal3-io:mainfrom
elfosardo:fix-cs10-build

Conversation

@elfosardo
Copy link
Member

@elfosardo elfosardo commented Aug 20, 2025

During efi build, remove the shim package but not its
dependencies to avoid conflict between architectures.

This has been tested locally with both CS9 and CS10
base images.

Signed-off-by: Riccardo Pittau elfosardo@gmail.com

@metal3-io-bot metal3-io-bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Aug 20, 2025
@elfosardo elfosardo changed the title Fix CentOS Stream 10 build 🐛 Fix CentOS Stream 10 build Aug 20, 2025
@elfosardo
Copy link
Member Author

/test metal3-centos-e2e-integration-test-main metal3-ubuntu-e2e-integration-test-main

During efi build, remove the shim package but not its
dependencies to avoid conflict between architectures.

This has been tested locally with both CS9 and CS10
base images.

Signed-off-by: Riccardo Pittau <elfosardo@gmail.com>
@elfosardo
Copy link
Member Author

/test metal3-centos-e2e-integration-test-main metal3-ubuntu-e2e-integration-test-main

@tuminoid
Copy link
Member

/retest

Copy link
Member

@tuminoid tuminoid left a comment

Choose a reason for hiding this comment

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

/lgtm

Tested manually both CS9 and CS10 builds, they work.

@metal3-io-bot metal3-io-bot added the lgtm Indicates that a PR is ready to be merged. label Aug 21, 2025
@diconico07
Copy link
Contributor

lgtm as well, I only tested with CS9 initially.

@elfosardo
Copy link
Member Author

/retest

@tuminoid
Copy link
Member

/cc @dtantsur @Rozzii

@metal3-io-bot metal3-io-bot requested a review from Rozzii August 21, 2025 14:06
@elfosardo
Copy link
Member Author

/cc @iurygregory

@tuminoid
Copy link
Member

Guys, please approve this so we get the CS10 not to fail every merge.

Copy link
Member

@Rozzii Rozzii left a comment

Choose a reason for hiding this comment

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

/approve

@metal3-io-bot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Rozzii

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@metal3-io-bot metal3-io-bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 26, 2025
@metal3-io-bot metal3-io-bot merged commit ace325b into metal3-io:main Aug 26, 2025
10 of 12 checks passed
@metal3-io-bot metal3-io-bot added this to the ironic-image - v32.0 milestone Aug 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants