Skip to content

Commit

Permalink
always include 3rd argument for onStageEnded hook
Browse files Browse the repository at this point in the history
refs DE-372

[canvas-builds-refspec=fc746b7045171fb46e3c11275eedc91f2612f35e]

Change-Id: I73cd0ee9e85e786acdfa46e83b5399801b40b0b4
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/279839
Reviewed-by: Andrea Cirulli <[email protected]>
Tested-by: Service Cloud Jenkins <[email protected]>
QA-Review: Aaron Ogata <[email protected]>
Product-Review: Aaron Ogata <[email protected]>
  • Loading branch information
aogata-inst committed Nov 30, 2021
1 parent 91c10df commit 89a3495
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion Jenkinsfile
Original file line number Diff line number Diff line change
Expand Up @@ -381,7 +381,7 @@ pipeline {
]

def postBuildHandler = [
onStageEnded: { stageName, stageConfig ->
onStageEnded: { stageName, stageConfig, result ->
buildSummaryReport.addFailureRun('Main Build', currentBuild)
postFn(stageConfig.status())
}
Expand Down
2 changes: 1 addition & 1 deletion Jenkinsfile.selenium.flakey_spec_catcher
Original file line number Diff line number Diff line change
Expand Up @@ -200,7 +200,7 @@ pipeline {
script {
extendedStage('Runner').nodeRequirements(label: 'canvas-docker').obeysAllowStages(false).execute {
def postBuildHandler = [
onStageEnded: { stageName, stageConfig ->
onStageEnded: { stageName, stageConfig, result ->
cleanupFn(stageConfig.status())
}
]
Expand Down

0 comments on commit 89a3495

Please sign in to comment.