diff options
| author | Ed Santiago <santiago@redhat.com> | 2022-05-09 07:40:12 -0600 |
|---|---|---|
| committer | Ed Santiago <santiago@redhat.com> | 2022-05-10 05:47:15 -0600 |
| commit | 4fe115094ccffad29d614024b9156f19907e682e (patch) | |
| tree | 90ba2bf8159bdf6e4bffeb54a6e81e26e1fd63da | |
| parent | a87bba317e403c69b38c65958f2121d7e3f8a58e (diff) | |
| download | podman-4fe115094ccffad29d614024b9156f19907e682e.tar.gz podman-4fe115094ccffad29d614024b9156f19907e682e.tar.bz2 podman-4fe115094ccffad29d614024b9156f19907e682e.zip | |
test skipper: check for $DEST_BRANCH
The test-skipping optimization is failing as rootless on non-main,
because $DEST_BRANCH is not set. Solution: check for envariable,
skip test if missing. (This was part of my original PR, but was
accidentally removed in #14013)
Also: DEST_BRANCH was silently being defaulted to 'v4.1' in lib.sh.
Remove that: per @cevich, it is no longer necessary.
Signed-off-by: Ed Santiago <santiago@redhat.com>
| -rw-r--r-- | contrib/cirrus/lib.sh | 1 | ||||
| -rwxr-xr-x | contrib/cirrus/runner.sh | 2 |
2 files changed, 1 insertions, 2 deletions
diff --git a/contrib/cirrus/lib.sh b/contrib/cirrus/lib.sh index 5d58042a5..46b245a45 100644 --- a/contrib/cirrus/lib.sh +++ b/contrib/cirrus/lib.sh @@ -74,7 +74,6 @@ PODMAN_SERVER_LOG=$CIRRUS_WORKING_DIR/server.log # Defaults when not running under CI export CI="${CI:-false}" CIRRUS_CI="${CIRRUS_CI:-false}" -DEST_BRANCH="${DEST_BRANCH:-v4.1}" CONTINUOUS_INTEGRATION="${CONTINUOUS_INTEGRATION:-false}" CIRRUS_REPO_NAME=${CIRRUS_REPO_NAME:-podman} # Cirrus only sets $CIRRUS_BASE_SHA properly for PRs, but $EPOCH_TEST_COMMIT diff --git a/contrib/cirrus/runner.sh b/contrib/cirrus/runner.sh index 101270703..c4a714691 100755 --- a/contrib/cirrus/runner.sh +++ b/contrib/cirrus/runner.sh @@ -375,7 +375,7 @@ function _bail_if_test_can_be_skipped() { # Cirrus sets these for PRs but not branches or cron. In cron and branches, #we never want to skip. - for v in CIRRUS_CHANGE_IN_REPO CIRRUS_PR; do + for v in CIRRUS_CHANGE_IN_REPO CIRRUS_PR DEST_BRANCH; do if [[ -z "${!v}" ]]; then msg "[ _cannot do selective skip: \$$v is undefined ]" return 0 |