diff options
author | Ed Santiago <santiago@redhat.com> | 2021-02-02 14:36:19 -0700 |
---|---|---|
committer | Ed Santiago <santiago@redhat.com> | 2021-02-02 15:23:48 -0700 |
commit | e0bc8ffb5932ce93a40b8a58f7f0a253db98a588 (patch) | |
tree | 55b59eb314bb9a3494c1acba7bd0c42e64791170 /test/system/140-diff.bats | |
parent | d1e0afdd47c1868ce0d22e48ae9b0b9d89ba6274 (diff) | |
download | podman-e0bc8ffb5932ce93a40b8a58f7f0a253db98a588.tar.gz podman-e0bc8ffb5932ce93a40b8a58f7f0a253db98a588.tar.bz2 podman-e0bc8ffb5932ce93a40b8a58f7f0a253db98a588.zip |
Gating tests: diff test: workaround for RHEL8 failure
RHEL8 rootless gating tests are inconsistently failing with:
$ podman diff --format json -l
#
{"changed":["/etc"],"added":["/sys/fs","/sys/fs/cgroup","/pMOm1Q0fnN"],"deleted":["/etc/services"]}
# #/vvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvv
# #| FAIL: added
# #| expected: '/pMOm1Q0fnN'
# #| actual: '/sys/fs'
# #| > '/sys/fs/cgroup'
# #| > '/pMOm1Q0fnN'
# #\^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Reason: PR #8561, I think (something to do with /sys on RHEL).
Workaround: ignore '/sys/fs' in diffs.
Signed-off-by: Ed Santiago <santiago@redhat.com>
Diffstat (limited to 'test/system/140-diff.bats')
-rw-r--r-- | test/system/140-diff.bats | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/test/system/140-diff.bats b/test/system/140-diff.bats index 1277f9bbe..02b3a86ca 100644 --- a/test/system/140-diff.bats +++ b/test/system/140-diff.bats @@ -25,7 +25,12 @@ load helpers ) for field in ${!expect[@]}; do - result=$(jq -r -c ".${field}[]" <<<"$output") + # ARGH! The /sys/fs kludgery is for RHEL8 rootless, which mumble mumble + # does some sort of magic muckery with /sys - I think the relevant + # PR is https://github.com/containers/podman/pull/8561 + # Anyhow, without the egrep below, this test fails about 50% of the + # time on rootless RHEL8. (No, I don't know why it's not 100%). + result=$(jq -r -c ".${field}[]" <<<"$output" | egrep -v '^/sys/fs') is "$result" "${expect[$field]}" "$field" done |