Skip to content

tests/int/selinux: test keyring security label#4672

Merged
lifubang merged 2 commits intoopencontainers:mainfrom
kolyshkin:key-selinux-label
Mar 27, 2025
Merged

tests/int/selinux: test keyring security label#4672
lifubang merged 2 commits intoopencontainers:mainfrom
kolyshkin:key-selinux-label

Conversation

@kolyshkin
Copy link
Copy Markdown
Contributor

This tests the functionality added by commit cd96170
("Need to setup labeling of kernel keyrings.", PR #2012), for both
runc run and runc exec, with and without user namespace.

Related to opencontainers/selinux#222

Copy link
Copy Markdown
Member

@rata rata left a comment

Choose a reason for hiding this comment

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

LGTM

Comment thread tests/integration/selinux.bats Outdated
Comment thread tests/integration/selinux.bats Outdated
@kolyshkin kolyshkin force-pushed the key-selinux-label branch 2 times, most recently from 0ca17e3 to 45557ba Compare March 13, 2025 17:20
Copy link
Copy Markdown
Member

@rata rata left a comment

Choose a reason for hiding this comment

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

LGTM

@kolyshkin kolyshkin force-pushed the key-selinux-label branch 2 times, most recently from bc3c452 to 8180002 Compare March 24, 2025 19:06
@kolyshkin kolyshkin requested review from lifubang and thaJeztah March 26, 2025 02:28
Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
This tests the functionality added by commit cd96170
("Need to setup labeling of kernel keyrings."), for both
runc run and runc exec, with and without user namespace.

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
@lifubang lifubang force-pushed the key-selinux-label branch from 8180002 to 131bdac Compare March 27, 2025 00:42
@lifubang lifubang merged commit 1dc89f7 into opencontainers:main Mar 27, 2025
34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants