Skip to content

[installer-tests] Add slack webhook for test failures #11749

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.

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
merged 1 commit into from
Aug 10, 2022
Merged

Conversation

nandajavarma
Copy link
Contributor

Description

This PR adds slack webhook alerting capabilities to the nightly tests. Currently if any of the setup fails, alert goes to #self-hosted-jobs channel on slack and if any of the integration tests fails it goes to the corresponding team's job channel (currently only workspace and IDE).

Related Issue(s)

Fixes #11273

How to test

You can run any tests like in this internal doc to check the alerts

Release Notes

NONE

Documentation

Werft options:

  • /werft with-preview

@nandajavarma nandajavarma requested a review from a team July 29, 2022 13:56
@github-actions github-actions bot added the team: delivery Issue belongs to the self-hosted team label Jul 29, 2022
@roboquat roboquat added size/XL and removed size/L labels Jul 29, 2022
@nandajavarma
Copy link
Contributor Author

/hold waiting to confirm with teams before sending out alerts

@roboquat roboquat added size/L and removed size/XL labels Aug 2, 2022
@roboquat roboquat added size/XL and removed size/L labels Aug 3, 2022
@nandajavarma nandajavarma force-pushed the nvn/test-alerts branch 5 times, most recently from ee12f8b to 6e8beaa Compare August 10, 2022 13:12
@nandajavarma
Copy link
Contributor Author

/unhold

@andrew-farries
Copy link
Contributor

/hold as this is blocking the merge pool.

It looks like it needs rebased onto main, to pick up the gofmt changes: #11958

@nandajavarma
Copy link
Contributor Author

Sorry about that, @andrew-farries ! Rebased and builds are passing. unholding! /unhold

@andrew-farries
Copy link
Contributor

/unhold

@roboquat roboquat merged commit a3c81b6 into main Aug 10, 2022
@roboquat roboquat deleted the nvn/test-alerts branch August 10, 2022 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note-none size/XL team: delivery Issue belongs to the self-hosted team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Visualisation for automated (self-hosted) failed tests for other teams
4 participants