diff options
author | Jon Marius Venstad <venstad@gmail.com> | 2021-10-27 13:53:49 +0200 |
---|---|---|
committer | Jon Marius Venstad <venstad@gmail.com> | 2021-10-27 13:53:49 +0200 |
commit | 4bb849bb8c208dfbe5a72f23ad2166f231617632 (patch) | |
tree | aa4d2e1efd0d0f0939007614abd3bc53ea32f970 | |
parent | 37d050d026ae1cdbcb1ba2dd4848107fa8d6fd61 (diff) |
Check also that the first failing run has in fact ended
-rw-r--r-- | controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/DeploymentStatus.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/DeploymentStatus.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/DeploymentStatus.java index 8981ad4e6db..68f5147516f 100644 --- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/DeploymentStatus.java +++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/DeploymentStatus.java @@ -573,7 +573,7 @@ public class DeploymentStatus { public Optional<Instant> coolingDownUntil(Change change) { if (job.lastTriggered().isEmpty()) return Optional.empty(); if (job.lastCompleted().isEmpty()) return Optional.empty(); - if (job.firstFailing().isEmpty()) return Optional.empty(); + if (job.firstFailing().isEmpty() || ! job.firstFailing().get().hasEnded()) return Optional.empty(); Versions lastVersions = job.lastCompleted().get().versions(); if (change.platform().isPresent() && ! change.platform().get().equals(lastVersions.targetPlatform())) return Optional.empty(); if (change.application().isPresent() && ! change.application().get().equals(lastVersions.targetApplication())) return Optional.empty(); |