stop running post-merge builds for translations
refs DE-1351 flag=none TEST PLAN: Build Change-Id: I85bb0c77820551bbbaf12e3d46b4b664897dcb64 Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/299194 Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com> Reviewed-by: Aaron Ogata <aogata@instructure.com> Build-Review: Aaron Ogata <aogata@instructure.com> QA-Review: Bobby Buten <bobby.buten@instructure.com> Product-Review: Bobby Buten <bobby.buten@instructure.com>
This commit is contained in:
parent
ef38342df3
commit
5d63e48131
|
@ -332,6 +332,11 @@ pipeline {
|
|||
script {
|
||||
lock(label: 'canvas_build_global_mutex', quantity: 1) {
|
||||
timeout(60) {
|
||||
// Skip translation builds for patchsets uploaded by svc.cloudjenkins
|
||||
if (env.GERRIT_PATCHSET_UPLOADER_EMAIL == 'svc.cloudjenkins@instructure.com' && env.GERRIT_CHANGE_SUBJECT =~ /translation$/) {
|
||||
return
|
||||
}
|
||||
|
||||
node('master') {
|
||||
// For builds like Rails 6.1 prototype, we want to be able to see the build link, but
|
||||
// not have Gerrit vote on it. This isn't currently supported through the Gerrit Trigger
|
||||
|
@ -343,12 +348,6 @@ pipeline {
|
|||
gerrit.submitReview("", "Build Started ${RUN_DISPLAY_URL}")
|
||||
}
|
||||
|
||||
// Skip builds for patchsets uploaded by svc.cloudjenkins, these are usually translation updates.
|
||||
if (env.GERRIT_PATCHSET_UPLOADER_EMAIL == 'svc.cloudjenkins@instructure.com' && !configuration.isChangeMerged()) {
|
||||
currentBuild.result = 'ABORTED'
|
||||
error('No pre-merge builds for Service Cloud Jenkins user.')
|
||||
}
|
||||
|
||||
if (configuration.skipCi()) {
|
||||
currentBuild.result = 'NOT_BUILT'
|
||||
gerrit.submitLintReview('-2', 'Build not executed due to [skip-ci] flag')
|
||||
|
|
Loading…
Reference in New Issue