diff options
2 files changed, 3 insertions, 12 deletions
diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/application/Change.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/application/Change.java index 0b50ec36a32..603f16a9ef6 100644 --- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/application/Change.java +++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/application/Change.java @@ -98,8 +98,4 @@ public final class Change { return new Change(Optional.of(platformChange), Optional.empty()); } - public static Change of(Version platformChange, ApplicationVersion applicationVersion) { - return new Change(Optional.of(platformChange), Optional.of(applicationVersion)); - } - } 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 70c1702d0b5..c768aea8248 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 @@ -86,14 +86,9 @@ public class DeploymentTrigger { if (report.success()) { if (report.jobType() == JobType.component) { if (acceptNewApplicationVersionNow(application)) { - // If there's an ongoing upgrade, we allow the upgrade and new application package to be - // deployed together - if (application.change().platform().isPresent()) { - application = application.withChange(Change.of(application.change().platform().get(), - applicationVersion)); - } else { - application = application.withChange(Change.of(applicationVersion)); - } + // Note that in case of an ongoing upgrade this may result in both the upgrade and application + // change being deployed together + application = application.withChange(application.change().with(applicationVersion)); } else { // postpone applications().store(application.withOutstandingChange(Change.of(applicationVersion))); |