Skip to content

Commit

Permalink
Add back slack notification; Only trigger when not a workflow_call
Browse files Browse the repository at this point in the history
  • Loading branch information
snf2ye committed Feb 7, 2023
1 parent 974cb87 commit 73b115a
Show file tree
Hide file tree
Showing 2 changed files with 20 additions and 2 deletions.
2 changes: 0 additions & 2 deletions .github/workflows/dev-image-update.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -108,8 +108,6 @@ jobs:
status: ${{ job.status }}
fields: repo,message,commit,author,action,eventName,ref,workflow,job,took
author_name: Integration Test
mention: fb,muscles
if_mention: failure
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
SLACK_WEBHOOK_URL: ${{ secrets.SLACK_WEBHOOK_URL }}
Expand Down
20 changes: 20 additions & 0 deletions .github/workflows/helmtagbumper.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,16 @@ jobs:
GITHUB_REPO: datarepo-helm-definitions
SWITCH_DIRECTORIES: true
MERGE_BRANCH: master
- name: "Notify Slack"
if: ${{ github.event_name != 'workflow_call' && always() }} # workflow_call has own slack notification
uses: broadinstitute/action-slack@v3.8.0
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
SLACK_WEBHOOK_URL: ${{ secrets.SLACK_WEBHOOK_URL }}
with:
status: ${{ job.status }}
fields: job,repo,message,author,took
author_name: "[API] [datarepo-helm-definitions] Version update for Integration namespaces"
datarepo_helm_chart_update:
runs-on: ubuntu-20.04
steps:
Expand Down Expand Up @@ -101,3 +111,13 @@ jobs:
GITHUB_REPO: datarepo-helm
SWITCH_DIRECTORIES: true
MERGE_BRANCH: master
- name: "Notify Slack"
if: ${{ github.event_name != 'workflow_call' && always() }} # workflow_call has own slack notification
uses: broadinstitute/action-slack@v3.8.0
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
SLACK_WEBHOOK_URL: ${{ secrets.SLACK_WEBHOOK_URL }}
with:
status: ${{ job.status }}
fields: job,repo,message,author,took
author_name: "[API] [datarepo-helm] Version update for Helm Charts"

0 comments on commit 73b115a

Please sign in to comment.