From 5b65bebb1408d5217e303b8281751c252bb626ce Mon Sep 17 00:00:00 2001 From: Jon Bratseth Date: Thu, 28 Apr 2022 22:45:20 +0200 Subject: Aliases were not used here --- .../java/com/yahoo/config/provision/HostSpec.java | 23 ++++++++++------------ .../serialization/AllocatedHostsSerializer.java | 17 ---------------- .../AllocatedHostsSerializerTest.java | 1 - 3 files changed, 10 insertions(+), 31 deletions(-) (limited to 'config-provisioning/src') diff --git a/config-provisioning/src/main/java/com/yahoo/config/provision/HostSpec.java b/config-provisioning/src/main/java/com/yahoo/config/provision/HostSpec.java index 11f8da95059..1818046e810 100644 --- a/config-provisioning/src/main/java/com/yahoo/config/provision/HostSpec.java +++ b/config-provisioning/src/main/java/com/yahoo/config/provision/HostSpec.java @@ -18,9 +18,6 @@ public class HostSpec implements Comparable { /** The name of this host */ private final String hostname; - /** Aliases of this host */ - private final List aliases; - private final NodeResources realResources; private final NodeResources advertisedResources; private final NodeResources requestedResources; @@ -35,8 +32,15 @@ public class HostSpec implements Comparable { private final Optional networkPorts; /** Create a host in a non-cloud system, where hosts are specified in config */ - public HostSpec(String hostname, List aliases, Optional networkPorts) { - this(hostname, aliases, + public HostSpec(String hostname, Optional networkPorts) { + this(hostname, + NodeResources.unspecified(), NodeResources.unspecified(), NodeResources.unspecified(), + Optional.empty(), Optional.empty(), networkPorts, Optional.empty()); + } + + // TODO: Remove after May 2022 + public HostSpec(String hostname, List ignored, Optional networkPorts) { + this(hostname, NodeResources.unspecified(), NodeResources.unspecified(), NodeResources.unspecified(), Optional.empty(), Optional.empty(), networkPorts, Optional.empty()); } @@ -51,7 +55,6 @@ public class HostSpec implements Comparable { Optional networkPorts, Optional dockerImageRepo) { this(hostname, - List.of(), realResources, advertisedResources, requestedResources, @@ -62,7 +65,6 @@ public class HostSpec implements Comparable { } private HostSpec(String hostname, - List aliases, NodeResources realResources, NodeResources advertisedResurces, NodeResources requestedResources, @@ -72,7 +74,6 @@ public class HostSpec implements Comparable { Optional dockerImageRepo) { if (hostname == null || hostname.isEmpty()) throw new IllegalArgumentException("Hostname must be specified"); this.hostname = hostname; - this.aliases = List.copyOf(aliases); this.realResources = Objects.requireNonNull(realResources); this.advertisedResources = Objects.requireNonNull(advertisedResurces); this.requestedResources = Objects.requireNonNull(requestedResources, "RequestedResources cannot be null"); @@ -85,9 +86,6 @@ public class HostSpec implements Comparable { /** Returns the name identifying this host */ public String hostname() { return hostname; } - /** Returns the aliases of this host as an immutable list. This may be empty but never null. */ - public List aliases() { return aliases; } - /** The real resources available for Vespa processes on this node, after subtracting infrastructure overhead. */ public NodeResources realResources() { return realResources; } @@ -109,13 +107,12 @@ public class HostSpec implements Comparable { public Optional dockerImageRepo() { return dockerImageRepo; } public HostSpec withPorts(Optional ports) { - return new HostSpec(hostname, aliases, realResources, advertisedResources, requestedResources, membership, version, ports, dockerImageRepo); + return new HostSpec(hostname, realResources, advertisedResources, requestedResources, membership, version, ports, dockerImageRepo); } @Override public String toString() { return hostname + - (! aliases.isEmpty() ? " (aliases: " + aliases + ")" : "") + (membership.isPresent() ? " (membership: " + membership.get() + ")" : " (no membership)"); } diff --git a/config-provisioning/src/main/java/com/yahoo/config/provision/serialization/AllocatedHostsSerializer.java b/config-provisioning/src/main/java/com/yahoo/config/provision/serialization/AllocatedHostsSerializer.java index d7d8d9647d3..5db7303f4bf 100644 --- a/config-provisioning/src/main/java/com/yahoo/config/provision/serialization/AllocatedHostsSerializer.java +++ b/config-provisioning/src/main/java/com/yahoo/config/provision/serialization/AllocatedHostsSerializer.java @@ -39,7 +39,6 @@ public class AllocatedHostsSerializer { private static final String mappingKey = "mapping"; private static final String hostSpecKey = "hostSpec"; private static final String hostSpecHostNameKey = "hostName"; - private static final String aliasesKey = "aliases"; private static final String hostSpecMembershipKey = "membership"; private static final String realResourcesKey = "realResources"; @@ -78,7 +77,6 @@ public class AllocatedHostsSerializer { private static void toSlime(HostSpec host, Cursor object) { object.setString(hostSpecHostNameKey, host.hostname()); - aliasesToSlime(host, object); host.membership().ifPresent(membership -> { object.setString(hostSpecMembershipKey, membership.stringValue()); object.setString(hostSpecVespaVersionKey, membership.cluster().vespaVersion().toFullString()); @@ -91,13 +89,6 @@ public class AllocatedHostsSerializer { host.networkPorts().ifPresent(ports -> NetworkPortsSerializer.toSlime(ports, object.setArray(hostSpecNetworkPortsKey))); } - private static void aliasesToSlime(HostSpec spec, Cursor cursor) { - if (spec.aliases().isEmpty()) return; - Cursor aliases = cursor.setArray(aliasesKey); - for (String alias : spec.aliases()) - aliases.addString(alias); - } - private static void toSlime(NodeResources resources, Cursor resourcesObject) { resourcesObject.setDouble(vcpuKey, resources.vcpu()); resourcesObject.setDouble(memoryKey, resources.memoryGb()); @@ -135,18 +126,10 @@ public class AllocatedHostsSerializer { } else { return new HostSpec(object.field(hostSpecHostNameKey).asString(), - aliasesFromSlime(object), NetworkPortsSerializer.fromSlime(object.field(hostSpecNetworkPortsKey))); } } - private static List aliasesFromSlime(Inspector object) { - if ( ! object.field(aliasesKey).valid()) return List.of(); - List aliases = new ArrayList<>(); - object.field(aliasesKey).traverse((ArrayTraverser)(index, alias) -> aliases.add(alias.asString())); - return aliases; - } - private static NodeResources nodeResourcesFromSlime(Inspector resources) { return new NodeResources(resources.field(vcpuKey).asDouble(), resources.field(memoryKey).asDouble(), diff --git a/config-provisioning/src/test/java/com/yahoo/config/provision/serialization/AllocatedHostsSerializerTest.java b/config-provisioning/src/test/java/com/yahoo/config/provision/serialization/AllocatedHostsSerializerTest.java index 3867664a3a6..ed87ff9f555 100644 --- a/config-provisioning/src/test/java/com/yahoo/config/provision/serialization/AllocatedHostsSerializerTest.java +++ b/config-provisioning/src/test/java/com/yahoo/config/provision/serialization/AllocatedHostsSerializerTest.java @@ -98,7 +98,6 @@ public class AllocatedHostsSerializerTest { assertEquals(expectedHost.requestedResources(), deserializedHost.requestedResources()); assertEquals(expectedHost.version(), deserializedHost.version()); assertEquals(expectedHost.networkPorts(), deserializedHost.networkPorts()); - assertEquals(expectedHost.aliases(), deserializedHost.aliases()); assertEquals(expectedHost.dockerImageRepo(), deserializedHost.dockerImageRepo()); } } -- cgit v1.2.3