summaryrefslogtreecommitdiffstats
path: root/controller-server/src
diff options
context:
space:
mode:
authorHÃ¥kon Hallingstad <hakon.hallingstad@gmail.com>2022-02-03 18:45:03 +0100
committerGitHub <noreply@github.com>2022-02-03 18:45:03 +0100
commit291c514e6eadfc6dd1c02b87c6d05303a9872a27 (patch)
treef7e5a40f2c8f9c60540775b7b3540c38e60424df /controller-server/src
parentd56accc88cc8bc2d4b29c4647586e4e262c9ea39 (diff)
Revert "Set archiveUrl for controllers"
Diffstat (limited to 'controller-server/src')
-rw-r--r--controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/ArchiveUriUpdater.java8
-rw-r--r--controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/ArchiveUriUpdaterTest.java3
2 files changed, 2 insertions, 9 deletions
diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/ArchiveUriUpdater.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/ArchiveUriUpdater.java
index 36ab2e6f384..7d7fa71e72c 100644
--- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/ArchiveUriUpdater.java
+++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/ArchiveUriUpdater.java
@@ -29,8 +29,8 @@ public class ArchiveUriUpdater extends ControllerMaintainer {
private final NodeRepository nodeRepository;
private final CuratorArchiveBucketDb archiveBucketDb;
- public ArchiveUriUpdater(Controller controller, Duration interval) {
- super(controller, interval);
+ public ArchiveUriUpdater(Controller controller, Duration duration) {
+ super(controller, duration);
this.applications = controller.applications();
this.nodeRepository = controller.serviceRegistry().configServer().nodeRepository();
this.archiveBucketDb = controller.archiveBucketDb();
@@ -39,10 +39,6 @@ public class ArchiveUriUpdater extends ControllerMaintainer {
@Override
protected double maintain() {
Map<ZoneId, Set<TenantName>> tenantsByZone = new HashMap<>();
-
- tenantsByZone.put(controller().zoneRegistry().systemZone().getVirtualId(),
- new HashSet<>(INFRASTRUCTURE_TENANTS));
-
for (var application : applications.asList()) {
for (var instance : application.instances().values()) {
for (var deployment : instance.deployments().values()) {
diff --git a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/ArchiveUriUpdaterTest.java b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/ArchiveUriUpdaterTest.java
index 0a2f5d9a236..451991f9604 100644
--- a/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/ArchiveUriUpdaterTest.java
+++ b/controller-server/src/test/java/com/yahoo/vespa/hosted/controller/maintenance/ArchiveUriUpdaterTest.java
@@ -43,9 +43,6 @@ public class ArchiveUriUpdaterTest {
// Initially we should not set any archive URIs as the archive service does not return any
updater.maintain();
assertArchiveUris(Map.of(), zone);
- // but the controller zone is always present
- assertArchiveUris(Map.of(TenantName.from("hosted-vespa"), "s3://bucketName/hosted-vespa/"),
- ZoneId.from("prod", "controller"));
// Archive service now has URI for tenant1, but tenant1 is not deployed in zone
setBucketNameInService(Map.of(tenant1, "uri-1"), zone);