Skip to content

Commit

Permalink
Merge pull request #10237 from edsantiago/ci_rootless_add_ubuntu
Browse files Browse the repository at this point in the history
CI: run rootless tests under ubuntu
  • Loading branch information
openshift-merge-robot authored May 7, 2021
2 parents 663ea96 + 95a50a9 commit 29b1317
Show file tree
Hide file tree
Showing 4 changed files with 22 additions and 17 deletions.
4 changes: 2 additions & 2 deletions .cirrus.yml
Original file line number Diff line number Diff line change
Expand Up @@ -539,7 +539,7 @@ rootless_integration_test_task:
skip: *branches_and_tags
depends_on:
- unit_test
matrix: *fedora_vm_axis
matrix: *platform_axis
gce_instance: *standardvm
timeout_in: 90m
env:
Expand Down Expand Up @@ -614,7 +614,7 @@ rootless_system_test_task:
only_if: *not_docs
depends_on:
- rootless_integration_test
matrix: *fedora_vm_axis
matrix: *platform_axis
gce_instance: *standardvm
env:
TEST_FLAVOR: sys
Expand Down
7 changes: 0 additions & 7 deletions test/e2e/common_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -605,13 +605,6 @@ func SkipIfRootlessCgroupsV1(reason string) {
}
}

func SkipIfUnprivilegedCPULimits() {
info := GetHostDistributionInfo()
if isRootless() && info.Distribution == "fedora" {
ginkgo.Skip("Rootless Fedora doesn't have permission to set CPU limits")
}
}

func SkipIfRootless(reason string) {
checkReason(reason)
if os.Geteuid() != 0 {
Expand Down
3 changes: 1 addition & 2 deletions test/e2e/play_kube_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -1999,8 +1999,7 @@ VOLUME %s`, ALPINE, hostPathDir+"/")

It("podman play kube allows setting resource limits", func() {
SkipIfContainerized("Resource limits require a running systemd")
SkipIfRootlessCgroupsV1("Limits require root or cgroups v2")
SkipIfUnprivilegedCPULimits()
SkipIfRootless("CPU limits require root")
podmanTest.CgroupManager = "systemd"

var (
Expand Down
25 changes: 19 additions & 6 deletions test/system/001-basic.bats
Original file line number Diff line number Diff line change
Expand Up @@ -10,16 +10,18 @@ function setup() {
:
}

@test "podman --context emits reasonable output" {
run_podman 125 --context=swarm version
is "$output" "Error: Podman does not support swarm, the only --context value allowed is \"default\"" "--context=default or fail"

run_podman --context=default version
}
#### DO NOT ADD ANY TESTS HERE! ADD NEW TESTS AT BOTTOM!

@test "podman version emits reasonable output" {
run_podman version

# FIXME FIXME FIXME: #10248: nasty message on Ubuntu cgroups v1, rootless
if [[ "$output" =~ "overlay test mount with multiple lowers failed" ]]; then
if is_rootless; then
lines=("${lines[@]:1}")
fi
fi

# First line of podman-remote is "Client:<blank>".
# Just delete it (i.e. remove the first entry from the 'lines' array)
if is_remote; then
Expand All @@ -41,6 +43,17 @@ function setup() {
}


@test "podman --context emits reasonable output" {
# All we care about here is that the command passes
run_podman --context=default version

# This one must fail
run_podman 125 --context=swarm version
is "$output" \
"Error: Podman does not support swarm, the only --context value allowed is \"default\"" \
"--context=default or fail"
}

@test "podman can pull an image" {
run_podman pull $IMAGE
}
Expand Down

0 comments on commit 29b1317

Please sign in to comment.