Remove default flavors

1 files changed, 0 insertions, 68 deletions

diff --git a/config-provisioning/src/main/java/com/yahoo/config/provision/Zone.java b/config-provisioning/src/main/java/com/yahoo/config/provision/Zone.java
index f2e32b1b996..eebedd15e2c 100644
--- a/config-provisioning/src/main/java/com/yahoo/config/provision/Zone.java
+++ b/config-provisioning/src/main/java/com/yahoo/config/provision/Zone.java
@@ -21,7 +21,6 @@ public class Zone {
     private final SystemName systemName;
     private final Environment environment;
     private final RegionName region;
-    private final FlavorDefaults flavorDefaults;
     private final Optional<NodeFlavors> nodeFlavors;
 
     @Inject
@@ -30,7 +29,6 @@ public class Zone {
              SystemName.from(configserverConfig.system()),
              Environment.from(configserverConfig.environment()),
              RegionName.from(configserverConfig.region()),
-             new FlavorDefaults(configserverConfig),
              nodeFlavors);
     }
 
@@ -50,25 +48,15 @@ public class Zone {
     }
 
     /** Create from cloud, system, environment, region and node flavors. Use for testing. */
-    public Zone(CloudName cloudName,
-                SystemName systemName,
-                Environment environment,
-                RegionName region,
-                NodeFlavors nodeFlavors) {
-        this(cloudName, systemName, environment, region, new FlavorDefaults("default"), nodeFlavors);
-    }
-
     private Zone(CloudName cloudName,
                  SystemName systemName,
                  Environment environment,
                  RegionName region,
-                 FlavorDefaults flavorDefaults,
                  NodeFlavors nodeFlavors) {
         this.cloudName = cloudName;
         this.systemName = systemName;
         this.environment = environment;
         this.region = region;
-        this.flavorDefaults = flavorDefaults;
         this.nodeFlavors = Optional.ofNullable(nodeFlavors);
     }
 
@@ -88,9 +76,6 @@ public class Zone {
         return region;
     }
 
-    /** Returns the default hardware flavor to assign in this zone */
-    public String defaultFlavor(ClusterSpec.Type clusterType) { return flavorDefaults.flavor(clusterType); }
-
     /** Returns all available node flavors for the zone, or empty if not set for this Zone. */
     public Optional<NodeFlavors> nodeFlavors() { return nodeFlavors; }
 
@@ -118,58 +103,5 @@ public class Zone {
         return Objects.hash(environment, region);
     }
 
-    private static class FlavorDefaults {
-
-        /** The default default flavor */
-        private final String defaultFlavor;
-
-        /** The default flavor for each cluster type, or empty to use defaultFlavor */
-        private final Optional<String> adminFlavor;
-        private final Optional<String> containerFlavor;
-        private final Optional<String> contentFlavor;
-
-        /** Creates this with a default flavor and all cluster type flavors empty */
-        public FlavorDefaults(String defaultFlavor) {
-            this(defaultFlavor, Optional.empty(), Optional.empty(), Optional.empty());
-        }
-
-        /** Creates this with a default flavor and all cluster type flavors empty */
-        public FlavorDefaults(String defaultFlavor,
-                              Optional<String> adminFlavor, Optional<String> containerFlavor, Optional<String> contentFlavor) {
-            this.defaultFlavor = defaultFlavor;
-            this.adminFlavor = adminFlavor;
-            this.containerFlavor = containerFlavor;
-            this.contentFlavor = contentFlavor;
-        }
-
-        public FlavorDefaults(ConfigserverConfig config) {
-            this(config.defaultFlavor(),
-                 emptyIfDefault(config.defaultAdminFlavor()),
-                 emptyIfDefault(config.defaultContainerFlavor()),
-                 emptyIfDefault(config.defaultContentFlavor()));
-        }
-
-        /** Map "default" to empty - this config cannot have missing values due to the need for supporting non-hosted */
-        private static Optional<String> emptyIfDefault(String value) {
-            if (Strings.isNullOrEmpty(value)) return Optional.empty();
-            if (value.equals("default")) return Optional.empty();
-            return Optional.of(value);
-        }
-
-        /**
-         * Returns the flavor default for a given cluster type.
-         * This may be "default" - which is an invalid value - but never null.
-         */
-        public String flavor(ClusterSpec.Type clusterType) {
-            switch (clusterType) {
-                case admin: return adminFlavor.orElse(defaultFlavor);
-                case container: return containerFlavor.orElse(defaultFlavor);
-                case content: return contentFlavor.orElse(defaultFlavor);
-                default: return defaultFlavor; // future cluster types
-            }
-        }
-
-    }
-
 }