Pass in status of job from calling action step

Correct branch for jade-data-ui
DataBiosphere · Jan 11, 2024 · a52e5d8 · a52e5d8
1 parent e1855d5
commit a52e5d8
Showing 5 changed files with 14 additions and 3 deletions.
diff --git a/.github/workflows/dev-image-update.yaml b/.github/workflows/dev-image-update.yaml
@@ -102,6 +102,8 @@ jobs:
     needs: [ update_image, helm_tag_bump, cherry_pick_image_to_production_gcr ]
     uses: ./.github/workflows/notify-slack.yaml
     secrets: inherit
+    if: ${{ !cancelled() }}
     with:
       workflow_name: Dev Image Update
       notify_on_success: true
+      success: ${{ needs.update_image.result == 'success' && needs.helm_tag_bump.result == 'success' && needs.cherry_pick_image_to_production_gcr.result == 'success' }}
diff --git a/.github/workflows/helmtagbump.yaml b/.github/workflows/helmtagbump.yaml
@@ -113,8 +113,10 @@ jobs:
     needs: [ integration_helm_tag_update, ui_helm_default_chart_tag ]
     uses: ./.github/workflows/notify-slack.yaml
     secrets: inherit
+    if: ${{ !cancelled() }}
     with:
       workflow_name: Helm Tag Bump
       notify_on_failure: ${{ inputs.notify-slack }}
       notify_on_success: ${{ inputs.notify-slack }}
+      success: ${{ needs.integration_helm_tag_update.result == 'success' && needs.ui_helm_default_chart_tag.result == 'success' }}
 
diff --git a/.github/workflows/notify-slack.yaml b/.github/workflows/notify-slack.yaml
@@ -5,6 +5,9 @@ env:
 on:
     workflow_call:
       inputs:
+        success:
+          required: true
+          type: boolean
         workflow_name:
           required: true
           type: string
@@ -27,10 +30,10 @@ on:
 jobs:
     notify-slack:
         runs-on: ubuntu-latest
-        if: ${{ !cancelled() && github.ref == 'refs/heads/develop' }}
+        if: ${{ github.ref == 'refs/heads/develop' }}
         steps:
           - name: Notify slack on failure
-            if: inputs.notify_on_failure && failure()
+            if: ${{ inputs.notify_on_failure && !inputs.success }}
             uses: slackapi/slack-github-action@v1.24.0
             with:
               channel-id: ${{ inputs.failure_slack_channel_id }}
@@ -55,7 +58,7 @@ jobs:
             env:
               SLACK_BOT_TOKEN: ${{ env.SLACK_BOT_TOKEN}}
           - name: Notify slack on Success
-            if: inputs.notify_on_success && success()
+            if: ${{ inputs.notify_on_success && inputs.success }}
             uses: slackapi/slack-github-action@v1.24.0
             with:
               channel-id: ${{ inputs.success_slack_channel_id }}

diff --git a/.github/workflows/test-e2e.yml b/.github/workflows/test-e2e.yml
@@ -118,8 +118,10 @@ jobs:
     needs: [e2e_test]
     uses: ./.github/workflows/notify-slack.yaml
     secrets: inherit
+    if: ${{ !cancelled() }}
     with:
       workflow_name: Nightly E2E Tests
       notify_on_success: true
       failure_slack_channel_id: "CMYTGJVFY,C53JYBV9A" # #jade-alerts & #dsde-qa
       success_slack_channel_id: "CK9M0ENRJ,C53JYBV9A" # #jade-spam & #dsde-qa
+      success: ${{ needs.e2e_test.result == 'success' }}
diff --git a/.github/workflows/test-unit.yml b/.github/workflows/test-unit.yml
@@ -37,6 +37,8 @@ jobs:
     needs: [ unit_test ]
     uses: ./.github/workflows/notify-slack.yaml
     secrets: inherit
+    if: ${{ !cancelled() }}
     with:
       workflow_name: Nightly Unit Tests
       notify_on_success: true
+      success: ${{ needs.unit_test.result == 'success' }}