Skip to content

Commit

Permalink
Merge pull request containers#15894 from edsantiago/reenable_some_aar…
Browse files Browse the repository at this point in the history
…ch64_tests

System tests: reenable some skipped aarch64 tests
  • Loading branch information
openshift-merge-robot committed Sep 22, 2022
2 parents ecaefee + d5f044e commit a49aa13
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 9 deletions.
1 change: 0 additions & 1 deletion test/system/030-run.bats
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@
load helpers

@test "podman run - basic tests" {
skip_if_aarch64 "FIXME: #15074 - fails on aarch64"
rand=$(random_string 30)

err_no_such_cmd="Error:.*/no/such/command.*[Nn]o such file or directory"
Expand Down
2 changes: 0 additions & 2 deletions test/system/075-exec.bats
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,6 @@
load helpers

@test "podman exec - basic test" {
skip_if_aarch64 "FIXME: #15074 - fails on aarch64"

rand_filename=$(random_string 20)
rand_content=$(random_string 50)

Expand Down
3 changes: 0 additions & 3 deletions test/system/260-sdnotify.bats
Original file line number Diff line number Diff line change
Expand Up @@ -142,7 +142,6 @@ READY=1" "sdnotify sent MAINPID and READY"
# These tests can fail in dev. environment because of SELinux.
# quick fix: chcon -t container_runtime_exec_t ./bin/podman
@test "sdnotify : container" {
skip_if_aarch64 "FIXME: #15277 sdnotify doesn't work on aarch64"
# Sigh... we need to pull a humongous image because it has systemd-notify.
# (IMPORTANT: fedora:32 and above silently removed systemd-notify; this
# caused CI to hang. That's why we explicitly require fedora:31)
Expand Down Expand Up @@ -248,8 +247,6 @@ READY=1" "sdnotify sent MAINPID and READY"
}

@test "sdnotify : play kube - with policies" {
skip_if_aarch64 "FIXME: #15277 sdnotify doesn't work on aarch64"

# Sigh... we need to pull a humongous image because it has systemd-notify.
# (IMPORTANT: fedora:32 and above silently removed systemd-notify; this
# caused CI to hang. That's why we explicitly require fedora:31)
Expand Down
3 changes: 0 additions & 3 deletions test/system/410-selinux.bats
Original file line number Diff line number Diff line change
Expand Up @@ -39,12 +39,10 @@ function check_label() {
}

@test "podman selinux: container with label=disable" {
skip_if_aarch64 "FIXME: #15074 - fails on aarch64"
check_label "--security-opt label=disable" "spc_t"
}

@test "podman selinux: privileged container" {
skip_if_aarch64 "FIXME: #15074 - fails on aarch64"
check_label "--privileged --userns=host" "spc_t"
}

Expand All @@ -65,7 +63,6 @@ function check_label() {
}

@test "podman selinux: pid=host" {
skip_if_aarch64 "FIXME: #15074 - fails on aarch64"
# FIXME this test fails when run rootless with runc:
# Error: container_linux.go:367: starting container process caused: process_linux.go:495: container init caused: readonly path /proc/asound: operation not permitted: OCI permission denied
if is_rootless; then
Expand Down

0 comments on commit a49aa13

Please sign in to comment.