aboutsummaryrefslogtreecommitdiffstats
path: root/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance
diff options
context:
space:
mode:
authorJon Marius Venstad <venstad@gmail.com>2019-09-18 13:20:21 +0200
committerJon Marius Venstad <venstad@gmail.com>2019-09-18 16:39:10 +0200
commita64c2bec5539a200769a973702ba6543ae621837 (patch)
tree0fd86ab8d14cca25018f2a5218becd47f413c61c /controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance
parentb6a52c8fda45d7d3f45a8606b419b5f4becfbaed (diff)
Rename Application -> Instance
Diffstat (limited to 'controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance')
-rw-r--r--controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/ApplicationOwnershipConfirmerTest.java6
-rw-r--r--controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/DeploymentExpirerTest.java14
-rw-r--r--controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/DeploymentIssueReporterTest.java8
-rw-r--r--controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/DeploymentMetricsMaintainerTest.java8
-rw-r--r--controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/MetricsReporterTest.java52
-rw-r--r--controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/OutstandingChangeDeployerTest.java4
-rw-r--r--controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/RotationStatusUpdaterTest.java4
-rw-r--r--controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/RoutingPoliciesTest.java10
-rw-r--r--controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/UpgraderTest.java240
9 files changed, 173 insertions, 173 deletions
diff --git a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/ApplicationOwnershipConfirmerTest.java b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/ApplicationOwnershipConfirmerTest.java
index f9c8b66acda..98e8eff508e 100644
--- a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/ApplicationOwnershipConfirmerTest.java
+++ b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/ApplicationOwnershipConfirmerTest.java
@@ -3,7 +3,7 @@ package com.yahoo.vespa.hosted.controller.maintenance;
import com.yahoo.config.provision.ApplicationId;
import com.yahoo.config.provision.TenantName;
-import com.yahoo.vespa.hosted.controller.Application;
+import com.yahoo.vespa.hosted.controller.Instance;
import com.yahoo.vespa.hosted.controller.api.integration.organization.ApplicationSummary;
import com.yahoo.vespa.hosted.controller.api.integration.organization.Contact;
import com.yahoo.vespa.hosted.controller.api.integration.organization.IssueId;
@@ -44,12 +44,12 @@ public class ApplicationOwnershipConfirmerTest {
Optional<Contact> contact = Optional.of(tester.controllerTester().serviceRegistry().contactRetrieverMock().contact());
TenantName property = tester.controllerTester().createTenant("property", "domain", 1L, contact);
tester.createAndDeploy(property, "application", 1, "default");
- Supplier<Application> propertyApp = () -> tester.controller().applications().require(ApplicationId.from("property", "application", "default"));
+ Supplier<Instance> propertyApp = () -> tester.controller().applications().require(ApplicationId.from("property", "application", "default"));
UserTenant user = UserTenant.create("by-user", contact);
tester.controller().tenants().createUser(user);
tester.createAndDeploy(user.name(), "application", 2, "default");
- Supplier<Application> userApp = () -> tester.controller().applications().require(ApplicationId.from("by-user", "application", "default"));
+ Supplier<Instance> userApp = () -> tester.controller().applications().require(ApplicationId.from("by-user", "application", "default"));
assertFalse("No issue is initially stored for a new application.", propertyApp.get().ownershipIssueId().isPresent());
assertFalse("No issue is initially stored for a new application.", userApp.get().ownershipIssueId().isPresent());
diff --git a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/DeploymentExpirerTest.java b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/DeploymentExpirerTest.java
index 730b2943431..b2ac44bf23b 100644
--- a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/DeploymentExpirerTest.java
+++ b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/DeploymentExpirerTest.java
@@ -4,7 +4,7 @@ package com.yahoo.vespa.hosted.controller.maintenance;
import com.yahoo.config.provision.Environment;
import com.yahoo.config.provision.RegionName;
import com.yahoo.config.provision.zone.ZoneId;
-import com.yahoo.vespa.hosted.controller.Application;
+import com.yahoo.vespa.hosted.controller.Instance;
import com.yahoo.vespa.hosted.controller.application.ApplicationPackage;
import com.yahoo.vespa.hosted.controller.application.Deployment;
import com.yahoo.vespa.hosted.controller.deployment.ApplicationPackageBuilder;
@@ -33,8 +33,8 @@ public class DeploymentExpirerTest {
);
DeploymentExpirer expirer = new DeploymentExpirer(tester.controller(), Duration.ofDays(10),
new JobControl(new MockCuratorDb()));
- Application devApp = tester.createApplication("app1", "tenant1", 123L, 1L);
- Application prodApp = tester.createApplication("app2", "tenant2", 456L, 2L);
+ Instance devApp = tester.createApplication("app1", "tenant1", 123L, 1L);
+ Instance prodApp = tester.createApplication("app2", "tenant2", 456L, 2L);
// Deploy dev
tester.controllerTester().deploy(devApp, tester.controllerTester().toZone(Environment.dev));
@@ -60,11 +60,11 @@ public class DeploymentExpirerTest {
assertEquals(1, permanentDeployments(prodApp).size());
}
- private List<Deployment> permanentDeployments(Application application) {
- return tester.controller().applications().get(application.id()).get().deployments().values().stream()
- .filter(deployment -> deployment.zone().environment() != Environment.test &&
+ private List<Deployment> permanentDeployments(Instance instance) {
+ return tester.controller().applications().get(instance.id()).get().deployments().values().stream()
+ .filter(deployment -> deployment.zone().environment() != Environment.test &&
deployment.zone().environment() != Environment.staging)
- .collect(Collectors.toList());
+ .collect(Collectors.toList());
}
}
diff --git a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/DeploymentIssueReporterTest.java b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/DeploymentIssueReporterTest.java
index 7da1064cb23..d20ccdf5963 100644
--- a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/DeploymentIssueReporterTest.java
+++ b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/DeploymentIssueReporterTest.java
@@ -4,7 +4,7 @@ package com.yahoo.vespa.hosted.controller.maintenance;
import com.yahoo.component.Version;
import com.yahoo.config.provision.ApplicationId;
import com.yahoo.config.provision.Environment;
-import com.yahoo.vespa.hosted.controller.Application;
+import com.yahoo.vespa.hosted.controller.Instance;
import com.yahoo.vespa.hosted.controller.api.integration.organization.Contact;
import com.yahoo.vespa.hosted.controller.api.integration.organization.IssueId;
import com.yahoo.vespa.hosted.controller.api.integration.stubs.LoggingDeploymentIssues;
@@ -78,9 +78,9 @@ public class DeploymentIssueReporterTest {
tester.controllerTester().createTenant("tenant3", "domain3", 1L, contact);
// Create and deploy one application for each of three tenants.
- Application app1 = tester.createApplication("application1", "tenant1", projectId1, propertyId1);
- Application app2 = tester.createApplication("application2", "tenant2", projectId2, propertyId2);
- Application app3 = tester.createApplication("application3", "tenant3", projectId3, propertyId3);
+ Instance app1 = tester.createApplication("application1", "tenant1", projectId1, propertyId1);
+ Instance app2 = tester.createApplication("application2", "tenant2", projectId2, propertyId2);
+ Instance app3 = tester.createApplication("application3", "tenant3", projectId3, propertyId3);
// NOTE: All maintenance should be idempotent within a small enough time interval, so maintain is called twice in succession throughout.
diff --git a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/DeploymentMetricsMaintainerTest.java b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/DeploymentMetricsMaintainerTest.java
index c8b3cd01fe8..c4a835cd7e7 100644
--- a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/DeploymentMetricsMaintainerTest.java
+++ b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/DeploymentMetricsMaintainerTest.java
@@ -6,7 +6,7 @@ import com.yahoo.config.provision.ApplicationId;
import com.yahoo.config.provision.Environment;
import com.yahoo.config.provision.RegionName;
import com.yahoo.config.provision.zone.ZoneId;
-import com.yahoo.vespa.hosted.controller.Application;
+import com.yahoo.vespa.hosted.controller.Instance;
import com.yahoo.vespa.hosted.controller.Controller;
import com.yahoo.vespa.hosted.controller.api.application.v4.model.ClusterMetrics;
import com.yahoo.vespa.hosted.controller.api.identifiers.DeploymentId;
@@ -40,7 +40,7 @@ public class DeploymentMetricsMaintainerTest {
deploy(application, Version.fromString("7.1"));
DeploymentMetricsMaintainer maintainer = maintainer(tester.controller());
- Supplier<Application> app = () -> tester.application(application.id());
+ Supplier<Instance> app = () -> tester.application(application.id());
Supplier<Deployment> deployment = () -> app.get().deployments().values().stream().findFirst().get();
// No metrics gathered yet
@@ -127,8 +127,8 @@ public class DeploymentMetricsMaintainerTest {
return new DeploymentMetricsMaintainer(controller, Duration.ofDays(1), new JobControl(controller.curator()));
}
- private void deploy(Application application, Version version) {
- tester.controllerTester().deploy(application,
+ private void deploy(Instance instance, Version version) {
+ tester.controllerTester().deploy(instance,
ZoneId.from(Environment.dev, RegionName.from("us-east-1")),
Optional.of(new ApplicationPackage(new byte[0])),
false,
diff --git a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/MetricsReporterTest.java b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/MetricsReporterTest.java
index 148be3f258e..46e6d9c3f50 100644
--- a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/MetricsReporterTest.java
+++ b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/MetricsReporterTest.java
@@ -4,7 +4,7 @@ package com.yahoo.vespa.hosted.controller.maintenance;
import com.yahoo.component.Version;
import com.yahoo.config.provision.Environment;
import com.yahoo.config.provision.zone.ZoneId;
-import com.yahoo.vespa.hosted.controller.Application;
+import com.yahoo.vespa.hosted.controller.Instance;
import com.yahoo.vespa.hosted.controller.Controller;
import com.yahoo.vespa.hosted.controller.ControllerTester;
import com.yahoo.vespa.hosted.controller.api.identifiers.DeploymentId;
@@ -46,10 +46,10 @@ public class MetricsReporterTest {
assertEquals(0.0, metrics.getMetric(MetricsReporter.DEPLOYMENT_FAIL_METRIC));
// Deploy all apps successfully
- Application app1 = tester.createApplication("app1", "tenant1", 1, 11L);
- Application app2 = tester.createApplication("app2", "tenant1", 2, 22L);
- Application app3 = tester.createApplication("app3", "tenant1", 3, 33L);
- Application app4 = tester.createApplication("app4", "tenant1", 4, 44L);
+ Instance app1 = tester.createApplication("app1", "tenant1", 1, 11L);
+ Instance app2 = tester.createApplication("app2", "tenant1", 2, 22L);
+ Instance app3 = tester.createApplication("app3", "tenant1", 3, 33L);
+ Instance app4 = tester.createApplication("app4", "tenant1", 4, 44L);
tester.deployCompletely(app1, applicationPackage);
tester.deployCompletely(app2, applicationPackage);
tester.deployCompletely(app3, applicationPackage);
@@ -76,7 +76,7 @@ public class MetricsReporterTest {
MetricsReporter reporter = createReporter(tester.controller());
- Application app = tester.createApplication("app1", "tenant1", 1, 11L);
+ Instance app = tester.createApplication("app1", "tenant1", 1, 11L);
tester.deployCompletely(app, applicationPackage);
reporter.maintain();
assertEquals(Duration.ZERO, getAverageDeploymentDuration(app)); // An exceptionally fast deployment :-)
@@ -117,7 +117,7 @@ public class MetricsReporterTest {
.build();
MetricsReporter reporter = createReporter(tester.controller());
- Application app = tester.createApplication("app1", "tenant1", 1, 11L);
+ Instance app = tester.createApplication("app1", "tenant1", 1, 11L);
// Initial deployment without failures
tester.deployCompletely(app, applicationPackage);
@@ -168,12 +168,12 @@ public class MetricsReporterTest {
.region("us-east-3")
.build();
MetricsReporter reporter = createReporter(tester.controller());
- Application application = tester.createApplication("app1", "tenant1", 1, 11L);
- tester.configServer().generateWarnings(new DeploymentId(application.id(), ZoneId.from("prod", "us-west-1")), 3);
- tester.configServer().generateWarnings(new DeploymentId(application.id(), ZoneId.from("prod", "us-east-3")), 4);
- tester.deployCompletely(application, applicationPackage);
+ Instance instance = tester.createApplication("app1", "tenant1", 1, 11L);
+ tester.configServer().generateWarnings(new DeploymentId(instance.id(), ZoneId.from("prod", "us-west-1")), 3);
+ tester.configServer().generateWarnings(new DeploymentId(instance.id(), ZoneId.from("prod", "us-east-3")), 4);
+ tester.deployCompletely(instance, applicationPackage);
reporter.maintain();
- assertEquals(4, getDeploymentWarnings(application));
+ assertEquals(4, getDeploymentWarnings(instance));
}
@Test
@@ -198,11 +198,11 @@ public class MetricsReporterTest {
.region("us-east-3")
.build();
MetricsReporter reporter = createReporter(tester.controller());
- Application application = tester.createApplication("app1", "tenant1", 1, 11L);
+ Instance instance = tester.createApplication("app1", "tenant1", 1, 11L);
reporter.maintain();
assertEquals("Queue is empty initially", 0, metrics.getMetric(MetricsReporter.NAME_SERVICE_REQUESTS_QUEUED).intValue());
- tester.deployCompletely(application, applicationPackage);
+ tester.deployCompletely(instance, applicationPackage);
reporter.maintain();
assertEquals("Deployment queues name services requests", 6, metrics.getMetric(MetricsReporter.NAME_SERVICE_REQUESTS_QUEUED).intValue());
@@ -211,31 +211,31 @@ public class MetricsReporterTest {
assertEquals("Queue consumed", 0, metrics.getMetric(MetricsReporter.NAME_SERVICE_REQUESTS_QUEUED).intValue());
}
- private Duration getAverageDeploymentDuration(Application application) {
- return Duration.ofSeconds(getMetric(MetricsReporter.DEPLOYMENT_AVERAGE_DURATION, application).longValue());
+ private Duration getAverageDeploymentDuration(Instance instance) {
+ return Duration.ofSeconds(getMetric(MetricsReporter.DEPLOYMENT_AVERAGE_DURATION, instance).longValue());
}
- private int getDeploymentsFailingUpgrade(Application application) {
- return getMetric(MetricsReporter.DEPLOYMENT_FAILING_UPGRADES, application).intValue();
+ private int getDeploymentsFailingUpgrade(Instance instance) {
+ return getMetric(MetricsReporter.DEPLOYMENT_FAILING_UPGRADES, instance).intValue();
}
- private int getDeploymentWarnings(Application application) {
- return getMetric(MetricsReporter.DEPLOYMENT_WARNINGS, application).intValue();
+ private int getDeploymentWarnings(Instance instance) {
+ return getMetric(MetricsReporter.DEPLOYMENT_WARNINGS, instance).intValue();
}
- private Number getMetric(String name, Application application) {
- return metrics.getMetric((dimensions) -> application.id().tenant().value().equals(dimensions.get("tenant")) &&
- appDimension(application).equals(dimensions.get("app")),
+ private Number getMetric(String name, Instance instance) {
+ return metrics.getMetric((dimensions) -> instance.id().tenant().value().equals(dimensions.get("tenant")) &&
+ appDimension(instance).equals(dimensions.get("app")),
name)
- .orElseThrow(() -> new RuntimeException("Expected metric to exist for " + application.id()));
+ .orElseThrow(() -> new RuntimeException("Expected metric to exist for " + instance.id()));
}
private MetricsReporter createReporter(Controller controller) {
return new MetricsReporter(controller, metrics, new JobControl(new MockCuratorDb()));
}
- private static String appDimension(Application application) {
- return application.id().application().value() + "." + application.id().instance().value();
+ private static String appDimension(Instance instance) {
+ return instance.id().application().value() + "." + instance.id().instance().value();
}
}
diff --git a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/OutstandingChangeDeployerTest.java b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/OutstandingChangeDeployerTest.java
index 3a7c62c900a..4bd1bfcffae 100644
--- a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/OutstandingChangeDeployerTest.java
+++ b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/OutstandingChangeDeployerTest.java
@@ -3,7 +3,7 @@ package com.yahoo.vespa.hosted.controller.maintenance;
import com.yahoo.component.Version;
import com.yahoo.config.provision.Environment;
-import com.yahoo.vespa.hosted.controller.Application;
+import com.yahoo.vespa.hosted.controller.Instance;
import com.yahoo.vespa.hosted.controller.api.integration.BuildService;
import com.yahoo.vespa.hosted.controller.api.integration.deployment.JobType;
import com.yahoo.vespa.hosted.controller.application.ApplicationPackage;
@@ -53,7 +53,7 @@ public class OutstandingChangeDeployerTest {
.uploadArtifact(applicationPackage)
.submit();
- Application app = tester.application("app1");
+ Instance app = tester.application("app1");
assertTrue(app.outstandingChange().hasTargets());
assertEquals("1.0.43-cafed00d", app.outstandingChange().application().get().id());
assertEquals(2, tester.buildService().jobs().size());
diff --git a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/RotationStatusUpdaterTest.java b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/RotationStatusUpdaterTest.java
index f0b12cb48b6..7f97f1bc4a9 100644
--- a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/RotationStatusUpdaterTest.java
+++ b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/RotationStatusUpdaterTest.java
@@ -3,7 +3,7 @@ package com.yahoo.vespa.hosted.controller.maintenance;
import com.yahoo.config.provision.Environment;
import com.yahoo.config.provision.zone.ZoneId;
-import com.yahoo.vespa.hosted.controller.Application;
+import com.yahoo.vespa.hosted.controller.Instance;
import com.yahoo.vespa.hosted.controller.api.integration.routing.RotationStatus;
import com.yahoo.vespa.hosted.controller.application.Deployment;
import com.yahoo.vespa.hosted.controller.deployment.ApplicationPackageBuilder;
@@ -42,7 +42,7 @@ public class RotationStatusUpdaterTest {
.build();
tester.deployCompletely(application, applicationPackage);
- Supplier<Application> app = () -> tester.application(application.id());
+ Supplier<Instance> app = () -> tester.application(application.id());
Supplier<Deployment> deployment1 = () -> app.get().deployments().get(zone1);
Supplier<Deployment> deployment2 = () -> app.get().deployments().get(zone2);
Supplier<Deployment> deployment3 = () -> app.get().deployments().get(zone3);
diff --git a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/RoutingPoliciesTest.java b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/RoutingPoliciesTest.java
index 0dee5fcb78c..1763ff74fb0 100644
--- a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/RoutingPoliciesTest.java
+++ b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/RoutingPoliciesTest.java
@@ -5,7 +5,7 @@ import com.yahoo.config.provision.ApplicationId;
import com.yahoo.config.provision.ClusterSpec;
import com.yahoo.config.provision.HostName;
import com.yahoo.config.provision.zone.ZoneId;
-import com.yahoo.vespa.hosted.controller.Application;
+import com.yahoo.vespa.hosted.controller.Instance;
import com.yahoo.vespa.hosted.controller.api.identifiers.DeploymentId;
import com.yahoo.vespa.hosted.controller.api.integration.configserver.LoadBalancer;
import com.yahoo.vespa.hosted.controller.api.integration.dns.Record;
@@ -38,8 +38,8 @@ public class RoutingPoliciesTest {
private final DeploymentTester tester = new DeploymentTester();
- private final Application app1 = tester.createApplication("app1", "tenant1", 1, 1L);
- private final Application app2 = tester.createApplication("app2", "tenant1", 1, 1L);
+ private final Instance app1 = tester.createApplication("app1", "tenant1", 1, 1L);
+ private final Instance app2 = tester.createApplication("app2", "tenant1", 1, 1L);
private final ZoneId zone1 = ZoneId.from("prod", "us-west-1");
private final ZoneId zone2 = ZoneId.from("prod", "us-central-1");
@@ -238,8 +238,8 @@ public class RoutingPoliciesTest {
tester.controller().applications().clusterEndpoints(new DeploymentId(app1.id(), zone1)));
}
- private Set<RoutingPolicy> policies(Application application) {
- return tester.controller().curator().readRoutingPolicies(application.id());
+ private Set<RoutingPolicy> policies(Instance instance) {
+ return tester.controller().curator().readRoutingPolicies(instance.id());
}
private Set<String> recordNames() {
diff --git a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/UpgraderTest.java b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/UpgraderTest.java
index ab4d4987d6a..a54d6f3ece7 100644
--- a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/UpgraderTest.java
+++ b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/UpgraderTest.java
@@ -5,7 +5,7 @@ import com.yahoo.component.Version;
import com.yahoo.config.provision.Environment;
import com.yahoo.config.provision.RegionName;
import com.yahoo.config.provision.zone.ZoneId;
-import com.yahoo.vespa.hosted.controller.Application;
+import com.yahoo.vespa.hosted.controller.Instance;
import com.yahoo.vespa.hosted.controller.ControllerTester;
import com.yahoo.vespa.hosted.controller.api.integration.deployment.JobType;
import com.yahoo.vespa.hosted.controller.application.ApplicationPackage;
@@ -54,12 +54,12 @@ public class UpgraderTest {
assertEquals("No applications: Nothing to do", 0, tester.buildService().jobs().size());
// Setup applications
- Application canary0 = tester.createAndDeploy("canary0", 1, "canary");
- Application canary1 = tester.createAndDeploy("canary1", 2, "canary");
- Application default0 = tester.createAndDeploy("default0", 3, "default");
- Application default1 = tester.createAndDeploy("default1", 4, "default");
- Application default2 = tester.createAndDeploy("default2", 5, "default");
- Application conservative0 = tester.createAndDeploy("conservative0", 6, "conservative");
+ Instance canary0 = tester.createAndDeploy("canary0", 1, "canary");
+ Instance canary1 = tester.createAndDeploy("canary1", 2, "canary");
+ Instance default0 = tester.createAndDeploy("default0", 3, "default");
+ Instance default1 = tester.createAndDeploy("default1", 4, "default");
+ Instance default2 = tester.createAndDeploy("default2", 5, "default");
+ Instance conservative0 = tester.createAndDeploy("conservative0", 6, "conservative");
tester.upgrader().maintain();
tester.triggerUntilQuiescence();
@@ -183,8 +183,8 @@ public class UpgraderTest {
// --- Starting upgrading to a new version which breaks, causing upgrades to commence on the previous version
Version version4 = Version.fromString("6.6");
- Application default3 = tester.createAndDeploy("default3", 7, "default"); // need 4 to break a version
- Application default4 = tester.createAndDeploy("default4", 8, "default");
+ Instance default3 = tester.createAndDeploy("default3", 7, "default"); // need 4 to break a version
+ Instance default4 = tester.createAndDeploy("default4", 8, "default");
tester.upgradeSystem(version4);
tester.upgrader().maintain(); // cause canary upgrades to new version
tester.triggerUntilQuiescence();
@@ -271,18 +271,18 @@ public class UpgraderTest {
assertEquals("No applications: Nothing to do", 0, tester.buildService().jobs().size());
// Setup applications
- Application canary0 = tester.createAndDeploy("canary0", 1, "canary");
- Application canary1 = tester.createAndDeploy("canary1", 2, "canary");
- Application default0 = tester.createAndDeploy("default0", 3, "default");
- Application default1 = tester.createAndDeploy("default1", 4, "default");
- Application default2 = tester.createAndDeploy("default2", 5, "default");
- Application default3 = tester.createAndDeploy("default3", 6, "default");
- Application default4 = tester.createAndDeploy("default4", 7, "default");
- Application default5 = tester.createAndDeploy("default5", 8, "default");
- Application default6 = tester.createAndDeploy("default6", 9, "default");
- Application default7 = tester.createAndDeploy("default7", 10, "default");
- Application default8 = tester.createAndDeploy("default8", 11, "default");
- Application default9 = tester.createAndDeploy("default9", 12, "default");
+ Instance canary0 = tester.createAndDeploy("canary0", 1, "canary");
+ Instance canary1 = tester.createAndDeploy("canary1", 2, "canary");
+ Instance default0 = tester.createAndDeploy("default0", 3, "default");
+ Instance default1 = tester.createAndDeploy("default1", 4, "default");
+ Instance default2 = tester.createAndDeploy("default2", 5, "default");
+ Instance default3 = tester.createAndDeploy("default3", 6, "default");
+ Instance default4 = tester.createAndDeploy("default4", 7, "default");
+ Instance default5 = tester.createAndDeploy("default5", 8, "default");
+ Instance default6 = tester.createAndDeploy("default6", 9, "default");
+ Instance default7 = tester.createAndDeploy("default7", 10, "default");
+ Instance default8 = tester.createAndDeploy("default8", 11, "default");
+ Instance default9 = tester.createAndDeploy("default9", 12, "default");
tester.upgrader().maintain();
tester.triggerUntilQuiescence();
@@ -338,7 +338,7 @@ public class UpgraderTest {
Version version = Version.fromString("6.2");
tester.upgradeSystem(version);
- Application app = tester.createApplication("app1", "tenant1", 1, 11L);
+ Instance app = tester.createApplication("app1", "tenant1", 1, 11L);
tester.jobCompletion(component).application(app).uploadArtifact(applicationPackage).submit();
tester.deployAndNotify(app, applicationPackage, true, systemTest);
tester.deployAndNotify(app, applicationPackage, true, stagingTest);
@@ -389,13 +389,13 @@ public class UpgraderTest {
tester.upgradeSystem(version);
// Setup applications
- Application canary0 = tester.createAndDeploy("canary0", 1, "canary");
- Application canary1 = tester.createAndDeploy("canary1", 2, "canary");
- Application default0 = tester.createAndDeploy("default0", 3, "default");
- Application default1 = tester.createAndDeploy("default1", 4, "default");
- Application default2 = tester.createAndDeploy("default2", 5, "default");
- Application default3 = tester.createAndDeploy("default3", 6, "default");
- Application default4 = tester.createAndDeploy("default4", 7, "default");
+ Instance canary0 = tester.createAndDeploy("canary0", 1, "canary");
+ Instance canary1 = tester.createAndDeploy("canary1", 2, "canary");
+ Instance default0 = tester.createAndDeploy("default0", 3, "default");
+ Instance default1 = tester.createAndDeploy("default1", 4, "default");
+ Instance default2 = tester.createAndDeploy("default2", 5, "default");
+ Instance default3 = tester.createAndDeploy("default3", 6, "default");
+ Instance default4 = tester.createAndDeploy("default4", 7, "default");
// New version is released
version = Version.fromString("6.3");
@@ -455,13 +455,13 @@ public class UpgraderTest {
tester.upgradeSystem(v0);
// Setup applications on V0
- Application canary0 = tester.createAndDeploy("canary0", 1, "canary");
- Application canary1 = tester.createAndDeploy("canary1", 2, "canary");
- Application default0 = tester.createAndDeploy("default0", 3, "default");
- Application default1 = tester.createAndDeploy("default1", 4, "default");
- Application default2 = tester.createAndDeploy("default2", 5, "default");
- Application default3 = tester.createAndDeploy("default3", 6, "default");
- Application default4 = tester.createAndDeploy("default4", 7, "default");
+ Instance canary0 = tester.createAndDeploy("canary0", 1, "canary");
+ Instance canary1 = tester.createAndDeploy("canary1", 2, "canary");
+ Instance default0 = tester.createAndDeploy("default0", 3, "default");
+ Instance default1 = tester.createAndDeploy("default1", 4, "default");
+ Instance default2 = tester.createAndDeploy("default2", 5, "default");
+ Instance default3 = tester.createAndDeploy("default3", 6, "default");
+ Instance default4 = tester.createAndDeploy("default4", 7, "default");
// V1 is released
Version v1 = Version.fromString("6.3");
@@ -542,13 +542,13 @@ public class UpgraderTest {
ApplicationPackage defaultPolicy = DeploymentTester.applicationPackage("default");
// Setup applications
- Application canary0 = tester.createAndDeploy("canary0", 1, canaryPolicy);
- Application canary1 = tester.createAndDeploy("canary1", 2, canaryPolicy);
- Application default0 = tester.createAndDeploy("default0", 3, defaultPolicy);
- Application default1 = tester.createAndDeploy("default1", 4, defaultPolicy);
- Application default2 = tester.createAndDeploy("default2", 5, defaultPolicy);
- Application default3 = tester.createAndDeploy("default3", 6, defaultPolicy);
- Application default4 = tester.createAndDeploy("default4", 7, defaultPolicy);
+ Instance canary0 = tester.createAndDeploy("canary0", 1, canaryPolicy);
+ Instance canary1 = tester.createAndDeploy("canary1", 2, canaryPolicy);
+ Instance default0 = tester.createAndDeploy("default0", 3, defaultPolicy);
+ Instance default1 = tester.createAndDeploy("default1", 4, defaultPolicy);
+ Instance default2 = tester.createAndDeploy("default2", 5, defaultPolicy);
+ Instance default3 = tester.createAndDeploy("default3", 6, defaultPolicy);
+ Instance default4 = tester.createAndDeploy("default4", 7, defaultPolicy);
// New version is released
version = Version.fromString("6.3");
@@ -600,7 +600,7 @@ public class UpgraderTest {
.region("us-west-1")
.build();
- Application app = tester.createAndDeploy("app1", 1, applicationPackage);
+ Instance app = tester.createAndDeploy("app1", 1, applicationPackage);
// New version is released
version = Version.fromString("6.3");
@@ -643,7 +643,7 @@ public class UpgraderTest {
.region("us-east-3")
.build();
- Application app = tester.createAndDeploy("app1", 1, applicationPackage);
+ Instance app = tester.createAndDeploy("app1", 1, applicationPackage);
// New version is released
version = Version.fromString("6.3");
@@ -680,7 +680,7 @@ public class UpgraderTest {
.region("us-east-3")
.build();
- Application app = tester.createAndDeploy("app1", 1, applicationPackage);
+ Instance app = tester.createAndDeploy("app1", 1, applicationPackage);
// New version is released
version = Version.fromString("6.3");
@@ -758,13 +758,13 @@ public class UpgraderTest {
.build();
// Setup applications
- Application canary0 = tester.createAndDeploy("canary0", 1, canaryApplicationPackage);
- Application canary1 = tester.createAndDeploy("canary1", 2, canaryApplicationPackage);
- Application default0 = tester.createAndDeploy("default0", 3, defaultApplicationPackage);
- Application default1 = tester.createAndDeploy("default1", 4, defaultApplicationPackage);
- Application default2 = tester.createAndDeploy("default2", 5, defaultApplicationPackage);
- Application default3 = tester.createAndDeploy("default3", 6, defaultApplicationPackage);
- Application default4 = tester.createAndDeploy("default4", 7, defaultApplicationPackage);
+ Instance canary0 = tester.createAndDeploy("canary0", 1, canaryApplicationPackage);
+ Instance canary1 = tester.createAndDeploy("canary1", 2, canaryApplicationPackage);
+ Instance default0 = tester.createAndDeploy("default0", 3, defaultApplicationPackage);
+ Instance default1 = tester.createAndDeploy("default1", 4, defaultApplicationPackage);
+ Instance default2 = tester.createAndDeploy("default2", 5, defaultApplicationPackage);
+ Instance default3 = tester.createAndDeploy("default3", 6, defaultApplicationPackage);
+ Instance default4 = tester.createAndDeploy("default4", 7, defaultApplicationPackage);
assertEquals(version, default0.oldestDeployedPlatform().get());
@@ -810,7 +810,7 @@ public class UpgraderTest {
tester.jobCompletion(stagingTest).application(default4).unsuccessful().submit();
// 5th app never reports back and has a dead job, but no ongoing change
- Application deadLocked = tester.applications().require(default4.id());
+ Instance deadLocked = tester.applications().require(default4.id());
tester.assertRunning(systemTest, deadLocked.id());
assertFalse("No change present", deadLocked.change().hasTargets());
@@ -857,15 +857,15 @@ public class UpgraderTest {
upgrader.setUpgradesPerMinute(0.2);
// Setup applications
- Application canary0 = tester.createAndDeploy("canary0", 1, "canary");
- Application canary1 = tester.createAndDeploy("canary1", 2, "canary");
- Application default0 = tester.createAndDeploy("default0", 3, "default");
- Application default1 = tester.createAndDeploy("default1", 4, "default");
- Application default2 = tester.createAndDeploy("default2", 5, "default");
- Application default3 = tester.createAndDeploy("default3", 6, "default");
+ Instance canary0 = tester.createAndDeploy("canary0", 1, "canary");
+ Instance canary1 = tester.createAndDeploy("canary1", 2, "canary");
+ Instance default0 = tester.createAndDeploy("default0", 3, "default");
+ Instance default1 = tester.createAndDeploy("default1", 4, "default");
+ Instance default2 = tester.createAndDeploy("default2", 5, "default");
+ Instance default3 = tester.createAndDeploy("default3", 6, "default");
// Dev deployment which should be ignored
- Application dev0 = tester.createApplication("dev0", "tenant1", 7, 1L);
+ Instance dev0 = tester.createApplication("dev0", "tenant1", 7, 1L);
tester.controllerTester().deploy(dev0, ZoneId.from(Environment.dev, RegionName.from("dev-region")));
// New version is released and canaries upgrade
@@ -910,8 +910,8 @@ public class UpgraderTest {
.build();
// Setup applications
- Application canary0 = tester.createAndDeploy("canary0", 1, "canary");
- Application default0 = tester.createAndDeploy("default0", 2, version6ApplicationPackage);
+ Instance canary0 = tester.createAndDeploy("canary0", 1, "canary");
+ Instance default0 = tester.createAndDeploy("default0", 2, version6ApplicationPackage);
// New major version is released
version = Version.fromString("7.0");
@@ -944,8 +944,8 @@ public class UpgraderTest {
.build();
// Setup applications
- Application canary0 = tester.createAndDeploy("canary0", 1, "canary");
- Application default0 = tester.createAndDeploy("default0", 2, default0ApplicationPackage);
+ Instance canary0 = tester.createAndDeploy("canary0", 1, "canary");
+ Instance default0 = tester.createAndDeploy("default0", 2, default0ApplicationPackage);
tester.applications().lockOrThrow(default0.id(), a -> tester.applications().store(a.withMajorVersion(6)));
assertEquals(OptionalInt.of(6), tester.applications().get(default0.id()).get().majorVersion());
@@ -987,9 +987,9 @@ public class UpgraderTest {
.build();
// Setup applications
- Application canary0 = tester.createAndDeploy("canary", 1, version7CanaryApplicationPackage);
- Application default0 = tester.createAndDeploy("default0", 2, version7DefaultApplicationPackage);
- Application default1 = tester.createAndDeploy("default1", 3, "default");
+ Instance canary0 = tester.createAndDeploy("canary", 1, version7CanaryApplicationPackage);
+ Instance default0 = tester.createAndDeploy("default0", 2, version7DefaultApplicationPackage);
+ Instance default1 = tester.createAndDeploy("default1", 3, "default");
// New major version is released, but we don't want to upgrade to it yet
tester.upgrader().setTargetMajorVersion(Optional.of(6));
@@ -1034,7 +1034,7 @@ public class UpgraderTest {
.region("us-west-1")
.build();
- Application app = tester.createAndDeploy("app1", 1, applicationPackage);
+ Instance app = tester.createAndDeploy("app1", 1, applicationPackage);
// New version is released
version = Version.fromString("6.3");
@@ -1087,7 +1087,7 @@ public class UpgraderTest {
.region("us-east-3")
.build();
- Application app = tester.createAndDeploy("app1", 1, applicationPackage);
+ Instance app = tester.createAndDeploy("app1", 1, applicationPackage);
tester.jobCompletion(component).application(app).nextBuildNumber().uploadArtifact(applicationPackage).submit();
@@ -1139,7 +1139,7 @@ public class UpgraderTest {
.region("us-east-3")
.build();
- Application app = tester.createAndDeploy("app1", 1, applicationPackage);
+ Instance app = tester.createAndDeploy("app1", 1, applicationPackage);
tester.jobCompletion(component).application(app).nextBuildNumber().uploadArtifact(applicationPackage).submit();
@@ -1183,61 +1183,61 @@ public class UpgraderTest {
tester.upgradeSystem(version0);
// Create an application with pinned platform version.
- Application application = tester.createApplication("application", "tenant", 2, 3);
+ Instance instance = tester.createApplication("application", "tenant", 2, 3);
ApplicationPackage applicationPackage = new ApplicationPackageBuilder().environment(Environment.prod)
.region("us-east-3")
.region("us-west-1")
.build();
- tester.deploymentTrigger().forceChange(application.id(), Change.empty().withPin());
+ tester.deploymentTrigger().forceChange(instance.id(), Change.empty().withPin());
- tester.deployCompletely(application, applicationPackage);
- assertFalse(tester.application(application.id()).change().hasTargets());
- assertTrue(tester.application(application.id()).change().isPinned());
- assertEquals(2, tester.application(application.id()).deployments().size());
+ tester.deployCompletely(instance, applicationPackage);
+ assertFalse(tester.application(instance.id()).change().hasTargets());
+ assertTrue(tester.application(instance.id()).change().isPinned());
+ assertEquals(2, tester.application(instance.id()).deployments().size());
// Application does not upgrade.
Version version1 = Version.fromString("6.3");
tester.upgradeSystem(version1);
tester.upgrader().maintain();
- assertFalse(tester.application(application.id()).change().hasTargets());
- assertTrue(tester.application(application.id()).change().isPinned());
+ assertFalse(tester.application(instance.id()).change().hasTargets());
+ assertTrue(tester.application(instance.id()).change().isPinned());
// New application package is deployed.
- tester.deployCompletely(application, applicationPackage, BuildJob.defaultBuildNumber + 1);
- assertFalse(tester.application(application.id()).change().hasTargets());
- assertTrue(tester.application(application.id()).change().isPinned());
+ tester.deployCompletely(instance, applicationPackage, BuildJob.defaultBuildNumber + 1);
+ assertFalse(tester.application(instance.id()).change().hasTargets());
+ assertTrue(tester.application(instance.id()).change().isPinned());
// Application upgrades to new version when pin is removed.
- tester.deploymentTrigger().cancelChange(application.id(), PIN);
+ tester.deploymentTrigger().cancelChange(instance.id(), PIN);
tester.upgrader().maintain();
- assertTrue(tester.application(application.id()).change().hasTargets());
- assertFalse(tester.application(application.id()).change().isPinned());
+ assertTrue(tester.application(instance.id()).change().hasTargets());
+ assertFalse(tester.application(instance.id()).change().isPinned());
// Application is pinned to new version, and upgrade is therefore not cancelled, even though confidence is broken.
- tester.deploymentTrigger().forceChange(application.id(), Change.empty().withPin());
+ tester.deploymentTrigger().forceChange(instance.id(), Change.empty().withPin());
tester.upgrader().maintain();
tester.readyJobTrigger().maintain();
- assertEquals(version1, tester.application(application.id()).change().platform().get());
+ assertEquals(version1, tester.application(instance.id()).change().platform().get());
// Application fails upgrade after one zone is complete, and is pinned again to the old version.
- tester.deployAndNotify(application, true, systemTest);
- tester.deployAndNotify(application, true, stagingTest);
- tester.deployAndNotify(application, true, productionUsEast3);
- tester.deploy(productionUsWest1, application, Optional.empty(), false);
- tester.deployAndNotify(application, false, productionUsWest1);
- tester.deploymentTrigger().cancelChange(application.id(), ALL);
- tester.deploymentTrigger().forceChange(application.id(), Change.of(version0).withPin());
+ tester.deployAndNotify(instance, true, systemTest);
+ tester.deployAndNotify(instance, true, stagingTest);
+ tester.deployAndNotify(instance, true, productionUsEast3);
+ tester.deploy(productionUsWest1, instance, Optional.empty(), false);
+ tester.deployAndNotify(instance, false, productionUsWest1);
+ tester.deploymentTrigger().cancelChange(instance.id(), ALL);
+ tester.deploymentTrigger().forceChange(instance.id(), Change.of(version0).withPin());
tester.buildService().clear();
- assertEquals(version0, tester.application(application.id()).change().platform().get());
+ assertEquals(version0, tester.application(instance.id()).change().platform().get());
// Application downgrades to pinned version.
tester.readyJobTrigger().maintain();
- tester.deployAndNotify(application, true, systemTest);
- tester.deployAndNotify(application, true, stagingTest);
- tester.deployAndNotify(application, true, productionUsEast3);
- assertTrue(tester.application(application.id()).change().hasTargets());
- tester.deployAndNotify(application, true, productionUsWest1);
- assertFalse(tester.application(application.id()).change().hasTargets());
+ tester.deployAndNotify(instance, true, systemTest);
+ tester.deployAndNotify(instance, true, stagingTest);
+ tester.deployAndNotify(instance, true, productionUsEast3);
+ assertTrue(tester.application(instance.id()).change().hasTargets());
+ tester.deployAndNotify(instance, true, productionUsWest1);
+ assertFalse(tester.application(instance.id()).change().hasTargets());
}
@Test
@@ -1250,8 +1250,8 @@ public class UpgraderTest {
tester.upgrader().setTargetMajorVersion(Optional.of(6));
// All applications deploy on current version
- Application app1 = tester.createAndDeploy("app1", 1, "default");
- Application app2 = tester.createAndDeploy("app2", 1, "default");
+ Instance app1 = tester.createAndDeploy("app1", 1, "default");
+ Instance app2 = tester.createAndDeploy("app2", 1, "default");
// Keep app 1 on current version
tester.controller().applications().lockIfPresent(app1.id(), app -> tester.controller().applications().store(app.withChange(app.get().change().withPin())));
@@ -1283,8 +1283,8 @@ public class UpgraderTest {
@Test
public void testsEachUpgradeCombinationWithFailingDeployments() {
DeploymentTester tester = new DeploymentTester();
- Application application = tester.createApplication("app1", "tenant1", 1, 1L);
- Supplier<Application> app = () -> tester.application(application.id());
+ Instance instance = tester.createApplication("app1", "tenant1", 1, 1L);
+ Supplier<Instance> app = () -> tester.application(instance.id());
ApplicationPackage applicationPackage = new ApplicationPackageBuilder()
.environment(Environment.prod)
.region("us-central-1")
@@ -1294,22 +1294,22 @@ public class UpgraderTest {
// Application deploys on system version
Version v1 = Version.fromString("6.1");
- tester.deployCompletely(application, applicationPackage);
+ tester.deployCompletely(instance, applicationPackage);
// Next version is released and 2/3 deployments upgrade
Version v2 = Version.fromString("6.2");
tester.upgradeSystem(v2);
tester.upgrader().maintain();
assertEquals(Change.of(v2), app.get().change());
- tester.deployAndNotify(application, true, systemTest);
- tester.deployAndNotify(application, true, stagingTest);
- tester.deployAndNotify(application, true, productionUsCentral1);
+ tester.deployAndNotify(instance, true, systemTest);
+ tester.deployAndNotify(instance, true, stagingTest);
+ tester.deployAndNotify(instance, true, productionUsCentral1);
// While second deployment completes upgrade, version confidence becomes broken and upgrade is cancelled
tester.upgrader().overrideConfidence(v2, VespaVersion.Confidence.broken);
tester.computeVersionStatus();
tester.upgrader().maintain();
- tester.deployAndNotify(application, true, productionUsWest1);
+ tester.deployAndNotify(instance, true, productionUsWest1);
assertTrue(app.get().change().isEmpty());
// Next version is released
@@ -1317,21 +1317,21 @@ public class UpgraderTest {
tester.upgradeSystem(v3);
tester.upgrader().maintain();
assertEquals(Change.of(v3), app.get().change());
- tester.deployAndNotify(application, true, systemTest);
- tester.deployAndNotify(application, true, stagingTest);
+ tester.deployAndNotify(instance, true, systemTest);
+ tester.deployAndNotify(instance, true, stagingTest);
// First deployment starts upgrading
- tester.deploy(productionUsCentral1, application, applicationPackage);
+ tester.deploy(productionUsCentral1, instance, applicationPackage);
// Before deployment completes, v1->v3 combination is tested as us-east-3 is still on v1
tester.readyJobTrigger().maintain();
- tester.deployAndNotify(application, true, stagingTest);
+ tester.deployAndNotify(instance, true, stagingTest);
assertEquals(v1, app.get().deploymentJobs().jobStatus().get(stagingTest).lastSuccess().get().sourcePlatform().get());
assertEquals(v3, app.get().deploymentJobs().jobStatus().get(stagingTest).lastSuccess().get().platform());
// First deployment fails and then successfully upgrades to v3
- tester.jobCompletion(productionUsCentral1).application(application).unsuccessful().submit();
- tester.jobCompletion(productionUsCentral1).application(application).submit();
+ tester.jobCompletion(productionUsCentral1).application(instance).unsuccessful().submit();
+ tester.jobCompletion(productionUsCentral1).application(instance).submit();
// Deployments are now on 3 versions
assertEquals(v3, app.get().deployments().get(productionUsCentral1.zone(main)).version());
@@ -1342,19 +1342,19 @@ public class UpgraderTest {
tester.readyJobTrigger().maintain();
assertEquals(v2, app.get().deploymentJobs().jobStatus().get(stagingTest).lastTriggered().get().sourcePlatform().get());
assertEquals(v3, app.get().deploymentJobs().jobStatus().get(stagingTest).lastTriggered().get().platform());
- tester.deployAndNotify(application, true, stagingTest);
+ tester.deployAndNotify(instance, true, stagingTest);
// Second deployment upgrades
- tester.deployAndNotify(application, true, productionUsWest1);
+ tester.deployAndNotify(instance, true, productionUsWest1);
// ... now we have to test v1->v3 again :(
tester.readyJobTrigger().maintain();
assertEquals(v1, app.get().deploymentJobs().jobStatus().get(stagingTest).lastTriggered().get().sourcePlatform().get());
assertEquals(v3, app.get().deploymentJobs().jobStatus().get(stagingTest).lastTriggered().get().platform());
- tester.deployAndNotify(application, true, stagingTest);
+ tester.deployAndNotify(instance, true, stagingTest);
// Upgrade completes
- tester.deployAndNotify(application, true, productionUsEast3);
+ tester.deployAndNotify(instance, true, productionUsEast3);
assertTrue("Upgrade complete", app.get().change().isEmpty());
}