ylqjgm
/
canvas-lms
mirror of https://github.com/instructure/canvas-lms.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

ensure all success / failure messages use summary url 

refs DE-551

Change-Id: I6e93c39275c9fd92a400ce0b0cafdbda8c6b1a57
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/260782
Reviewed-by: James Butters <jbutters@instructure.com>
Reviewed-by: Andrea Cirulli <andrea.cirulli@instructure.com>
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
QA-Review: Aaron Ogata <aogata@instructure.com>
Product-Review: Aaron Ogata <aogata@instructure.com>
This commit is contained in:
Aaron Ogata 2021-03-16 07:52:49 -07:00
parent d69aa5ab29
commit f48c4fe2d3
1 changed files with 8 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
Jenkinsfile vendored
View File

@ -50,6 +50,10 @@ def jenkinsFiles = [
'build/new-jenkins/*'
]
def getSummaryUrl() {
return "${env.BUILD_URL}/build-summary-report"
}
def getDockerWorkDir() {
return env.GERRIT_PROJECT == "canvas-lms" ? "/usr/src/app" : "/usr/src/app/gems/plugins/${env.GERRIT_PROJECT}"
}
@ -129,7 +133,7 @@ def postFn(status) {
if(timedStage.isAllowStagesFilterUsed()) {
node('master') {
gerrit.submitVerified("-1", "Build Failed\n\n$BUILD_URL/build-summary-report/")
gerrit.submitVerified("-1", "Build Failed\n\n${getSummaryUrl()}")
}
}
} else if(status == 'SUCCESS') {
@ -137,7 +141,7 @@ def postFn(status) {
if(timedStage.isAllowStagesFilterUsed()) {
node('master') {
gerrit.submitVerified("+1", "Build Successful\n\n$BUILD_URL/build-summary-report/")
gerrit.submitVerified("+1", "Build Successful\n\n${getSummaryUrl()}")
}
}
}
@ -178,7 +182,7 @@ def maybeSlackSendFailure() {
slackSend(
channel: getSlackChannel(),
color: 'danger',
message: "${authorSegment}. Build <${env.BUILD_URL}|#${env.BUILD_NUMBER}>\n\n$extra"
message: "${authorSegment}. Build <${getSummaryUrl()}|#${env.BUILD_NUMBER}>\n\n$extra"
)
}
}
@ -188,7 +192,7 @@ def maybeSlackSendSuccess() {
slackSend(
channel: getSlackChannel(),
color: 'good',
message: "Patchset <${env.GERRIT_CHANGE_URL}|#${env.GERRIT_CHANGE_NUMBER}> succeeded on re-trigger. Build <${env.BUILD_URL}|#${env.BUILD_NUMBER}>"
message: "Patchset <${env.GERRIT_CHANGE_URL}|#${env.GERRIT_CHANGE_NUMBER}> succeeded on re-trigger. Build <${getSummaryUrl()}|#${env.BUILD_NUMBER}>"
)
}
}