fix: adding condition check failing part for post cmd (#119)

* fix: adding condition check failing part * fix: add another condition to handle empty number of failing * refactor: remove unused qoutes --------- Co-authored-by: Muhammad Luthfi Fahlevi <muhammad.fahlevi@gojek.com>
goto · Oct 18, 2024 · 6361b97 · 6361b97
1 parent 87e4af7
commit 6361b97
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/stable/meteor/templates/cronjob.yaml b/stable/meteor/templates/cronjob.yaml
@@ -32,10 +32,16 @@ spec:
               - /bin/sh
               - -c
               - |
-                meteor run /opt/recipes;
-                {{- if .Values.postCmd.enabled }}
+                OUTPUT=$(meteor run /opt/recipes;)
                 METEOR_EXIT_CODE=$?
-                if [ $METEOR_EXIT_CODE -eq 0 ]; then
+                {{- if .Values.postCmd.enabled }}
+                NUM_FAILING=$(echo $OUTPUT | grep -o '^[0-9]* failing' | awk '{print $1}')
+                # if the output does not contains number of failing, then it is failed
+                if [ -z $NUM_FAILING ]; then
+                  NUM_FAILING=-1
+                fi
+                # run post cmd only if exit code is success and no any failing occurred in meteor job
+                if [ $METEOR_EXIT_CODE -eq 0 ] && [ $NUM_FAILING -eq 0 ]; then
                   {{- .Values.postCmd.cmd | nindent 18 }}
                 fi
                 {{- end }}