Merge pull request #8210 from vespa-engine/jvenstad/deploy-to-dev-on-spec-major

Jvenstad/deploy to dev on spec major MERGEOK

2 files changed, 22 insertions, 1 deletions

diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/ApplicationController.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/ApplicationController.java
index 3185b7d5f45..4a8670b9f9e 100644
--- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/ApplicationController.java
+++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/ApplicationController.java
@@ -58,6 +58,7 @@ import com.yahoo.vespa.hosted.controller.rotation.RotationLock;
 import com.yahoo.vespa.hosted.controller.rotation.RotationRepository;
 import com.yahoo.vespa.hosted.controller.tenant.AthenzTenant;
 import com.yahoo.vespa.hosted.controller.tenant.Tenant;
+import com.yahoo.vespa.hosted.controller.versions.VespaVersion;
 import com.yahoo.vespa.hosted.rotation.config.RotationsConfig;
 import com.yahoo.yolean.Exceptions;
 
@@ -67,8 +68,10 @@ import java.net.URI;
 import java.time.Clock;
 import java.time.Duration;
 import java.time.Instant;
+import java.util.ArrayDeque;
 import java.util.Collections;
 import java.util.Comparator;
+import java.util.Deque;
 import java.util.EnumSet;
 import java.util.HashSet;
 import java.util.List;
@@ -277,7 +280,9 @@ public class ApplicationController {
             ApplicationVersion applicationVersion;
             ApplicationPackage applicationPackage;
             if (canDeployDirectly) {
-                platformVersion = options.vespaVersion.map(Version::new).orElse(controller.systemVersion());
+                platformVersion = options.vespaVersion.map(Version::new).orElse(application.get().deploymentSpec().majorVersion()
+                                                                                           .flatMap(this::lastCompatibleVersion)
+                                                                                           .orElse(controller.systemVersion()));
                 applicationVersion = applicationVersionFromDeployer.orElse(ApplicationVersion.unknown);
                 applicationPackage = applicationPackageFromDeployer.orElseThrow(
                         () -> new IllegalArgumentException("Application package must be given when deploying to " + zone));
@@ -727,6 +732,14 @@ public class ApplicationController {
                           });
     }
 
+    /** Returns the latest known version within the given major. */
+    private Optional<Version> lastCompatibleVersion(int targetMajorVersion) {
+        return controller.versionStatus().versions().stream()
+                         .map(VespaVersion::versionNumber)
+                         .filter(version -> version.getMajor() == targetMajorVersion)
+                         .max(naturalOrder());
+    }
+
     private boolean isUserDeployment(Optional<AthenzIdentity> identity) {
         return identity
                 .filter(id -> id instanceof AthenzUser)
diff --git a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/ControllerTest.java b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/ControllerTest.java
index 1199f0229b6..343deaec752 100644
--- a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/ControllerTest.java
+++ b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/ControllerTest.java
@@ -446,10 +446,13 @@ public class ControllerTest {
     @Test
     public void testDeployDirectly() {
         DeploymentTester tester = new DeploymentTester();
+        Version six = Version.fromString("6.1");
+        tester.upgradeSystem(six);
         tester.controllerTester().zoneRegistry().setSystemName(SystemName.cd);
         tester.controllerTester().zoneRegistry().setZones(ZoneId.from("prod", "cd-us-central-1"));
         ApplicationPackage applicationPackage = new ApplicationPackageBuilder()
                 .environment(Environment.prod)
+                .majorVersion(6)
                 .region("cd-us-central-1")
                 .build();
 
@@ -468,6 +471,11 @@ public class ControllerTest {
 
         assertTrue("No job status added",
                    tester.applications().require(app.id()).deploymentJobs().jobStatus().isEmpty());
+
+        Version seven = Version.fromString("7.2");
+        tester.upgradeSystem(seven);
+        tester.controller().applications().deploy(app.id(), zone, Optional.of(applicationPackage), options);
+        assertEquals(six, tester.application(app.id()).deployments().get(zone).version());
     }
 
     @Test