diff options
author | Martin Polden <mpolden@mpolden.no> | 2017-08-29 14:25:01 +0200 |
---|---|---|
committer | Martin Polden <mpolden@mpolden.no> | 2017-08-29 14:25:01 +0200 |
commit | 1aba6bd53e92db8349a8934b6743ad5b7a2b9c0a (patch) | |
tree | ac984d2eaec926ce6a28580e4798f41c919f8ce1 /controller-server | |
parent | 1f1f0ff74c512b9b92f2fbf087f4323486ecbd59 (diff) |
Make delayed deployer ignore applications with no delay steps
Diffstat (limited to 'controller-server')
-rw-r--r-- | controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/DeploymentTrigger.java | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/DeploymentTrigger.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/DeploymentTrigger.java index 3cd70c1ec46..ce1fee4dd14 100644 --- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/DeploymentTrigger.java +++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/DeploymentTrigger.java @@ -119,6 +119,9 @@ public class DeploymentTrigger { if ( ! application.deploying().isPresent() ) continue; if (application.deploymentJobs().hasFailures()) continue; if (application.deploymentJobs().inProgress()) continue; + if (application.deploymentSpec().steps().stream().noneMatch(step -> step instanceof DeploymentSpec.Delay)) { + continue; // Application does not have any delayed deployments + } Optional<JobStatus> lastSuccessfulJob = application.deploymentJobs().jobStatus().values() .stream() @@ -131,7 +134,7 @@ public class DeploymentTrigger { try (Lock lock = applications().lock(application.id())) { application = applications().require(application.id()); application = trigger(nextAfter(lastSuccessfulJob.get().type(), application), application, - "Delayed by deployment spec", lock); + "Resuming delayed deployment", lock); applications().store(application, lock); } } |