Merge "Use double bracket conditional compound command"
authorAnil Belur <abelur@linuxfoundation.org>
Tue, 14 Jul 2020 00:58:13 +0000 (00:58 +0000)
committerGerrit Code Review <gerrit@linuxfoundation.org>
Tue, 14 Jul 2020 00:58:13 +0000 (00:58 +0000)
releasenotes/notes/lf-rtdv3-jobs-809e51d2bf6794d8.yaml [new file with mode: 0644]
shell/rtdv3.sh

diff --git a/releasenotes/notes/lf-rtdv3-jobs-809e51d2bf6794d8.yaml b/releasenotes/notes/lf-rtdv3-jobs-809e51d2bf6794d8.yaml
new file mode 100644 (file)
index 0000000..3c887b5
--- /dev/null
@@ -0,0 +1,4 @@
+---
+fixes:
+  - |
+    Fix quoting bug in rtdv3.sh script that caused failures to exit improperly.
index 23d373d..08f98a2 100644 (file)
@@ -18,7 +18,7 @@ watchbuild(){
   buildid=$(lftools rtd project-build-trigger "$rtdproject" "$1" | jq '.build.id')
 
   result=null
-  while [ $result == null ]; do
+  while [[ "$result" == null ]]; do
     sleep 10
     result=$(lftools rtd project-build-details "$rtdproject" "$buildid"  | jq '.success')
     echo "INFO Current result of running build $result"