diff options
author | Martin Polden <mpolden@mpolden.no> | 2023-10-11 11:03:24 +0200 |
---|---|---|
committer | Martin Polden <mpolden@mpolden.no> | 2023-10-11 13:56:17 +0200 |
commit | 2d4a465fd170c8624d2d481b66f7105ddd5d16c5 (patch) | |
tree | ca1fef96e638c95334cb7e56ad89f0e6a37f24c3 /configserver | |
parent | abac62c851c27907c7a77c6ca5c7566fdbe64b75 (diff) |
Require zone-scoped endpoint in config-model
Diffstat (limited to 'configserver')
5 files changed, 46 insertions, 5 deletions
diff --git a/configserver/src/test/java/com/yahoo/vespa/config/server/deploy/DeployTester.java b/configserver/src/test/java/com/yahoo/vespa/config/server/deploy/DeployTester.java index 225bcb8dbed..67a1a335067 100644 --- a/configserver/src/test/java/com/yahoo/vespa/config/server/deploy/DeployTester.java +++ b/configserver/src/test/java/com/yahoo/vespa/config/server/deploy/DeployTester.java @@ -140,7 +140,19 @@ public class DeployTester { * Do the initial "deploy" with the existing API-less code as the deploy API doesn't support first deploys yet. */ public PrepareResult deployApp(String applicationPath, String vespaVersion) { - return deployApp(applicationPath, new PrepareParams.Builder().vespaVersion(vespaVersion)); + String endpoints = """ + [ + { + "clusterId": "container", + "names": [ + "c.example.com" + ], + "scope": "zone", + "routingMethod": "exclusive" + } + ] + """; + return deployApp(applicationPath, new PrepareParams.Builder().containerEndpoints(endpoints).vespaVersion(vespaVersion)); } /** diff --git a/configserver/src/test/java/com/yahoo/vespa/config/server/deploy/HostedDeployNodeAllocationTest.java b/configserver/src/test/java/com/yahoo/vespa/config/server/deploy/HostedDeployNodeAllocationTest.java index ec9bd3f7245..44344b6d394 100644 --- a/configserver/src/test/java/com/yahoo/vespa/config/server/deploy/HostedDeployNodeAllocationTest.java +++ b/configserver/src/test/java/com/yahoo/vespa/config/server/deploy/HostedDeployNodeAllocationTest.java @@ -63,10 +63,22 @@ public class HostedDeployNodeAllocationTest { .provisioner(new MockProvisioner().hostProvisioner(provisioner)) .hostedConfigserverConfig(Zone.defaultZone()) .build(); - + String endpoints = """ + [ + { + "clusterId": "container", + "names": [ + "c.example.com" + ], + "scope": "zone", + "routingMethod": "exclusive" + } + ] + """; try { tester.deployApp("src/test/apps/hosted/", new PrepareParams.Builder() .vespaVersion("7.3") + .containerEndpoints(endpoints) .quota(new Quota(Optional.of(4), Optional.of(0)))); fail("Expected to get a QuotaExceededException"); } catch (QuotaExceededException e) { diff --git a/configserver/src/test/java/com/yahoo/vespa/config/server/maintenance/TenantsMaintainerTest.java b/configserver/src/test/java/com/yahoo/vespa/config/server/maintenance/TenantsMaintainerTest.java index 0021e5f4b6a..5fd23a95eed 100644 --- a/configserver/src/test/java/com/yahoo/vespa/config/server/maintenance/TenantsMaintainerTest.java +++ b/configserver/src/test/java/com/yahoo/vespa/config/server/maintenance/TenantsMaintainerTest.java @@ -63,7 +63,19 @@ public class TenantsMaintainerTest { } private PrepareParams.Builder prepareParams(TenantName tenantName) { - return new PrepareParams.Builder().applicationId(applicationId(tenantName)); + String endpoints = """ + [ + { + "clusterId": "container", + "names": [ + "c.example.com" + ], + "scope": "zone", + "routingMethod": "exclusive" + } + ] + """; + return new PrepareParams.Builder().containerEndpoints(endpoints).applicationId(applicationId(tenantName)); } private ApplicationId applicationId(TenantName tenantName) { diff --git a/configserver/src/test/java/com/yahoo/vespa/config/server/model/LbServicesProducerTest.java b/configserver/src/test/java/com/yahoo/vespa/config/server/model/LbServicesProducerTest.java index 9954a40512f..e1ad6bf51f0 100644 --- a/configserver/src/test/java/com/yahoo/vespa/config/server/model/LbServicesProducerTest.java +++ b/configserver/src/test/java/com/yahoo/vespa/config/server/model/LbServicesProducerTest.java @@ -81,7 +81,7 @@ public class LbServicesProducerTest { private LbServicesConfig createModelAndGetLbServicesConfig(RegionName regionName) { Zone zone = new Zone(Environment.prod, regionName); - Map<TenantName, Set<ApplicationInfo>> testModel = createTestModel(new DeployState.Builder().zone(zone)); + Map<TenantName, Set<ApplicationInfo>> testModel = createTestModel(new DeployState.Builder().endpoints(Set.of(new ContainerEndpoint("mydisc", ApplicationClusterEndpoint.Scope.zone, List.of("md.example.com")))).zone(zone)); return getLbServicesConfig(new Zone(Environment.prod, regionName), testModel); } @@ -126,7 +126,7 @@ public class LbServicesProducerTest { @Test public void testRoutingConfigForTesterApplication() { - Map<TenantName, Set<ApplicationInfo>> testModel = createTestModel(new DeployState.Builder()); + Map<TenantName, Set<ApplicationInfo>> testModel = createTestModel(new DeployState.Builder().endpoints(Set.of(new ContainerEndpoint("mydisc", ApplicationClusterEndpoint.Scope.zone, List.of("md.example.com"))))); // No config for tester application assertNull(getLbServicesConfig(Zone.defaultZone(), testModel) diff --git a/configserver/src/test/java/com/yahoo/vespa/config/server/provision/StaticProvisionerTest.java b/configserver/src/test/java/com/yahoo/vespa/config/server/provision/StaticProvisionerTest.java index 29515a6c872..b64c0417043 100644 --- a/configserver/src/test/java/com/yahoo/vespa/config/server/provision/StaticProvisionerTest.java +++ b/configserver/src/test/java/com/yahoo/vespa/config/server/provision/StaticProvisionerTest.java @@ -4,6 +4,8 @@ package com.yahoo.vespa.config.server.provision; import com.yahoo.cloud.config.ModelConfig; import com.yahoo.config.application.api.ApplicationPackage; import com.yahoo.config.model.NullConfigModelRegistry; +import com.yahoo.config.model.api.ApplicationClusterEndpoint; +import com.yahoo.config.model.api.ContainerEndpoint; import com.yahoo.config.model.api.HostProvisioner; import com.yahoo.config.model.application.provider.FilesApplicationPackage; import com.yahoo.config.model.deploy.DeployState; @@ -16,6 +18,8 @@ import org.xml.sax.SAXException; import java.io.File; import java.io.IOException; +import java.util.List; +import java.util.Set; import static org.junit.Assert.assertEquals; @@ -50,6 +54,7 @@ public class StaticProvisionerTest { DeployState deployState = new DeployState.Builder() .applicationPackage(app) .modelHostProvisioner(provisioner) + .endpoints(Set.of(new ContainerEndpoint("container", ApplicationClusterEndpoint.Scope.zone, List.of("c.example.com")))) .properties(new TestProperties() .setMultitenant(true) .setHostedVespa(true)) |