Merge pull request #3 from ankorstore/fix/wait-for-jobs

fix: fix wait-for-job and alter its defaults
ankorstore · Mar 9, 2023 · ed5706d · ed5706d
2 parents 95d7d82 + 064ced4
commit ed5706d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/commands/wait-for-job.yml b/src/commands/wait-for-job.yml
@@ -13,7 +13,7 @@ parameters:
       For multiple jobs the list must be comma separated.
     type: string
   continue-on-timeout:
-    default: true
+    default: false
     description: If true and wait time exceeds the max wait time, completes and lets future steps continue
     type: boolean
   max-wait-time:

diff --git a/src/scripts/wait-for-job.sh b/src/scripts/wait-for-job.sh
@@ -21,7 +21,7 @@ get_job_status() {
   local WORKFLOW_JOBS_JSON="/tmp/aks/wf_$CIRCLE_WORKFLOW_ID.json"
   curl  -f -s --retry 3 "$WORKFLOW_JOBS_URL" > "$WORKFLOW_JOBS_JSON"
   STATUS=$(jq -r ".items[] | select(.name==\"$NAME_OF_JOB\") | .status | values" "$WORKFLOW_JOBS_JSON")
-  NUMBER=$(jq -r ".items[] | select(.name==\"$NAME_OF_JOB\") | .JOB_NUMBER | values" "$WORKFLOW_JOBS_JSON")
+  NUMBER=$(jq -r ".items[] | select(.name==\"$NAME_OF_JOB\") | .job_number | values" "$WORKFLOW_JOBS_JSON")
   echo "$STATUS $NUMBER"
 }