Merge pull request #28236 from vespa-engine/lesters/warning-missing-onnx-model

Add warning when trying to configure missing ONNX model

1 files changed, 7 insertions, 2 deletions

diff --git a/config-model/src/main/java/com/yahoo/vespa/model/container/xml/ContainerModelBuilder.java b/config-model/src/main/java/com/yahoo/vespa/model/container/xml/ContainerModelBuilder.java
index 80b676159cb..e7692aeee7b 100644
--- a/config-model/src/main/java/com/yahoo/vespa/model/container/xml/ContainerModelBuilder.java
+++ b/config-model/src/main/java/com/yahoo/vespa/model/container/xml/ContainerModelBuilder.java
@@ -755,8 +755,13 @@ public class ContainerModelBuilder extends ConfigModelBuilder<ContainerModel> {
         Element modelsElement = XML.getChild(onnxElement, "models");
         for (Element modelElement : XML.getChildren(modelsElement, "model") ) {
             OnnxModel onnxModel = profiles.getOnnxModels().asMap().get(modelElement.getAttribute("name"));
-            if (onnxModel == null)
-                continue; // Skip if model is not found
+            if (onnxModel == null) {
+                String availableModels = String.join(", ", profiles.getOnnxModels().asMap().keySet());
+                context.getDeployState().getDeployLogger().logApplicationPackage(WARNING,
+                        "Model '" + modelElement.getAttribute("name") + "' not found. Available ONNX " +
+                        "models are: " + availableModels + ". Skipping this configuration.");
+                continue;
+            }
             onnxModel.setStatelessExecutionMode(getStringValue(modelElement, "execution-mode", null));
             onnxModel.setStatelessInterOpThreads(getIntValue(modelElement, "interop-threads", -1));
             onnxModel.setStatelessIntraOpThreads(getIntValue(modelElement, "intraop-threads", -1));