Skip to content

Commit

Permalink
fix(script, jenkins): put results of properties files in its own map …
Browse files Browse the repository at this point in the history
…in addition to context (spinnaker#1480)
  • Loading branch information
tomaslin authored Jul 25, 2017
1 parent 502b63c commit c2e3ebb
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -85,6 +85,7 @@ public class WaitOnJobCompletion extends AbstractCloudProviderAwareTask implemen
throw new IllegalStateException("expected properties file ${stage.context.propertyFile} but one was not found or was empty")
}
outputs << properties
outputs.propertyFileContents = properties
}

return
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -61,6 +61,7 @@ class MonitorJenkinsJobTask implements RetryableTask {
def buildNumber = (int) stage.context.buildNumber
try {
Map<String, Object> build = buildService.getBuild(buildNumber, master, job)
Map outputs = [:]
String result = build.result
if ((build.building && build.building != 'false') || (build.running && build.running != 'false')) {
return new TaskResult(ExecutionStatus.RUNNING, [buildInfo: build])
Expand All @@ -70,6 +71,8 @@ class MonitorJenkinsJobTask implements RetryableTask {
build.artifacts = buildArtifactFilter.filterArtifacts(build.artifacts as List<Map>)
}

outputs.buildInfo = build

if (statusMap.containsKey(result)) {
ExecutionStatus status = statusMap[result]
Map<String, Object> properties = [:]
Expand All @@ -78,11 +81,13 @@ class MonitorJenkinsJobTask implements RetryableTask {
if (properties.size() == 0 && result == 'SUCCESS') {
throw new IllegalStateException("expected properties file ${stage.context.propertyFile} but one was not found or was empty")
}
outputs << properties
outputs.propertyFileContents = properties
}
if (result == 'UNSTABLE' && stage.context.markUnstableAsSuccessful) {
status = ExecutionStatus.SUCCEEDED
}
return new TaskResult(status, [buildInfo: build] + properties, [buildInfo: build] + properties)
return new TaskResult(status, outputs, outputs)
} else {
return new TaskResult(ExecutionStatus.RUNNING, [buildInfo: build])
}
Expand Down

0 comments on commit c2e3ebb

Please sign in to comment.