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

ci/cirrus: enable some rootless tests on cs9 #3553

Merged
merged 3 commits into from
Jun 10, 2023

Conversation

kolyshkin
Copy link
Contributor

@kolyshkin kolyshkin commented Aug 8, 2022

We were not running localrootlessintegration test on CentOS Stream 9
because of some failures.

Fix those, and enable rootless integration with both systemd and fs drivers.

Amends: #3427

@kolyshkin
Copy link
Contributor Author

Need to figure out the reason of failures, but currently don't have enough time.

@kolyshkin kolyshkin force-pushed the moar-ci branch 8 times, most recently from 8552107 to 145d03a Compare May 18, 2023 00:09
@kolyshkin kolyshkin marked this pull request as ready for review June 7, 2023 20:28
@kolyshkin kolyshkin added the backport/1.1-todo A PR in main branch which needs to be backported to release-1.1 label Jun 7, 2023
Filter out rdma controller since systemd is unable to delegate it.
Similar to commits 0527271 and 601cf58.

Signed-off-by: Kir Kolyshkin <[email protected]>
Rename CGROUP_PATH to CGROUP_V2_PATH so it is more clear that it can
only be used for CGROUP_V2, and to resolve ambiguity with CGROUP_PATH
variable used in tests/rootless.sh.

Signed-off-by: Kir Kolyshkin <[email protected]>
We were not running localrootlessintegration test on CentOS Stream 9
because of some failures fixed by previous commits.

Enable rootless integration with both systemd and fs drivers.

Signed-off-by: Kir Kolyshkin <[email protected]>
Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM

@AkihiroSuda AkihiroSuda merged commit 0b9d545 into opencontainers:main Jun 10, 2023
@kolyshkin kolyshkin added backport/1.1-done A PR in main branch which has been backported to release-1.1 and removed backport/1.1-todo A PR in main branch which needs to be backported to release-1.1 labels Jun 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ci backport/1.1-done A PR in main branch which has been backported to release-1.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants