diff options
author | Harald Musum <musum@yahooinc.com> | 2024-01-03 23:45:27 +0100 |
---|---|---|
committer | Harald Musum <musum@yahooinc.com> | 2024-01-03 23:45:27 +0100 |
commit | 7b84c88f35eb0abb4ae0eeed56239d9d3cada37d (patch) | |
tree | 44ceb64811836b8ad5adc4aa60b0a4a9ff151c5a /config-model/src | |
parent | 776ace0650b26fd6ba0e1452c81c139d7554834d (diff) |
Supply cluster id to OnnxModelCost
Diffstat (limited to 'config-model/src')
3 files changed, 5 insertions, 2 deletions
diff --git a/config-model/src/main/java/com/yahoo/vespa/model/container/ApplicationContainerCluster.java b/config-model/src/main/java/com/yahoo/vespa/model/container/ApplicationContainerCluster.java index 8c4adfb96cb..9896ca95e97 100644 --- a/config-model/src/main/java/com/yahoo/vespa/model/container/ApplicationContainerCluster.java +++ b/config-model/src/main/java/com/yahoo/vespa/model/container/ApplicationContainerCluster.java @@ -16,6 +16,7 @@ import com.yahoo.config.model.api.OnnxModelCost; import com.yahoo.config.model.deploy.DeployState; import com.yahoo.config.model.producer.TreeConfigProducer; import com.yahoo.config.provision.AllocatedHosts; +import com.yahoo.config.provision.ClusterSpec; import com.yahoo.config.provision.HostSpec; import com.yahoo.config.provision.TenantName; import com.yahoo.container.bundle.BundleInstantiationSpecification; @@ -139,7 +140,7 @@ public final class ApplicationContainerCluster extends ContainerCluster<Applicat : defaultHeapSizePercentageOfAvailableMemory; onnxModelCost = deployState.onnxModelCost(); onnxModelCostCalculator = deployState.onnxModelCost().newCalculator( - deployState.getApplicationPackage(), deployState.getProperties().applicationId()); + deployState.getApplicationPackage(), deployState.getProperties().applicationId(), ClusterSpec.Id.from(clusterId)); logger = deployState.getDeployLogger(); } diff --git a/config-model/src/test/java/com/yahoo/vespa/model/application/validation/JvmHeapSizeValidatorTest.java b/config-model/src/test/java/com/yahoo/vespa/model/application/validation/JvmHeapSizeValidatorTest.java index 31e4c661151..e4ba003df64 100644 --- a/config-model/src/test/java/com/yahoo/vespa/model/application/validation/JvmHeapSizeValidatorTest.java +++ b/config-model/src/test/java/com/yahoo/vespa/model/application/validation/JvmHeapSizeValidatorTest.java @@ -13,6 +13,7 @@ import com.yahoo.config.model.deploy.TestProperties; import com.yahoo.config.model.provision.InMemoryProvisioner; import com.yahoo.config.model.test.MockApplicationPackage; import com.yahoo.config.provision.ApplicationId; +import com.yahoo.config.provision.ClusterSpec; import com.yahoo.config.provision.NodeResources; import com.yahoo.text.Text; import com.yahoo.vespa.model.VespaModel; @@ -121,6 +122,7 @@ class JvmHeapSizeValidatorTest { ModelCostDummy(long modelCost) { this.modelCost = modelCost; } @Override public Calculator newCalculator(ApplicationPackage appPkg, ApplicationId applicationId) { return this; } + @Override public Calculator newCalculator(ApplicationPackage appPkg, ApplicationId applicationId, ClusterSpec.Id clusterId) { return this; } @Override public Map<String, ModelInfo> models() { return Map.of(); } @Override public void setRestartOnDeploy() {} @Override public boolean restartOnDeploy() { return false;} diff --git a/config-model/src/test/java/com/yahoo/vespa/model/application/validation/change/RestartOnDeployForOnnxModelChangesValidatorTest.java b/config-model/src/test/java/com/yahoo/vespa/model/application/validation/change/RestartOnDeployForOnnxModelChangesValidatorTest.java index 13389689de5..f2013f0f26b 100644 --- a/config-model/src/test/java/com/yahoo/vespa/model/application/validation/change/RestartOnDeployForOnnxModelChangesValidatorTest.java +++ b/config-model/src/test/java/com/yahoo/vespa/model/application/validation/change/RestartOnDeployForOnnxModelChangesValidatorTest.java @@ -79,7 +79,7 @@ public class RestartOnDeployForOnnxModelChangesValidatorTest { } private static OnnxModelCost onnxModelCost(long estimatedCost, long hash) { - return (appPkg, applicationId) -> new OnnxModelCost.Calculator() { + return (appPkg, applicationId, clusterId) -> new OnnxModelCost.Calculator() { private final Map<String, OnnxModelCost.ModelInfo> models = new HashMap<>(); private boolean restartOnDeploy = false; |