Skip to content

Commit 0eaec1a

Browse files
committed
cirrus: check for system test leaks in nightly
The leak check is slower (over 5mins) so we do not wnat them on PR runs to speed system tests up. However that opens the door for someone to add a test which forgets to do the correct cleanup themselves. This might not cause a fatal error right away and only later when new tests would be added. To prevent this happening the nighlty run will check leaks so that we can fix them quickly and not notice them months/years later when a new test is added that might trip over it. Signed-off-by: Paul Holzinger <[email protected]>
1 parent 4e0cd49 commit 0eaec1a

File tree

2 files changed

+8
-2
lines changed

2 files changed

+8
-2
lines changed

contrib/cirrus/lib.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -94,7 +94,7 @@ EPOCH_TEST_COMMIT="$CIRRUS_BASE_SHA"
9494
# are consumed by the passthrough_envars() automation library function.
9595
#
9696
# List of envariables which must be EXACT matches
97-
PASSTHROUGH_ENV_EXACT='CGROUP_MANAGER|DEST_BRANCH|DISTRO_NV|GOCACHE|GOPATH|GOSRC|NETWORK_BACKEND|OCI_RUNTIME|PODMAN_IGNORE_CGROUPSV1_WARNING|ROOTLESS_USER|SCRIPT_BASE|SKIP_USERNS|EC2_INST_TYPE|PODMAN_DB|STORAGE_FS'
97+
PASSTHROUGH_ENV_EXACT='CGROUP_MANAGER|DEST_BRANCH|DISTRO_NV|GOCACHE|GOPATH|GOSRC|NETWORK_BACKEND|OCI_RUNTIME|PODMAN_IGNORE_CGROUPSV1_WARNING|ROOTLESS_USER|SCRIPT_BASE|SKIP_USERNS|EC2_INST_TYPE|PODMAN_DB|STORAGE_FS|PODMAN_BATS_LEAK_CHECK'
9898

9999
# List of envariable patterns which must match AT THE BEGINNING of the name.
100100
# Consumed by the passthrough_envars() automation library function.

contrib/cirrus/setup_environment.sh

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -365,7 +365,13 @@ case "$TEST_FLAVOR" in
365365
int)
366366
showrun make .install.ginkgo
367367
;&
368-
sys) ;&
368+
sys)
369+
# when run nighlty check for system test leaks
370+
# shellcheck disable=SC2154
371+
if [[ "$CIRRUS_CRON" != '' ]]; then
372+
export PODMAN_BATS_LEAK_CHECK=1
373+
fi
374+
;&
369375
upgrade_test) ;&
370376
bud) ;&
371377
bindings) ;&

0 commit comments

Comments
 (0)