Skip to content

ci: Update nightly E2E notifications#9177

Merged
thanethomson merged 3 commits intomainfrom
thane/slack-nightly-e2e
Aug 6, 2022
Merged

ci: Update nightly E2E notifications#9177
thanethomson merged 3 commits intomainfrom
thane/slack-nightly-e2e

Conversation

@thanethomson
Copy link
Contributor

Updates our nightly E2E notifications to use the incoming webhook approach, targeting our engineering team channel.

Signed-off-by: Thane Thomson <connect@thanethomson.com>
@thanethomson thanethomson requested a review from ebuchman as a code owner August 5, 2022 21:54
@thanethomson thanethomson requested a review from a team August 5, 2022 21:54
@@ -47,12 +47,27 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Notify Slack on failure
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we add a 'success' block to this workflow? Or, maybe it's present and I'm not seeing it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh there wasn't a success block for v0.34. I can easily add one though.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've added one now 👍

Signed-off-by: Thane Thomson <connect@thanethomson.com>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
@thanethomson thanethomson merged commit 1148759 into main Aug 6, 2022
@thanethomson thanethomson deleted the thane/slack-nightly-e2e branch August 6, 2022 01:38
samricotta pushed a commit that referenced this pull request Aug 9, 2022
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.

3 participants