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

[next-devel] overrides: fix grub2 arch specific overrides #3209

Merged
merged 1 commit into from
Oct 17, 2024

Conversation

dustymabe
Copy link
Member

We can't have any grub2 overrides in the manifest-lock.overrides.yaml file because there are no grub2 packages on s390x. Let's move those into the arch specific overrides files and also correct a few of them.

Fixes 31802d2

We can't have any grub2 overrides in the manifest-lock.overrides.yaml
file because there are no grub2 packages on s390x. Let's move those
into the arch specific overrides files and also correct a few of them.

Fixes 31802d2
Copy link
Member

@aaradhak aaradhak left a comment

Choose a reason for hiding this comment

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

LGTM

Cross-checked if we have everything with the below list generated by the overrides.py script.

'x86_64': {'grub2-tools-minimal', 'grub2-pc', 'grub2-common', 'grub2-pc-modules', 'grub2-tools', 'grub2-efi-x64'}, 
'ppc64le': {'grub2-ppc64le', 'grub2-tools-minimal', 'grub2-common', 'grub2-tools', 'grub2-ppc64le-modules'}, 
'aarch64': {'grub2-efi-aa64', 'grub2-common', 'grub2-tools', 'grub2-tools-minimal'}}

@dustymabe
Copy link
Member Author

CI already passed the build phase and the actual RPMs on x86_64 were tested in #3190 so I'll merge this.

@dustymabe dustymabe merged commit 9414d83 into coreos:next-devel Oct 17, 2024
1 of 2 checks passed
@dustymabe dustymabe deleted the dusty-next-devel-overrides branch October 17, 2024 20:24
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.

2 participants