Model Id is required

davidkyle · davidkyle · commit a7959ae997e0 · 2025-10-06T13:05:55.000+01:00
diff --git a/x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/services/cohere/CohereService.java b/x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/services/cohere/CohereService.java
@@ -378,7 +378,7 @@ public static InferenceServiceConfiguration get() {
                         "The name of the model to use for the inference task."
                     )
                         .setLabel("Model ID")
-                        .setRequired(false)
+                        .setRequired(true)
                         .setSensitive(false)
                         .setUpdatable(false)
                         .setType(SettingsConfigurationFieldType.STRING)
diff --git a/x-pack/plugin/inference/src/test/java/org/elasticsearch/xpack/inference/services/cohere/CohereServiceTests.java b/x-pack/plugin/inference/src/test/java/org/elasticsearch/xpack/inference/services/cohere/CohereServiceTests.java
@@ -1556,7 +1556,7 @@ public void testGetConfiguration() throws Exception {
                             "model_id": {
                                 "description": "The name of the model to use for the inference task.",
                                 "label": "Model ID",
-                                "required": false,
+                                "required": true,
                                 "sensitive": false,
                                 "updatable": false,
                                 "type": "str",

Original file line number	Diff line number	Diff line change
`@@ -378,7 +378,7 @@ public static InferenceServiceConfiguration get() {`
`378`	`378`	`"The name of the model to use for the inference task."`
`379`	`379`	`)`
`380`	`380`	`.setLabel("Model ID")`
`381`		`- .setRequired(false)`
	`381`	`+ .setRequired(true)`
`382`	`382`	`.setSensitive(false)`
`383`	`383`	`.setUpdatable(false)`
`384`	`384`	`.setType(SettingsConfigurationFieldType.STRING)`