Merge pull request #7934 from Agoric/mfig-ci-notifications

ci: send test failure notifications when not a PR
Agoric · Aug 7, 2023 · ac1c923 · ac1c923
2 parents 6d5f5ff + 9aea7b4
commit ac1c923
Show file tree

Hide file tree

Showing 9 changed files with 220 additions and 6 deletions.
diff --git a/.github/actions/notify-status/action.yml b/.github/actions/notify-status/action.yml
@@ -4,13 +4,13 @@ description: 'Notify on actions status'
 inputs:
   from:
     description: 'The email address to send the notification from'
-    required: true
+    required: false
   to:
     description: 'Recipients'
-    required: true
+    required: false
   password:
     description: 'Sendgrid Password'
-    required: true
+    required: false
   webhook:
     description: 'Slack webhook URL'
     required: true
@@ -19,7 +19,8 @@ runs:
   using: composite
   steps:
     - name: send ${{ job.status }} email
-      uses: dawidd6/action-send-mail@v2
+      if: ${{ inputs.to != '' }}
+      uses: dawidd6/action-send-mail@v3
       with:
         # mail server settings
         server_address: smtp.sendgrid.net
@@ -42,7 +43,7 @@ runs:
         from: ${{ inputs.from }}
     - name: Send GitHub trigger payload
       id: slack
-      uses: slackapi/slack-github-action@v1.18.0
+      uses: slackapi/slack-github-action@v1.24.0
       with:
         payload: |
           {

diff --git a/.github/workflows/test-all-packages.yml b/.github/workflows/test-all-packages.yml
diff --git a/.github/workflows/test-dapp-card-store.yml b/.github/workflows/test-dapp-card-store.yml
@@ -71,3 +71,12 @@ jobs:
       - name: yarn test in dapp
         run: yarn test
         working-directory: ./dapp
+
+      - name: notify on failure
+        if: failure() && github.event_name != 'pull_request'
+        uses: ./.github/actions/notify-status
+        with:
+          webhook: ${{ secrets.SLACK_WEBHOOK_URL }}
+          from: ${{ secrets.NOTIFY_EMAIL_FROM }}
+          to: ${{ secrets.NOTIFY_EMAIL_TO }}
+          password: ${{ secrets.NOTIFY_EMAIL_PASSWORD }}
diff --git a/.github/workflows/test-dapp-fungible-faucet.yml.DISABLED b/.github/workflows/test-dapp-fungible-faucet.yml.DISABLED
@@ -72,3 +72,12 @@ jobs:
       - name: yarn test in dapp
         run: yarn test
         working-directory: ./dapp
+
+      - name: notify on failure
+        if: failure() && github.event_name != 'pull_request'
+        uses: ./.github/actions/notify-status
+        with:
+          webhook: ${{ secrets.SLACK_WEBHOOK_URL }}
+          from: ${{ secrets.NOTIFY_EMAIL_FROM }}
+          to: ${{ secrets.NOTIFY_EMAIL_TO }}
+          password: ${{ secrets.NOTIFY_EMAIL_PASSWORD }}
diff --git a/.github/workflows/test-dapp-otc.yml b/.github/workflows/test-dapp-otc.yml
@@ -71,3 +71,12 @@ jobs:
       - name: yarn test in dapp
         run: yarn test
         working-directory: ./dapp
+
+      - name: notify on failure
+        if: failure() && github.event_name != 'pull_request'
+        uses: ./.github/actions/notify-status
+        with:
+          webhook: ${{ secrets.SLACK_WEBHOOK_URL }}
+          from: ${{ secrets.NOTIFY_EMAIL_FROM }}
+          to: ${{ secrets.NOTIFY_EMAIL_TO }}
+          password: ${{ secrets.NOTIFY_EMAIL_PASSWORD }}
diff --git a/.github/workflows/test-dapp-pegasus.yml b/.github/workflows/test-dapp-pegasus.yml
@@ -71,3 +71,12 @@ jobs:
       - name: yarn test in dapp
         run: yarn test
         working-directory: ./dapp
+
+      - name: notify on failure
+        if: failure() && github.event_name != 'pull_request'
+        uses: ./.github/actions/notify-status
+        with:
+          webhook: ${{ secrets.SLACK_WEBHOOK_URL }}
+          from: ${{ secrets.NOTIFY_EMAIL_FROM }}
+          to: ${{ secrets.NOTIFY_EMAIL_TO }}
+          password: ${{ secrets.NOTIFY_EMAIL_PASSWORD }}
diff --git a/.github/workflows/test-dapp-simple-exchange.yml b/.github/workflows/test-dapp-simple-exchange.yml
@@ -71,3 +71,12 @@ jobs:
       - name: yarn test in dapp
         run: yarn test
         working-directory: ./dapp
+
+      - name: notify on failure
+        if: failure() && github.event_name != 'pull_request'
+        uses: ./.github/actions/notify-status
+        with:
+          webhook: ${{ secrets.SLACK_WEBHOOK_URL }}
+          from: ${{ secrets.NOTIFY_EMAIL_FROM }}
+          to: ${{ secrets.NOTIFY_EMAIL_TO }}
+          password: ${{ secrets.NOTIFY_EMAIL_PASSWORD }}
diff --git a/.github/workflows/test-dapp-treasury.yml.DISABLED b/.github/workflows/test-dapp-treasury.yml.DISABLED
@@ -71,3 +71,12 @@ jobs:
       - name: yarn test in dapp
         run: yarn test
         working-directory: ./dapp
+
+      - name: notify on failure
+        if: failure() && github.event_name != 'pull_request'
+        uses: ./.github/actions/notify-status
+        with:
+          webhook: ${{ secrets.SLACK_WEBHOOK_URL }}
+          from: ${{ secrets.NOTIFY_EMAIL_FROM }}
+          to: ${{ secrets.NOTIFY_EMAIL_TO }}
+          password: ${{ secrets.NOTIFY_EMAIL_PASSWORD }}
diff --git a/.github/workflows/test-documentation.yml b/.github/workflows/test-documentation.yml
@@ -71,3 +71,12 @@ jobs:
       - name: yarn test in dapp
         run: yarn test
         working-directory: ./dapp
+
+      - name: notify on failure
+        if: failure() && github.event_name != 'pull_request'
+        uses: ./.github/actions/notify-status
+        with:
+          webhook: ${{ secrets.SLACK_WEBHOOK_URL }}
+          from: ${{ secrets.NOTIFY_EMAIL_FROM }}
+          to: ${{ secrets.NOTIFY_EMAIL_TO }}
+          password: ${{ secrets.NOTIFY_EMAIL_PASSWORD }}