Skip to content

[QA] slack notify on failure #68126

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
wayneseymour merged 23 commits into from
Jun 5, 2020
Merged

[QA] slack notify on failure #68126

Changes from all commits
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
4beec54
Test notify on failure
wayneseymour Jun 3, 2020
f4607c2
Test notify on failure
wayneseymour Jun 3, 2020
47e3713
Test notify on failure
wayneseymour Jun 3, 2020
3ce3062
First attempt at sending the slack msg.
wayneseymour Jun 3, 2020
4c7872d
Merge branch 'master' of github.com:elastic/kibana into qa-slack-send…
wayneseymour Jun 3, 2020
3e0782d
Only send the msg on failure.
wayneseymour Jun 3, 2020
0e7351f
Induce a msg send, with refactor.
wayneseymour Jun 3, 2020
87b796f
Turn all the logic back on.
wayneseymour Jun 3, 2020
b11c30c
Induce syntax error to see how the sys responds.
wayneseymour Jun 3, 2020
cad9099
Use catch Errors instead and induce the error.
wayneseymour Jun 3, 2020
697ef89
Change msg
wayneseymour Jun 3, 2020
450df78
Add fail h
wayneseymour Jun 3, 2020
f9f539f
Add timestamp
wayneseymour Jun 3, 2020
d0e959a
Add timestamp, again
wayneseymour Jun 3, 2020
2e7486f
Print params.
wayneseymour Jun 3, 2020
349811e
Add ts.
wayneseymour Jun 3, 2020
6dbc723
Drop ts
wayneseymour Jun 3, 2020
26237d2
Drop the forced error.
wayneseymour Jun 3, 2020
4ada19f
Merge branch 'master' of github.com:elastic/kibana into qa-slack-send…
wayneseymour Jun 4, 2020
3d2e211
Simplify, per cr. Thanks CJO!
wayneseymour Jun 4, 2020
0a42bbe
Induce error
wayneseymour Jun 4, 2020
e073190
Merge branch 'master' of github.com:elastic/kibana into qa-slack-send…
wayneseymour Jun 4, 2020
10fefa9
Drop induced error.
wayneseymour Jun 5, 2020
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
16 changes: 14 additions & 2 deletions .ci/Jenkinsfile_coverage
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,8 +11,11 @@ kibanaPipeline(timeoutMinutes: 240) {
'CODE_COVERAGE=1', // Enables coverage. Needed for multiple ci scripts, such as remote.ts, test/scripts/*.sh, schema.js, etc.
]) {
workers.base(name: 'coverage-worker', size: 'l', ramDisk: false, bootstrapped: false) {
kibanaCoverage.runTests()
handleIngestion(TIME_STAMP)
catchError {
kibanaCoverage.runTests()
handleIngestion(TIME_STAMP)
}
handleFail()
}
}
kibanaPipeline.sendMail()
Expand All @@ -29,4 +32,13 @@ def handleIngestion(timestamp) {
kibanaCoverage.uploadCoverageStaticSite(timestamp)
}

def handleFail() {
def buildStatus = buildUtils.getBuildStatus()
if(params.NOTIFY_ON_FAILURE && buildStatus != 'SUCCESS' && buildStatus != 'ABORTED') {
slackNotifications.sendFailedBuild(
channel: '#kibana-qa',
username: 'Kibana QA'
)
}
}