Skip to content

Add slack notifications#134

Open
donoghuc wants to merge 3 commits into
1.xfrom
add-slack-notifications
Open

Add slack notifications#134
donoghuc wants to merge 3 commits into
1.xfrom
add-slack-notifications

Conversation

@donoghuc

Copy link
Copy Markdown
Contributor

Update shared workflow to notify a (soon to be created based on the outcome of PR review here) slack channel when a shared workflow fails.

NOTE: this assumes that we will add a SLACK_BOT_TOKEN to allow pushing messages to slack which consumers for the workflow will explicitly inherit. I'm open to discussing other alternative to how to manage that secret.

jobs:
integration-tests:
name: Integration Tests / ES ${{ matrix.elastic-stack-version }} ${{ matrix.snapshot && '(Snapshot)' || '' }}
name: Integration Tests / Stack ${{ matrix.elastic-stack-version }} ${{ matrix.snapshot && '(Snapshot)' || '' }}

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Oops, i did not notice #133 has been targetted on the old feature branch. I'll just include it in this as we are still in pilot phase. Going to delete that branch so we are no longer confused.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants