Remove HostRenamer

vespa-engine · Dec 4, 2024 · b107293 · b107293
1 parent 2cb782f
commit b107293
Show file tree

Hide file tree

Showing 5 changed files with 0 additions and 258 deletions.
diff --git a/flags/src/main/java/com/yahoo/vespa/flags/Flags.java b/flags/src/main/java/com/yahoo/vespa/flags/Flags.java
@@ -462,14 +462,6 @@ public class Flags {
             "Whether to sync companies to HubSpot",
             "Takes effect immediately");
 
-    public static final UnboundStringFlag HOSTNAME_SCHEME = defineStringFlag(
-            "hostname-scheme", "legacy",
-            List.of("mpolden"), "2024-08-07", "2025-01-15",
-            "The hostname scheme to use when provisioning hosts. Must be 'legacy' or 'standard",
-            "Takes effect immediately",
-            (value) -> "legacy".equals(value) || "standard".equals(value),
-            TENANT_ID, APPLICATION, INSTANCE_ID);
-
     public static final UnboundBooleanFlag SNAPSHOTS_ENABLED = defineFeatureFlag(
             "snapshots-enabled", false,
             List.of("mpolden"), "2024-10-22", "2025-01-10",

diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/HostRenamer.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/HostRenamer.java
diff --git a/...src/main/java/com/yahoo/vespa/hosted/provision/maintenance/NodeRepositoryMaintenance.java b/...src/main/java/com/yahoo/vespa/hosted/provision/maintenance/NodeRepositoryMaintenance.java
@@ -63,7 +63,6 @@ public NodeRepositoryMaintenance(NodeRepository nodeRepository, Deployer deploye
         maintainers.add(new ScalingSuggestionsMaintainer(nodeRepository, defaults.scalingSuggestionsInterval, metric));
         maintainers.add(new SwitchRebalancer(nodeRepository, defaults.switchRebalancerInterval, metric, deployer));
         maintainers.add(new DeprovisionedExpirer(nodeRepository, defaults.deprovisionedExpiry, metric));
-        maintainers.add(new HostRenamer(nodeRepository, defaults.hostRenamerInterval, metric));
         maintainers.add(new SnapshotExpirer(nodeRepository, defaults.snapshotExpirerInterval, metric));
 
         provisionServiceProvider.getLoadBalancerService()
@@ -130,7 +129,6 @@ private static class DefaultTimes {
         private final Duration hostRetirerInterval;
         private final Duration hostFlavorUpgraderInterval;
         private final Duration deprovisionedExpiry;
-        private final Duration hostRenamerInterval;
         private final Duration snapshotExpirerInterval;
 
         private final NodeFailer.ThrottlePolicy throttlePolicy;
@@ -172,7 +170,6 @@ private static class DefaultTimes {
             hostFlavorUpgraderInterval = Duration.ofHours(1);
             // CD, test and staging (de)provisions hosts frequently. Expire deprovisioned ones earlier
             deprovisionedExpiry = (isCdZone || isTest) ? Duration.ofDays(3) : Duration.ofDays(30);
-            hostRenamerInterval = Duration.ofMinutes(5);
             snapshotExpirerInterval = Duration.ofHours(1);
 
             if (isProduction && ! isCdZone) {

diff --git a/...epository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/HostRenamerTest.java b/...epository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/HostRenamerTest.java
diff --git a/...ository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/responses/maintenance.json b/...ository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/responses/maintenance.json
@@ -16,9 +16,6 @@
     {
       "name": "FailedExpirer"
     },
-    {
-      "name": "HostRenamer"
-    },
     {
       "name": "InactiveExpirer"
     },
-Original file line number
+Diff line change
@@ Expand Up / @@ -16,9 +16,6 @@ @@
         {
           "name": "FailedExpirer"
         },
-        {
-          "name": "HostRenamer"
-        },
         {
           "name": "InactiveExpirer"
         },
@@ Expand Down @@