Cirrus: Fix skipping all/most tests

The originally intent for skipping tests based on change-content was to
optimize the PR workflow.  However, a mistake in a conditional is
causing almost all tasks running for Cron and branches to be skipped.
Fix this by checking for an empty '$CIRRUS_PR' variable.  This value is
always empty when operating outside of PRs.

Signed-off-by: Chris Evich <cevich@redhat.com>
This commit is contained in:
Chris Evich 2022-04-26 11:04:53 -04:00
parent e7edf30a1b
commit 1a9f110b59
No known key found for this signature in database
GPG Key ID: 03EDC70FD578067F
1 changed files with 3 additions and 2 deletions

View File

@ -373,8 +373,9 @@ dotest() {
function _bail_if_test_can_be_skipped() {
local head base diffs
# Cirrus sets these for PRs but not cron. In cron, we never want to skip.
for v in CIRRUS_CHANGE_IN_REPO DEST_BRANCH; do
# 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
if [[ -z "${!v}" ]]; then
msg "[ _cannot do selective skip: \$$v is undefined ]"
return 0