Skip to content

Slack notification on failed Github Action executions on the master branch #5111

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
arm4b merged 27 commits into from
Jan 15, 2021
Merged

Slack notification on failed Github Action executions on the master branch #5111

Changes from all commits
Commits
Show all changes
27 commits
Select commit Hold shift + click to select a range
a2eceac
Add Slack notifications to the github actions workflow
winem Dec 26, 2020
970c355
Remove tag filter to run actions
winem Dec 26, 2020
d7898b3
Trigger Slack notifications only for failures on the master branch
winem Dec 30, 2020
57a3db4
Slack notification is triggered on failures not on success
winem Dec 30, 2020
197f140
Fix expression on the slack notifications for gh actions
winem Dec 30, 2020
a23ab67
Fix typo on the master reference for the slack notifications
winem Dec 30, 2020
4c01bb8
Test new slack action
winem Dec 31, 2020
e03f8a4
Test slack notification delayed by needs
winem Dec 31, 2020
973f031
Test slack notification with seperated job
winem Dec 31, 2020
f5c42bd
Use ${{}} syntax for job status
winem Dec 31, 2020
6423484
Add missing closing
winem Dec 31, 2020
62e65e0
Break lint checks to test notifications on failures
winem Jan 1, 2021
ab3fc46
Remove always from slack gha notification
winem Jan 1, 2021
2a98560
Debug ci job status
winem Jan 1, 2021
eff5093
Switch to workflow conclusion
winem Jan 1, 2021
42e0453
Always run the workflow conclusion
winem Jan 1, 2021
97accca
Always run the slack job
winem Jan 1, 2021
1edc1f0
Drop whole debug and info messages and no longer needed outputs
winem Jan 1, 2021
6571981
Fix lint checks to pass the tests again
winem Jan 1, 2021
76ca3cd
Add the tags filter again and change the slack settings to match the …
winem Jan 3, 2021
6fc8eab
Revert "Add the tags filter again and change the slack settings to ma…
winem Jan 3, 2021
b7cfb45
Revert "Fix lint checks to pass the tests again" to break the lint ch…
winem Jan 3, 2021
74cb78f
Switch back to the inital slack container for a shorter and more prec…
winem Jan 3, 2021
b0248cc
Trigger Slack Notification for master builds only
winem Jan 4, 2021
92af07e
Revert "Revert "Fix lint checks to pass the tests again" to break the…
winem Jan 4, 2021
177e378
Add the tags filter again
winem Jan 4, 2021
43fe605
Use the 'development' channel for GH Actions Slack notifications
Jan 14, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 18 additions & 0 deletions .github/workflows/ci.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -173,6 +173,24 @@ jobs:
if: "${{ success() && ((env.TASK == 'ci-unit') || (env.TASK == 'ci-integration')) && (env.ENABLE_COVERAGE == 'yes') }}"
run: |
./scripts/travis/submit-codecov-coverage.sh
slack-notification:
name: Slack notification for failed master builds
if: always()
needs: ci
runs-on: ubuntu-latest
steps:
- name: Workflow conclusion
# this step creates an environment variable WORKFLOW_CONCLUSION and is the most reliable way to check the status of previous jobs
uses: technote-space/workflow-conclusion-action@v2
- name: CI Run Failure Slack Notification
if: ${{ env.WORKFLOW_CONCLUSION == 'failure' && github.ref == 'refs/heads/master' }}
Copy link
Member

@arm4b arm4b Jan 14, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

env:
SLACK_BOT_TOKEN: ${{ secrets.SLACK_BOT_TOKEN }}
uses: voxmedia/github-action-slack-notify-build@v1
with:
channel: development
status: FAILED
color: danger

# HELPER FOR FUTURE DEVELOPERS:
# If your GitHub Actions job is failing and you need to debug it, by default there is
Expand Down