diff --git a/orca-clouddriver/src/main/groovy/com/netflix/spinnaker/orca/clouddriver/tasks/artifacts/FindArtifactFromExecutionTask.java b/orca-clouddriver/src/main/groovy/com/netflix/spinnaker/orca/clouddriver/tasks/artifacts/FindArtifactFromExecutionTask.java index fc2abdc7ff..0fc6f6e1b1 100644 --- a/orca-clouddriver/src/main/groovy/com/netflix/spinnaker/orca/clouddriver/tasks/artifacts/FindArtifactFromExecutionTask.java +++ b/orca-clouddriver/src/main/groovy/com/netflix/spinnaker/orca/clouddriver/tasks/artifacts/FindArtifactFromExecutionTask.java @@ -73,13 +73,18 @@ public TaskResult execute(@Nonnull Stage stage) { @Data private static class ExecutionOptions { + // Accept either 'succeeded' or 'successful' in the stage config. The front-end sets 'successful', but due to a bug + // this class was only looking for 'succeeded'. Fix this by accepting 'successful' but to avoid breaking anyone who + // discovered this bug and manually edited their stage to set 'succeeded', continue to accept 'succeeded'. boolean succeeded; + boolean successful; + boolean terminal; boolean running; ExecutionCriteria toCriteria() { List statuses = new ArrayList<>(); - if (succeeded) { + if (succeeded || successful) { statuses.add("SUCCEEDED"); }