Skip to content

Commit

Permalink
Merge pull request #11798 from giuseppe/skip-pid-shared-ns-on-rootles…
Browse files Browse the repository at this point in the history
…s-cgroups-v1

test: skip test on rootless cgroupsv1
  • Loading branch information
openshift-merge-robot authored Sep 30, 2021
2 parents 831842e + 5c1b3e8 commit 966b603
Show file tree
Hide file tree
Showing 3 changed files with 17 additions and 3 deletions.
4 changes: 4 additions & 0 deletions test/system/410-selinux.bats
Original file line number Diff line number Diff line change
Expand Up @@ -113,6 +113,10 @@ function check_label() {
@test "podman selinux: shared context in (some) namespaces" {
skip_if_no_selinux

# rootless users have no usable cgroups with cgroupsv1, so containers
# must use a pid namespace and not join an existing one.
skip_if_rootless_cgroupsv1

run_podman run -d --name myctr $IMAGE top
run_podman exec myctr cat -v /proc/self/attr/current
context_c1="$output"
Expand Down
4 changes: 1 addition & 3 deletions test/system/420-cgroups.bats
Original file line number Diff line number Diff line change
Expand Up @@ -8,9 +8,7 @@ load helpers
@test "podman run, preserves initial --cgroup-manager" {
skip_if_remote "podman-remote does not support --cgroup-manager"

if is_rootless && is_cgroupsv1; then
skip "not supported as rootless under cgroups v1"
fi
skip_if_rootless_cgroupsv1

# Find out our default cgroup manager, and from that, get the non-default
run_podman info --format '{{.Host.CgroupManager}}'
Expand Down
12 changes: 12 additions & 0 deletions test/system/helpers.bash
Original file line number Diff line number Diff line change
Expand Up @@ -428,6 +428,18 @@ function skip_if_cgroupsv1() {
fi
}

######################
# skip_if_rootless_cgroupsv1 # ...with an optional message
######################
function skip_if_rootless_cgroupsv1() {
if is_rootless; then
if ! is_cgroupsv2; then
local msg=$(_add_label_if_missing "$1" "rootless cgroupvs1")
skip "${msg:-not supported as rootless under cgroupsv1}"
fi
fi
}

##################################
# skip_if_journald_unavailable # rhbz#1895105: rootless journald permissions
##################################
Expand Down

0 comments on commit 966b603

Please sign in to comment.