diff options
author | jonmv <venstad@gmail.com> | 2023-10-05 16:59:32 +0200 |
---|---|---|
committer | jonmv <venstad@gmail.com> | 2023-10-05 16:59:32 +0200 |
commit | 48e3c14a3632ee61bb6716d149d1b741fa836f50 (patch) | |
tree | 0e323162a037548e48d1acf791f607f25533b667 /controller-server | |
parent | 45d5bef6ef5c722ebd9af66132bf82d488012ee2 (diff) |
Do not try to change revision when pinned
Diffstat (limited to 'controller-server')
-rw-r--r-- | controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/DeploymentTrigger.java | 1 |
1 files changed, 1 insertions, 0 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 1b40781fe0f..8297374f22e 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 @@ -430,6 +430,7 @@ public class DeploymentTrigger { private boolean acceptNewRevision(DeploymentStatus status, InstanceName instance, RevisionId revision) { if (status.application().deploymentSpec().instance(instance).isEmpty()) return false; // Unknown instance. + if (status.application().get(instance).map(Instance::change).map(Change::isRevisionPinned).orElse(false)) return false; if ( ! status.jobs().failingWithBrokenRevisionSince(revision, clock.instant().minus(maxFailingRevisionTime)) .isEmpty()) return false; // Don't deploy a broken revision. boolean isChangingRevision = status.application().require(instance).change().revision().isPresent(); |