Merge pull request #23505 from vespa-engine/hmusum/fail-if-node-count-not-given-for-content-cluster-in-hosted

Fail deployment if node count is not given for content cluster in hosted

1 files changed, 31 insertions, 1 deletions

diff --git a/config-model/src/test/java/com/yahoo/config/model/provision/ModelProvisioningTest.java b/config-model/src/test/java/com/yahoo/config/model/provision/ModelProvisioningTest.java
index f5ff48c0c69..03ab4f46c60 100644
--- a/config-model/src/test/java/com/yahoo/config/model/provision/ModelProvisioningTest.java
+++ b/config-model/src/test/java/com/yahoo/config/model/provision/ModelProvisioningTest.java
@@ -39,7 +39,6 @@ import com.yahoo.vespa.model.test.VespaModelTester;
 import com.yahoo.vespa.model.test.utils.VespaModelCreatorWithMockPkg;
 import com.yahoo.yolean.Exceptions;
 import org.junit.Test;
-
 import java.io.StringReader;
 import java.util.ArrayList;
 import java.util.List;
@@ -1662,6 +1661,37 @@ public class ModelProvisioningTest {
         assertEquals(1, model.getAdmin().getClusterControllers().getContainers().size());
     }
 
+    @Test
+    public void testThatStandaloneSyntaxWithClusterControllerWorksOnHostedManuallyDeployed() {
+        String services =
+                "<?xml version='1.0' encoding='utf-8' ?>" +
+                        "<services>" +
+                        "   <container id='foo' version='1.0'>" +
+                        "      <nodes count=\"2\" />" +
+                        "   </container>" +
+                        "   <content id='bar' version='1.0'>" +
+                        "      <documents>" +
+                        "         <document type='type1' mode='index'/>" +
+                        "      </documents>" +
+                        "      <redundancy>1</redundancy>" +
+                        "      <nodes>" +
+                        "          <group>" +
+                        "            <node distribution-key='0' hostalias='node3'/>" +
+                        "          </group>" +
+                        "      </nodes>" +
+                        "   </content>" +
+                        "</services>";
+        VespaModelTester tester = new VespaModelTester();
+        tester.setHosted(true);
+        tester.addHosts(4);
+        try {
+            VespaModel model = tester.createModel(new Zone(Environment.staging, RegionName.from("us-central-1")), services, true);
+            fail("expected failure");
+        } catch (IllegalArgumentException e) {
+            assertTrue(e.getMessage().startsWith("Clusters in hosted environments must have a <nodes count='N'> tag"));
+        }
+    }
+
     /** Deploying an application with "nodes count" standalone should give a single-node deployment */
     @Test
     public void testThatHostedSyntaxWorksOnStandalone() {