Fix logic for discovering new branches

When a new branch exists on gerrit, but has never been seen by RTD
we need to trigger a job so that the branch is discovered
we can then mark it active in the following step.

API changed under out feet, where a 404 was returned before
we now get a 200 with the string null.
Change code to explicitly match the returned string of
"null"

Issue-Id: RELENG-3027
Signed-off-by: Aric Gardner <agardner@linuxfoundation.org>
Change-Id: I769b66384f5b53d87527faeb0f467350a847b65c

diff --git a/releasenotes/notes/rtd-activate-version-652c0d9730b936b1.yaml b/releasenotes/notes/rtd-activate-version-652c0d9730b936b1.yaml
new file mode 100644 (file)
index 0000000..5db6225
--- /dev/null
+++ b/releasenotes/notes/rtd-activate-version-652c0d9730b936b1.yaml
@@ -0,0 +1,11 @@
+---
+fixes:
+  - |
+    Fixed logic for discovering new branches
+    When a new branch exists on gerrit, but has never been seen by RTD
+    we need to trigger a job so that the branch is discovered
+    we can then mark it active in the following step.
+    API changed under out feet, where a 404 was returned before
+    we now get a 200 with the string null.
+    Change code to explicitly match the returned string of
+    "null"

diff --git a/shell/rtdv3.sh b/shell/rtdv3.sh
index 7b4fef8..f5be6e6 100644 (file)
--- a/shell/rtdv3.sh
+++ b/shell/rtdv3.sh
@@ -130,8 +130,8 @@ echo "INFO: Performing merge action"
     branch=$(echo "$GERRIT_BRANCH" | tr '[:upper:]' '[:lower:]')
     echo "INFO: Checking if read the docs has seen branch $branch"
 
-    #if this is 404. then run discover branch
-    if ! lftools rtd project-version-details "$rtdproject" "$branch" | jq '.active'; then
+    #if this is 200 "null" Then run discover branch
+    if [[ $(lftools rtd project-version-details "$rtdproject" "$branch" | jq '.active') == "null" ]]; then
       echo "INFO: read the docs has not seen branch $branch for project $rtdproject"
       echo "INFO: triggering $rtdproject latest to instantiate new branch discovery"
       watchbuild latest