From 326ef19d942c8922ebba68e96d870d5117f67472 Mon Sep 17 00:00:00 2001
From: Chris Evich <cevich@redhat.com>
Date: Wed, 30 Oct 2019 10:35:20 -0400
Subject: Cirrus: Fix upload_release_archive on branch or tag

Cirrus-CI only sets `$CIRRUS_BASE_SHA` when testing PRs since the
destination and it's state is easy to discover.  However, when
post-merge and/or tag-push testing, the previous state is not
easily discoverable (changes have already merged).  The
`upload_release_archive` script incorrectly assumed this
variable was always set, causing a constant stream of post-merge
testing failures.

Tweak the `is_release()` function to properly handle an empty
`$CIRRUS_BASE_SHA` whether or not `$CIRRUS_TAG` is also set.  Also
update the unit-tests to check for this.  Also account for a
corner case where hack/get_ci_vm.sh is running on a VM w/o git.

Signed-off-by: Chris Evich <cevich@redhat.com>
---
 contrib/cirrus/lib.sh | 46 +++++++++++++++++++++++++++++-----------------
 1 file changed, 29 insertions(+), 17 deletions(-)

(limited to 'contrib/cirrus/lib.sh')

diff --git a/contrib/cirrus/lib.sh b/contrib/cirrus/lib.sh
index 051157702..297ed49ce 100644
--- a/contrib/cirrus/lib.sh
+++ b/contrib/cirrus/lib.sh
@@ -238,34 +238,46 @@ ircmsg() {
 # there is at least one release tag not having any '-' characters (return 0)
 # or otherwise (return non-0).
 is_release() {
-    req_env_var CIRRUS_BASE_SHA CIRRUS_CHANGE_IN_REPO
-    local range="${CIRRUS_BASE_SHA}..${CIRRUS_CHANGE_IN_REPO}"
-    # Easy check first, default non-useful values
-    if echo "${range}$CIRRUS_TAG" | grep -iq 'unknown'; then
-        die 11 "is_release() unusable range ${range} or tag $CIRRUS_TAG"
-    fi
-    # Next easy check, is CIRRUS_TAG set
     unset RELVER
+    local ret
+    req_env_var CIRRUS_CHANGE_IN_REPO
     if [[ -n "$CIRRUS_TAG" ]]; then
         RELVER="$CIRRUS_TAG"
-    else # Lastly, look through the range for tags
-        git fetch --all --tags &> /dev/null|| \
-            die 12 "is_release() failed to fetch tags"
-        RELVER=$(git log --pretty='format:%d' $range | \
-                 grep '(tag:' | sed -r -e 's/\s+[(]tag:\s+(v[0-9].*)[)]/\1/' | \
-                 sort -uV | tail -1)
-        [[ "$?" -eq "0" ]] || \
+    elif [[ ! "$CIRRUS_BASE_SHA" =~ "unknown" ]]
+    then
+        # Normally not possible for this to be empty, except when unittesting.
+        req_env_var CIRRUS_BASE_SHA
+        local range="${CIRRUS_BASE_SHA}..${CIRRUS_CHANGE_IN_REPO}"
+        if echo "${range}$CIRRUS_TAG" | grep -iq 'unknown'; then
+            die 11 "is_release() unusable range ${range} or tag $CIRRUS_TAG"
+        fi
+
+        if type -P git &> /dev/null
+        then
+            git fetch --all --tags &> /dev/null|| \
+                die 12 "is_release() failed to fetch tags"
+            RELVER=$(git log --pretty='format:%d' $range | \
+                     grep '(tag:' | sed -r -e 's/\s+[(]tag:\s+(v[0-9].*)[)]/\1/' | \
+                     sort -uV | tail -1)
+            ret=$?
+        else
+            warn -1 "Git command not found while checking for release"
+            ret="-1"
+        fi
+        [[ "$ret" -eq "0" ]] || \
             die 13 "is_release() failed to parse tags"
+    else  # Not testing a PR, but neither CIRRUS_BASE_SHA or CIRRUS_TAG are set
+        return 1
     fi
-    echo "Found \$RELVER $RELVER"
     if [[ -n "$RELVER" ]]; then
+        echo "Found \$RELVER $RELVER"
         if echo "$RELVER" | grep -q '-'; then
-            return 2
+            return 2  # development tag
         else
             return 0
         fi
     else
-        return 1
+        return 1  # not a release
     fi
 }
 
-- 
cgit v1.2.3-54-g00ecf