Allocation consistency between model versions

4 files changed, 28 insertions, 20 deletions

diff --git a/config-model-api/src/main/java/com/yahoo/config/application/api/ApplicationFile.java b/config-model-api/src/main/java/com/yahoo/config/application/api/ApplicationFile.java
index bcc8d222ca5..0384a5c7a1c 100644
--- a/config-model-api/src/main/java/com/yahoo/config/application/api/ApplicationFile.java
+++ b/config-model-api/src/main/java/com/yahoo/config/application/api/ApplicationFile.java
@@ -11,7 +11,7 @@ import java.util.List;
  * An application file represents a file within an application package. This class can be used to traverse the entire
  * application package file structure, as well as read and write files to it, and create directories.
  *
- * @author Ulf Lillengen
+ * @author Ulf Lilleengen
  */
 public abstract class ApplicationFile implements Comparable<ApplicationFile> {
 
diff --git a/config-model-api/src/main/java/com/yahoo/config/application/api/ApplicationPackage.java b/config-model-api/src/main/java/com/yahoo/config/application/api/ApplicationPackage.java
index 795e87b7690..23bbdd511b7 100644
--- a/config-model-api/src/main/java/com/yahoo/config/application/api/ApplicationPackage.java
+++ b/config-model-api/src/main/java/com/yahoo/config/application/api/ApplicationPackage.java
@@ -228,10 +228,22 @@ public interface ApplicationPackage {
         throw new UnsupportedOperationException("This application package cannot write its metadata");
     }
 
+    /** 
+     * Returns the single host allocation info of this, or an empty map if no allocation is available 
+     * 
+     * @deprecated please use #getProvisionInfo
+     */
+    // TODO: Remove on Vespa 7
+    @Deprecated
     default Map<Version, ProvisionInfo> getProvisionInfoMap() {
         return Collections.emptyMap();
     }
 
+    /** Returns the host allocation info of this, or empty if no allocation is available */
+    default Optional<ProvisionInfo> getProvisionInfo() {
+        return Optional.empty();
+    }
+
     default Map<Version, FileRegistry> getFileRegistryMap() {
         return Collections.emptyMap();
     }
diff --git a/config-model-api/src/main/java/com/yahoo/config/model/api/Model.java b/config-model-api/src/main/java/com/yahoo/config/model/api/Model.java
index 0c038077fe4..db589ef5d21 100644
--- a/config-model-api/src/main/java/com/yahoo/config/model/api/Model.java
+++ b/config-model-api/src/main/java/com/yahoo/config/model/api/Model.java
@@ -16,8 +16,7 @@ import java.util.Collection;
  * A {@link Model} represents the interface towards the model of an entire tenant, and defines methods
  * for querying this model.
  *
- * @author lulf
- * @since 5.1
+ * @author Ulf Lilleengen
  */
 public interface Model {
 
@@ -60,9 +59,20 @@ public interface Model {
 
     /**
      * Get the provisioning info for this model.
+     * 
      * @return {@link ProvisionInfo} instance, if available.
+     * @deprecated use provisionInfo
      */
-    Optional<ProvisionInfo> getProvisionInfo();
+    @Deprecated
+    // TODO: Remove this (and the implementation below) when no version older than 6.142 is deployed anywhere
+    default Optional<ProvisionInfo> getProvisionInfo() {
+        return Optional.of(provisionInfo());
+    }
+
+    @SuppressWarnings("deprecation")
+    default ProvisionInfo provisionInfo() {
+        return getProvisionInfo().get();
+    }
 
     /**
      * Returns whether this application allows serving config request for a different version.
@@ -71,11 +81,6 @@ public interface Model {
      */
     default boolean allowModelVersionMismatch(Instant now) { return false; }
 
-    /** @deprecated pass now. */
-    // TODO: Remove this when no version older than 6.115 is deployed anywhere
-    @Deprecated
-    default boolean allowModelVersionMismatch() { return allowModelVersionMismatch(Clock.systemUTC().instant()); }
-
     /**
      * Returns whether old config models should be loaded (default) or not.
      * Skipping old config models is a validation override which is useful when the old model
@@ -88,9 +93,4 @@ public interface Model {
      */
     default boolean skipOldConfigModels(Instant now) { return false; }
     
-    /** @deprecated pass now. */
-    // TODO: Remove this when no version older than 6.115 is deployed anywhere
-    @Deprecated 
-    default boolean skipOldConfigModels() { return skipOldConfigModels(Clock.systemUTC().instant()); }
-
 }
diff --git a/config-model-api/src/main/java/com/yahoo/config/model/api/ModelContext.java b/config-model-api/src/main/java/com/yahoo/config/model/api/ModelContext.java
index 5afc570d81b..5b79415c132 100644
--- a/config-model-api/src/main/java/com/yahoo/config/model/api/ModelContext.java
+++ b/config-model-api/src/main/java/com/yahoo/config/model/api/ModelContext.java
@@ -17,7 +17,7 @@ import java.util.Set;
 /**
  * Model context containing state provided to model factories.
  *
- * @author lulf
+ * @author Ulf Lilleengen
  */
 public interface ModelContext {
 
@@ -31,11 +31,6 @@ public interface ModelContext {
     Properties properties();
     default Optional<File> appDir() { return Optional.empty();}
     
-    /** @deprecated TODO: Remove this when no config models older than 6.98 are used */
-    @SuppressWarnings("unused")
-    @Deprecated
-    default Optional<com.yahoo.config.provision.Version> vespaVersion() { return Optional.empty(); }
-    
     /** The Vespa version this model is built for */
     Version modelVespaVersion();
     
@@ -50,4 +45,5 @@ public interface ModelContext {
         Zone zone();
         Set<Rotation> rotations();
     }
+
 }