Skip to content

ci/cirrus: disable selinux-dmz kludge for centos-stream-8#4107

Merged
AkihiroSuda merged 1 commit intoopencontainers:mainfrom
kolyshkin:dmz-selinux-followup
Nov 6, 2023
Merged

ci/cirrus: disable selinux-dmz kludge for centos-stream-8#4107
AkihiroSuda merged 1 commit intoopencontainers:mainfrom
kolyshkin:dmz-selinux-followup

Conversation

@kolyshkin
Copy link
Copy Markdown
Contributor

@kolyshkin kolyshkin commented Nov 3, 2023

This is a followup to #4053.

CentOS Stream 8 now comes with container-selinux 2:2.224.0-1.module_el8+712+4cd1bd69, so we only need the kludge for CentOS 7 (which, I guess, is the sole reason why we have this kludge at all).

This is only for CI.

@kolyshkin kolyshkin marked this pull request as ready for review November 4, 2023 02:00
@AkihiroSuda
Copy link
Copy Markdown
Member

CentOS 7 (which, I guess, is the sole reason why we have this kludge at all)

The new version of container-selinux isn't delivered to other EL8-like distros yet.

Comment thread .cirrus.yml Outdated
@kolyshkin
Copy link
Copy Markdown
Contributor Author

The new version of container-selinux isn't delivered to other EL8-like distros yet.

The change in this PR is only for our CI and only on Cirrus.

It now comes with container-selinux 2:2.224.0-1.module_el8+712+4cd1bd69,
so we only need the kludge for CentOS 7 (which, I guess, is the sole
reason why we have this kludge at all).

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
@kolyshkin kolyshkin force-pushed the dmz-selinux-followup branch from 78960d0 to 823636c Compare November 6, 2023 21:41
@AkihiroSuda AkihiroSuda merged commit 0580863 into opencontainers:main Nov 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants