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

Fixes for unblocking RHEL-10 container builds #5889

Merged
merged 2 commits into from
Sep 20, 2024

Conversation

KKoukiou
Copy link
Contributor

This will unblock container refreshes.
This affects also RHEL.

Related: https://bugzilla.redhat.com/show_bug.cgi?id=2295428
Related: https://issues.redhat.com/browse/RHEL-46558

Cherry-picked from master branch commit: 56e0269.

This will unblock container refreshes.
This affects also RHEL.

Related: https://bugzilla.redhat.com/show_bug.cgi?id=2295428
Related: https://issues.redhat.com/browse/RHEL-46558

Cherry-picked from master branch commit: 56e0269.
@github-actions github-actions bot added infrastructure Changes affecting mostly infrastructure rhel-10 labels Sep 20, 2024
@KKoukiou
Copy link
Contributor Author

/kickstart-tests --waive unit tests covered

Copy link
Contributor

@M4rtinK M4rtinK left a comment

Choose a reason for hiding this comment

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

Looking good, just not sure if the second commit is intended to be actually a part of this infra PR ?

@@ -169,7 +169,7 @@ def test_get_default_file_system(self):
def test_get_supported_raid_levels(self):
"""Test GetSupportedRaidLevels."""
assert self.interface.GetSupportedRaidLevels(DEVICE_TYPE_MD) == \
['linear', 'raid0', 'raid1', 'raid10', 'raid4', 'raid5', 'raid6']
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this intended to be there ? I find it strange this is part of an infra PR ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Well it's not an infra PR.
It's a PR with some fixes that are needed to re-enable building centos10 and centos9 containers

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Even the 'infra' marked commit - has tests changes.

@KKoukiou KKoukiou changed the title infra: temporarily install selinux-policy-targeted in container Fixes for unblocking RHEL-10 container builds Sep 20, 2024
@KKoukiou
Copy link
Contributor Author

infra check is naive - checking per-pr - not per-commit validity.

@KKoukiou KKoukiou merged commit 5bcaa58 into rhinstaller:rhel-10 Sep 20, 2024
9 of 10 checks passed
@KKoukiou KKoukiou deleted the RHEL-46558 branch September 20, 2024 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infrastructure Changes affecting mostly infrastructure rhel-10
Development

Successfully merging this pull request may close these issues.

2 participants