commit c4b4b5edc3e9fdacb4cd20a9a9019780ce2c7825
author Arthur She <arthur.she@linaro.org> Fri Feb 09 21:28:27 2024 -0800
committer Arthur She <arthur.she@linaro.org> Fri Feb 09 21:34:38 2024 -0800
tree 4bcfbb79936a3bd0b1d2a87a76477300c911fbc5
parent 6067c526eb11ab93ca060205698dc2a2071d41fe

tf-a-lts-docs-watcher.sh: Fix timing issue

When a new lts tag is created, it needs some time to let RTD
aware of it. So, checking the version status in a loop, once
RTD knows it, activate it right away

Signed-off-by: Arthur She <arthur.she@linaro.org>
Change-Id: Ibab3247fd303101e091b26459a2fe46a1045df66

scripts/tf-a-lts-docs-watcher.sh

diff --git a/scripts/tf-a-lts-docs-watcher.sh b/scripts/tf-a-lts-docs-watcher.sh
index 26b9c78..654c03e 100755
--- a/scripts/tf-a-lts-docs-watcher.sh
+++ b/scripts/tf-a-lts-docs-watcher.sh
@@ -7,7 +7,7 @@
 echo "########################################################################"
 
 RTD_WEBHOOK_URL="https://readthedocs.org/api/v2/webhook/trustedfirmware-a/87181/"
-RTD_VER_API="https://readthedocs.org/api/v3/projects/trustedfirmware-a/versions/"
+RTD_VER_API="https://readthedocs.org/api/v3/projects/trustedfirmware-a/versions"
 RTD_WEBHOOK_SECRET_KEY=${RTD_WEBHOOK_SECRET}
 RTD_API_TOKEN=${RTD_API_TOKEN}
 
@@ -18,19 +18,29 @@
 
 function activate_version() {
     version=$1
+    max_retry_time=20
     retry=0
 
-    status=$(curl -s -X PATCH -H "Content-Type: application/json" -H "Authorization: Token ${RTD_API_TOKEN}" \
-                -d "{\"active\": true}" ${RTD_VER_API}/${version}/ | jq .detail)
+    ver_status=$(curl -s -H "Authorization: Token ${RTD_API_TOKEN}" ${RTD_VER_API}/${version}/ | \
+                 jq -r '.detail')
 
-    while [ "${status}" = "\"Not found.\"" ];
+    while [ "${ver_status}" == "Not found." ];
     do
-        [ ${retry} -gt 5 ] && echo "RTD can not find the version: ${new_tag}" && exit 1
-        sleep 10
-	    status=$(curl -s -X PATCH -H "Content-Type: application/json" -H "Authorization: Token ${RTD_API_TOKEN}" \
-                 -d "{\"active\": true}" ${RTD_VER_API}/${version}/ | jq .detail)
+        [ ${retry} -gt ${max_retry_time} ] && break 
+        sleep 30
         retry=$((retry+1))
+        ver_status=$(curl -s -H "Authorization: Token ${RTD_API_TOKEN}" ${RTD_VER_API}/${version}/ | \
+                     jq -r '.detail')
     done
+
+    if [ ${retry} -le ${max_retry_time} ]; then
+        echo "Active new version: ${version}"
+        curl -s -X PATCH -H "Content-Type: application/json" -H "Authorization: Token ${RTD_API_TOKEN}" \
+             -d "{\"active\": true, \"hidden\": false}" ${RTD_VER_API}/${version}/
+    else
+        echo "RTD can not find the version: ${version}"
+        exit 1
+    fi
 }
 
 echo "Notifying ReadTheDocs of changes on: ${lts_branch}"