diff options
author | Martin Polden <mpolden@mpolden.no> | 2020-05-26 10:44:25 +0200 |
---|---|---|
committer | Martin Polden <mpolden@mpolden.no> | 2020-05-27 15:08:24 +0200 |
commit | 9be54355fa8938c3537fbf9b0ede5c0e597b997c (patch) | |
tree | 02d84386e87687e723e18967d8109d4c7b64a3c6 /node-repository | |
parent | f8400289c573467829db1d9fffa370d727b0487b (diff) |
Add Cloud builder
Diffstat (limited to 'node-repository')
3 files changed, 31 insertions, 21 deletions
diff --git a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/autoscale/AutoscalingTest.java b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/autoscale/AutoscalingTest.java index 08e4237fe00..4bb3ffafcd8 100644 --- a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/autoscale/AutoscalingTest.java +++ b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/autoscale/AutoscalingTest.java @@ -298,9 +298,10 @@ public class AutoscalingTest { flavors.add(new Flavor("aws-large", new NodeResources(3, 150, 100, 1, NodeResources.DiskSpeed.fast, NodeResources.StorageType.remote))); flavors.add(new Flavor("aws-medium", new NodeResources(3, 100, 100, 1, NodeResources.DiskSpeed.fast, NodeResources.StorageType.remote))); flavors.add(new Flavor("aws-small", new NodeResources(3, 80, 100, 1, NodeResources.DiskSpeed.fast, NodeResources.StorageType.remote))); - AutoscalingTester tester = new AutoscalingTester(new Zone(Cloud.defaultCloud() - .withDynamicProvisioning(true) - .withAllowHostSharing(false), + AutoscalingTester tester = new AutoscalingTester(new Zone(Cloud.builder() + .dynamicProvisioning(true) + .allowHostSharing(false) + .build(), SystemName.main, Environment.prod, RegionName.from("us-east")), flavors); diff --git a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/DynamicProvisioningMaintainerTest.java b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/DynamicProvisioningMaintainerTest.java index 2ffb103708f..507d875c69f 100644 --- a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/DynamicProvisioningMaintainerTest.java +++ b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/DynamicProvisioningMaintainerTest.java @@ -150,24 +150,34 @@ public class DynamicProvisioningMaintainerTest { private static final NodeFlavors flavors = FlavorConfigBuilder.createDummies("default", "docker"); private final ManualClock clock = new ManualClock(); - private final Zone zone = new Zone(Cloud.defaultCloud().withDynamicProvisioning(true), SystemName.defaultSystem(), - Environment.defaultEnvironment(), RegionName.defaultName()); - private final NodeRepository nodeRepository = new NodeRepository(flavors, - new HostResourcesCalculatorMock(), - new MockCurator(), - clock, - zone, - new MockNameResolver().mockAnyLookup(), - DockerImage.fromString("docker-image"), true); - - private final InMemoryFlagSource flagSource = new InMemoryFlagSource() .withListFlag(Flags.PREPROVISION_CAPACITY.id(), List.of(), PreprovisionCapacity.class); - private final HostProvisionerMock hostProvisioner = new HostProvisionerMock(nodeRepository); - private final DynamicProvisioningMaintainer maintainer = new DynamicProvisioningMaintainer(nodeRepository, - Duration.ofDays(1), - hostProvisioner, - flagSource); + + private final Zone zone; + private final NodeRepository nodeRepository; + private final HostProvisionerMock hostProvisioner; + private final DynamicProvisioningMaintainer maintainer; + + public DynamicProvisioningTester() { + this(Cloud.builder().dynamicProvisioning(true).build()); + } + + public DynamicProvisioningTester(Cloud cloud) { + this.zone = new Zone(cloud, SystemName.defaultSystem(), Environment.defaultEnvironment(), + RegionName.defaultName()); + this.nodeRepository = new NodeRepository(flavors, + new HostResourcesCalculatorMock(), + new MockCurator(), + clock, + zone, + new MockNameResolver().mockAnyLookup(), + DockerImage.fromString("docker-image"), true); + this.hostProvisioner = new HostProvisionerMock(nodeRepository); + this.maintainer = new DynamicProvisioningMaintainer(nodeRepository, + Duration.ofDays(1), + hostProvisioner, + flagSource); + } private DynamicProvisioningTester addInitialNodes() { List.of(createNode("host1", Optional.empty(), NodeType.host, Node.State.active, Optional.of(tenantHostApp)), diff --git a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/DynamicDockerProvisionTest.java b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/DynamicDockerProvisionTest.java index 7ef780b8b7f..cf37de07304 100644 --- a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/DynamicDockerProvisionTest.java +++ b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/DynamicDockerProvisionTest.java @@ -33,7 +33,6 @@ import java.util.function.Function; import java.util.stream.Collectors; import java.util.stream.IntStream; -import static com.yahoo.config.provision.NodeResources.DiskSpeed.any; import static com.yahoo.config.provision.NodeResources.DiskSpeed.fast; import static com.yahoo.config.provision.NodeResources.StorageType.local; import static com.yahoo.config.provision.NodeResources.StorageType.remote; @@ -51,7 +50,7 @@ import static org.mockito.Mockito.verify; public class DynamicDockerProvisionTest { private static final Zone zone = new Zone( - Cloud.defaultCloud().withDynamicProvisioning(true).withAllowHostSharing(false), + Cloud.builder().dynamicProvisioning(true).allowHostSharing(false).build(), SystemName.main, Environment.prod, RegionName.from("us-east")); |