diff options
author | Jon Bratseth <bratseth@verizonmedia.com> | 2019-05-28 10:26:56 +0200 |
---|---|---|
committer | Jon Bratseth <bratseth@verizonmedia.com> | 2019-05-28 10:26:56 +0200 |
commit | d00fc3b5a1cced343bc02aaeb513539abfa44096 (patch) | |
tree | ebb484a3c6da3d0c38d42b4fbf189ab76631e006 /config-provisioning | |
parent | e52c1f88a1ead27e7f4b9a5b108647abefa2adf1 (diff) |
Remove default flavors
Diffstat (limited to 'config-provisioning')
-rw-r--r-- | config-provisioning/abi-spec.json | 3 | ||||
-rw-r--r-- | config-provisioning/src/main/java/com/yahoo/config/provision/Zone.java | 68 |
2 files changed, 1 insertions, 70 deletions
diff --git a/config-provisioning/abi-spec.json b/config-provisioning/abi-spec.json index 2050721fef1..2b40b786431 100644 --- a/config-provisioning/abi-spec.json +++ b/config-provisioning/abi-spec.json @@ -141,6 +141,7 @@ "public com.yahoo.config.provision.NodeType type()", "public java.lang.String toString()", "public static com.yahoo.config.provision.Capacity fromNodeCount(int)", + "public static com.yahoo.config.provision.Capacity fromCount(int, com.yahoo.config.provision.NodeResources)", "public static com.yahoo.config.provision.Capacity fromCount(int, com.yahoo.config.provision.NodeResources, boolean, boolean)", "public static com.yahoo.config.provision.Capacity fromCount(int, java.util.Optional, boolean, boolean)", "public static com.yahoo.config.provision.Capacity fromNodeCount(int, java.util.Optional, boolean, boolean)", @@ -822,12 +823,10 @@ "public void <init>(com.yahoo.config.provision.Environment, com.yahoo.config.provision.RegionName)", "public void <init>(com.yahoo.config.provision.SystemName, com.yahoo.config.provision.Environment, com.yahoo.config.provision.RegionName)", "public void <init>(com.yahoo.config.provision.CloudName, com.yahoo.config.provision.SystemName, com.yahoo.config.provision.Environment, com.yahoo.config.provision.RegionName)", - "public void <init>(com.yahoo.config.provision.CloudName, com.yahoo.config.provision.SystemName, com.yahoo.config.provision.Environment, com.yahoo.config.provision.RegionName, com.yahoo.config.provision.NodeFlavors)", "public com.yahoo.config.provision.CloudName cloud()", "public com.yahoo.config.provision.SystemName system()", "public com.yahoo.config.provision.Environment environment()", "public com.yahoo.config.provision.RegionName region()", - "public java.lang.String defaultFlavor(com.yahoo.config.provision.ClusterSpec$Type)", "public java.util.Optional nodeFlavors()", "public static com.yahoo.config.provision.Zone defaultZone()", "public java.lang.String toString()", 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 - } - } - - } - } |