diff options
52 files changed, 116 insertions, 109 deletions
diff --git a/athenz-identity-provider-service/src/main/java/com/yahoo/vespa/hosted/athenz/instanceproviderservice/IdentityDocumentGenerator.java b/athenz-identity-provider-service/src/main/java/com/yahoo/vespa/hosted/athenz/instanceproviderservice/IdentityDocumentGenerator.java index 9b853604537..83b9bbb37da 100644 --- a/athenz-identity-provider-service/src/main/java/com/yahoo/vespa/hosted/athenz/instanceproviderservice/IdentityDocumentGenerator.java +++ b/athenz-identity-provider-service/src/main/java/com/yahoo/vespa/hosted/athenz/instanceproviderservice/IdentityDocumentGenerator.java @@ -3,13 +3,12 @@ package com.yahoo.vespa.hosted.athenz.instanceproviderservice; import com.google.inject.Inject; import com.yahoo.config.provision.Zone; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import com.yahoo.vespa.athenz.api.AthenzService; import com.yahoo.vespa.athenz.identityprovider.api.IdentityType; import com.yahoo.vespa.athenz.identityprovider.api.SignedIdentityDocument; import com.yahoo.vespa.athenz.identityprovider.api.VespaUniqueInstanceId; import com.yahoo.vespa.athenz.identityprovider.client.IdentityDocumentSigner; -import com.yahoo.vespa.hosted.athenz.instanceproviderservice.KeyProvider; import com.yahoo.vespa.hosted.athenz.instanceproviderservice.config.AthenzProviderServiceConfig; import com.yahoo.vespa.hosted.provision.Node; import com.yahoo.vespa.hosted.provision.NodeRepository; @@ -64,7 +63,7 @@ public class IdentityDocumentGenerator { PrivateKey privateKey = keyProvider.getPrivateKey(athenzProviderServiceConfig.secretVersion()); AthenzService providerService = new AthenzService(athenzProviderServiceConfig.domain(), athenzProviderServiceConfig.serviceName()); - String configServerHostname = HostName.getLocalhost(); + String configServerHostname = Hostnames.getLocalhost(); Instant createdAt = Instant.now(); String signature = signer.generateSignature( providerUniqueId, providerService, configServerHostname, diff --git a/clustercontroller-core/src/main/java/com/yahoo/vespa/clustercontroller/core/rpc/RpcServer.java b/clustercontroller-core/src/main/java/com/yahoo/vespa/clustercontroller/core/rpc/RpcServer.java index 4bd9b2fc9b8..26ac426f935 100644 --- a/clustercontroller-core/src/main/java/com/yahoo/vespa/clustercontroller/core/rpc/RpcServer.java +++ b/clustercontroller-core/src/main/java/com/yahoo/vespa/clustercontroller/core/rpc/RpcServer.java @@ -16,7 +16,7 @@ import com.yahoo.jrt.slobrok.api.BackOffPolicy; import com.yahoo.jrt.slobrok.api.Register; import com.yahoo.jrt.slobrok.api.SlobrokList; import java.util.logging.Level; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import com.yahoo.vdslib.state.ClusterState; import com.yahoo.vdslib.state.Node; import com.yahoo.vdslib.state.NodeState; @@ -111,7 +111,7 @@ public class RpcServer { slobroks.append(" )"); SlobrokList slist = new SlobrokList(); slist.setup(slobrokConnectionSpecs); - Spec spec = new Spec(HostName.getLocalhost(), acceptor.port()); + Spec spec = new Spec(Hostnames.getLocalhost(), acceptor.port()); log.log(Level.INFO, "Registering " + spec + " with slobrok at " + slobroks); if (slobrokBackOffPolicy != null) { register = new Register(supervisor, slist, spec, slobrokBackOffPolicy); diff --git a/clustercontroller-core/src/test/java/com/yahoo/vespa/clustercontroller/core/ZooKeeperTestServer.java b/clustercontroller-core/src/test/java/com/yahoo/vespa/clustercontroller/core/ZooKeeperTestServer.java index baaad164f8f..b35ff6ebfd0 100644 --- a/clustercontroller-core/src/test/java/com/yahoo/vespa/clustercontroller/core/ZooKeeperTestServer.java +++ b/clustercontroller-core/src/test/java/com/yahoo/vespa/clustercontroller/core/ZooKeeperTestServer.java @@ -1,7 +1,7 @@ // Copyright Yahoo. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. package com.yahoo.vespa.clustercontroller.core; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import org.apache.zookeeper.server.NIOServerCnxnFactory; import org.apache.zookeeper.server.ZooKeeperServer; @@ -52,7 +52,7 @@ public class ZooKeeperTestServer { } String getAddress() { - return HostName.getLocalhost() + ":" + getPort(); + return Hostnames.getLocalhost() + ":" + getPort(); } public void shutdown(boolean cleanupZooKeeperDir) { diff --git a/clustercontroller-reindexer/src/main/java/ai/vespa/reindexing/ReindexingMaintainer.java b/clustercontroller-reindexer/src/main/java/ai/vespa/reindexing/ReindexingMaintainer.java index afb491fa293..27d211e4bc5 100644 --- a/clustercontroller-reindexer/src/main/java/ai/vespa/reindexing/ReindexingMaintainer.java +++ b/clustercontroller-reindexer/src/main/java/ai/vespa/reindexing/ReindexingMaintainer.java @@ -12,7 +12,7 @@ import com.yahoo.concurrent.DaemonThreadFactory; import com.yahoo.document.DocumentTypeManager; import com.yahoo.documentapi.DocumentAccess; import com.yahoo.jdisc.Metric; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import com.yahoo.vespa.config.content.AllClustersBucketSpacesConfig; import com.yahoo.vespa.config.content.reindexing.ReindexingConfig; import com.yahoo.vespa.curator.Curator; @@ -75,7 +75,7 @@ public class ReindexingMaintainer extends AbstractComponent { this.executor = new ScheduledThreadPoolExecutor(reindexingConfig.clusters().size(), new DaemonThreadFactory("reindexer-")); if (reindexingConfig.enabled()) scheduleStaggered((delayMillis, intervalMillis) -> executor.scheduleAtFixedRate(this::maintain, delayMillis, intervalMillis, TimeUnit.MILLISECONDS), - Duration.ofMinutes(1), clock.instant(), HostName.getLocalhost(), zookeepersConfig.zookeeperserverlist()); + Duration.ofMinutes(1), clock.instant(), Hostnames.getLocalhost(), zookeepersConfig.zookeeperserverlist()); } private void maintain() { diff --git a/config-model/src/main/java/com/yahoo/config/model/provision/Hosts.java b/config-model/src/main/java/com/yahoo/config/model/provision/Hosts.java index c9a03dad65e..af1e94e14f2 100644 --- a/config-model/src/main/java/com/yahoo/config/model/provision/Hosts.java +++ b/config-model/src/main/java/com/yahoo/config/model/provision/Hosts.java @@ -3,7 +3,7 @@ package com.yahoo.config.model.provision; import com.google.common.collect.ImmutableMap; import com.yahoo.config.model.builder.xml.XmlHelper; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import com.yahoo.text.XML; import com.yahoo.vespa.model.builder.xml.dom.VespaDomBuilder; import org.w3c.dom.Document; @@ -75,7 +75,7 @@ public class Hosts { throw new IllegalArgumentException("Missing 'name' attribute for host."); } if ("localhost".equals(name)) { - name = HostName.getLocalhost(); + name = Hostnames.getLocalhost(); } List<String> hostAliases = VespaDomBuilder.getHostAliases(hostE.getChildNodes()); if (hostAliases.isEmpty()) { diff --git a/config-model/src/main/java/com/yahoo/config/model/provision/SingleNodeProvisioner.java b/config-model/src/main/java/com/yahoo/config/model/provision/SingleNodeProvisioner.java index c3b91c5da0f..d3a6ad9897f 100644 --- a/config-model/src/main/java/com/yahoo/config/model/provision/SingleNodeProvisioner.java +++ b/config-model/src/main/java/com/yahoo/config/model/provision/SingleNodeProvisioner.java @@ -9,7 +9,7 @@ import com.yahoo.config.provision.Flavor; import com.yahoo.config.provision.HostSpec; import com.yahoo.config.provision.NodeResources; import com.yahoo.config.provision.ProvisionLogger; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import java.util.ArrayList; import java.util.List; @@ -29,12 +29,12 @@ public class SingleNodeProvisioner implements HostProvisioner { private int counter = 0; public SingleNodeProvisioner() { - host = new Host(HostName.getLocalhost()); + host = new Host(Hostnames.getLocalhost()); this.hostSpec = new HostSpec(host.hostname(), host.aliases(), Optional.empty()); } public SingleNodeProvisioner(Flavor flavor) { - host = new Host(HostName.getLocalhost()); + host = new Host(Hostnames.getLocalhost()); this.hostSpec = new HostSpec(host.hostname(), flavor.resources(), flavor.resources(), flavor.resources(), ClusterMembership.from(ClusterSpec.specification(ClusterSpec.Type.content, ClusterSpec.Id.from("test")).group(ClusterSpec.Group.from(0)).vespaVersion("1").build(), 0), diff --git a/config-model/src/main/java/com/yahoo/vespa/model/HostSystem.java b/config-model/src/main/java/com/yahoo/vespa/model/HostSystem.java index 6e63a1d9e68..640210717b9 100644 --- a/config-model/src/main/java/com/yahoo/vespa/model/HostSystem.java +++ b/config-model/src/main/java/com/yahoo/vespa/model/HostSystem.java @@ -9,7 +9,7 @@ import com.yahoo.config.provision.ClusterMembership; import com.yahoo.config.provision.ClusterSpec; import com.yahoo.config.provision.HostSpec; import com.yahoo.config.provision.ProvisionLogger; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import java.net.UnknownHostException; import java.util.LinkedHashMap; @@ -78,7 +78,7 @@ public class HostSystem extends AbstractConfigProducer<Host> { HostResource hostResource = hostname2host.get(name); if (hostResource == null) { // Create a new HostResource if this is the host this code is running on (as it is when running tests) - if (HostName.getLocalhost().equals(name)) { + if (Hostnames.getLocalhost().equals(name)) { if (! getChildren().containsKey(localhost)) { new Host(this, localhost); } diff --git a/config-model/src/main/java/com/yahoo/vespa/model/container/configserver/ConfigserverCluster.java b/config-model/src/main/java/com/yahoo/vespa/model/container/configserver/ConfigserverCluster.java index 413c6f07efd..6ad18e195a8 100644 --- a/config-model/src/main/java/com/yahoo/vespa/model/container/configserver/ConfigserverCluster.java +++ b/config-model/src/main/java/com/yahoo/vespa/model/container/configserver/ConfigserverCluster.java @@ -12,7 +12,7 @@ import com.yahoo.config.provision.Zone; import com.yahoo.container.StatisticsConfig; import com.yahoo.container.core.VipStatusConfig; import com.yahoo.container.jdisc.config.HealthMonitorConfig; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import com.yahoo.vespa.defaults.Defaults; import com.yahoo.vespa.model.container.ContainerCluster; import com.yahoo.vespa.model.container.configserver.option.CloudConfigOptions; @@ -66,7 +66,7 @@ public class ConfigserverCluster extends AbstractConfigProducer configServers.length, zookeeperIds.length)); } - String myhostname = HostName.getLocalhost(); + String myhostname = Hostnames.getLocalhost(); // TODO: Server index should be in interval [1, 254] according to doc, // however, we cannot change this id for an existing server for (int i = 0; i < configServers.length; i++) { @@ -120,7 +120,7 @@ public class ConfigserverCluster extends AbstractConfigProducer builder.environment(options.system().get()); } - builder.serverId(HostName.getLocalhost()); + builder.serverId(Hostnames.getLocalhost()); if (!containerCluster.getHttp().getHttpServer().get().getConnectorFactories().isEmpty()) { builder.httpport(containerCluster.getHttp().getHttpServer().get().getConnectorFactories().get(0).getListenPort()); } @@ -150,7 +150,7 @@ public class ConfigserverCluster extends AbstractConfigProducer private ConfigServer[] getConfigServers() { return Optional.of(options.allConfigServers()) .filter(configServers -> configServers.length > 0) - .orElseGet(() -> new ConfigServer[]{new ConfigServer(HostName.getLocalhost(), Optional.empty())}); + .orElseGet(() -> new ConfigServer[]{new ConfigServer(Hostnames.getLocalhost(), Optional.empty())}); } private int[] getConfigServerZookeeperIds() { diff --git a/config-model/src/test/java/com/yahoo/config/model/deploy/SystemModelTestCase.java b/config-model/src/test/java/com/yahoo/config/model/deploy/SystemModelTestCase.java index 66cbfbd537f..9fea2cee88c 100644 --- a/config-model/src/test/java/com/yahoo/config/model/deploy/SystemModelTestCase.java +++ b/config-model/src/test/java/com/yahoo/config/model/deploy/SystemModelTestCase.java @@ -7,7 +7,7 @@ import com.yahoo.config.model.ConfigModel; import com.yahoo.config.model.ConfigModelRegistry; import com.yahoo.config.model.MapConfigModelRegistry; import com.yahoo.config.model.ApplicationConfigProducerRoot; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import com.yahoo.vespa.model.ConfigProducer; import com.yahoo.vespa.model.HostResource; import com.yahoo.vespa.model.HostSystem; @@ -83,7 +83,7 @@ public class SystemModelTestCase { // Verify configIds from vespaModel assertTrue(12 <= vespaModel.getConfigIds().size()); - String localhost = HostName.getLocalhost(); + String localhost = Hostnames.getLocalhost(); String localhostConfigId = "hosts/" + localhost; Set<String> configIds = vespaModel.getConfigIds(); assertTrue(configIds.contains("client")); diff --git a/config-model/src/test/java/com/yahoo/vespa/model/admin/AdminTestCase.java b/config-model/src/test/java/com/yahoo/vespa/model/admin/AdminTestCase.java index 96f6c7999cc..e2c316a57c5 100644 --- a/config-model/src/test/java/com/yahoo/vespa/model/admin/AdminTestCase.java +++ b/config-model/src/test/java/com/yahoo/vespa/model/admin/AdminTestCase.java @@ -16,7 +16,7 @@ import com.yahoo.config.provision.RegionName; import com.yahoo.config.provision.Zone; import com.yahoo.container.StatisticsConfig; import com.yahoo.container.jdisc.config.HealthMonitorConfig; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import com.yahoo.vespa.config.core.StateserverConfig; import com.yahoo.vespa.model.Service; import com.yahoo.vespa.model.VespaModel; @@ -60,7 +60,7 @@ public class AdminTestCase { // Verify configIds Set<String> configIds = vespaModel.getConfigIds(); - String localhost = HostName.getLocalhost(); + String localhost = Hostnames.getLocalhost(); String localhostConfigId = "hosts/" + localhost; assertTrue(configIds.contains(localhostConfigId)); assertTrue(configIds.contains("admin/logserver")); @@ -124,7 +124,7 @@ public class AdminTestCase { // Verify configIds Set<String> configIds = vespaModel.getConfigIds(); - String localhost = HostName.getLocalhost(); + String localhost = Hostnames.getLocalhost(); String localhostConfigId = "hosts/" + localhost; assertTrue(configIds.contains(localhostConfigId)); assertTrue(configIds.contains("admin/logserver")); @@ -164,7 +164,7 @@ public class AdminTestCase { .build())) .build(); TestRoot root = new TestDriver().buildModel(state); - String localhost = HostName.getLocalhost(); + String localhost = Hostnames.getLocalhost(); SentinelConfig config = root.getConfig(SentinelConfig.class, "hosts/" + localhost); assertEquals("quux", config.application().tenant()); assertEquals("foo", config.application().name()); @@ -187,7 +187,7 @@ public class AdminTestCase { // Verify configIds Set<String> configIds = vespaModel.getConfigIds(); - String localhost = HostName.getLocalhost(); + String localhost = Hostnames.getLocalhost(); String localhostConfigId = "hosts/" + localhost; assertTrue(configIds.contains(localhostConfigId)); assertTrue(configIds.contains("admin/logserver")); diff --git a/config-model/src/test/java/com/yahoo/vespa/model/container/HostProvisionerWithCustomRealResource.java b/config-model/src/test/java/com/yahoo/vespa/model/container/HostProvisionerWithCustomRealResource.java index e8f30d59aa9..e87388a0413 100644 --- a/config-model/src/test/java/com/yahoo/vespa/model/container/HostProvisionerWithCustomRealResource.java +++ b/config-model/src/test/java/com/yahoo/vespa/model/container/HostProvisionerWithCustomRealResource.java @@ -9,7 +9,7 @@ import com.yahoo.config.provision.ClusterSpec; import com.yahoo.config.provision.HostSpec; import com.yahoo.config.provision.NodeResources; import com.yahoo.config.provision.ProvisionLogger; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import java.util.List; import java.util.Optional; @@ -26,7 +26,7 @@ public class HostProvisionerWithCustomRealResource implements HostProvisioner { @Override public HostSpec allocateHost(String alias) { - Host host = new Host(HostName.getLocalhost()); + Host host = new Host(Hostnames.getLocalhost()); ClusterMembership membership = ClusterMembership.from( ClusterSpec .specification( diff --git a/config-model/src/test/java/com/yahoo/vespa/model/container/configserver/ConfigserverClusterTest.java b/config-model/src/test/java/com/yahoo/vespa/model/container/configserver/ConfigserverClusterTest.java index 29cff23d3bd..ecfade44f76 100644 --- a/config-model/src/test/java/com/yahoo/vespa/model/container/configserver/ConfigserverClusterTest.java +++ b/config-model/src/test/java/com/yahoo/vespa/model/container/configserver/ConfigserverClusterTest.java @@ -11,7 +11,7 @@ import com.yahoo.config.model.test.MockRoot; import com.yahoo.container.StatisticsConfig; import com.yahoo.container.di.config.PlatformBundlesConfig; import com.yahoo.container.jdisc.config.HealthMonitorConfig; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import com.yahoo.text.XML; import com.yahoo.vespa.defaults.Defaults; import com.yahoo.vespa.model.HostResource; @@ -108,7 +108,7 @@ public class ConfigserverClusterTest { assertEquals(Defaults.getDefaults().underVespaHome("lib/jars/config-models"), config.configModelPluginDir().get(0)); assertEquals(12345, config.rpcport()); assertEquals(1337, config.httpport()); - assertEquals(HostName.getLocalhost(), config.serverId()); + assertEquals(Hostnames.getLocalhost(), config.serverId()); assertTrue(config.useVespaVersionInRequest()); assertEquals(4, config.numParallelTenantLoaders()); assertFalse(config.multitenant()); diff --git a/config-model/src/test/java/com/yahoo/vespa/model/container/xml/ContainerModelBuilderTest.java b/config-model/src/test/java/com/yahoo/vespa/model/container/xml/ContainerModelBuilderTest.java index 16b4a8bed9b..04f1b418219 100644 --- a/config-model/src/test/java/com/yahoo/vespa/model/container/xml/ContainerModelBuilderTest.java +++ b/config-model/src/test/java/com/yahoo/vespa/model/container/xml/ContainerModelBuilderTest.java @@ -36,7 +36,7 @@ import com.yahoo.container.jdisc.JdiscBindingsConfig; import com.yahoo.container.jdisc.secretstore.SecretStoreConfig; import com.yahoo.container.usability.BindingsOverviewHandler; import com.yahoo.jdisc.http.ConnectorConfig; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import com.yahoo.path.Path; import com.yahoo.prelude.cluster.QrMonitorConfig; import com.yahoo.search.config.QrStartConfig; @@ -645,7 +645,7 @@ public class ContainerModelBuilderTest extends ContainerModelBuilderTestBase { @Test public void qrconfig_is_produced() throws IOException, SAXException { QrConfig qr = getQrConfig(new TestProperties()); - String hostname = HostName.getLocalhost(); // Using the same way of getting hostname as filedistribution model + String hostname = Hostnames.getLocalhost(); // Using the same way of getting hostname as filedistribution model assertEquals("default.container.0", qr.discriminator()); assertEquals(19102, qr.rpc().port()); assertEquals("vespa/service/default/container.0", qr.rpc().slobrokId()); diff --git a/config-model/src/test/java/com/yahoo/vespa/model/test/VespaModelTestCase.java b/config-model/src/test/java/com/yahoo/vespa/model/test/VespaModelTestCase.java index 7f8bca825d2..b8fbe960dc9 100644 --- a/config-model/src/test/java/com/yahoo/vespa/model/test/VespaModelTestCase.java +++ b/config-model/src/test/java/com/yahoo/vespa/model/test/VespaModelTestCase.java @@ -22,7 +22,7 @@ import com.yahoo.config.provision.AllocatedHosts; import com.yahoo.config.provision.ApplicationId; import com.yahoo.document.config.DocumentmanagerConfig; import com.yahoo.messagebus.MessagebusConfig; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import com.yahoo.vespa.config.UnknownConfigIdException; import com.yahoo.vespa.model.ConfigProducer; import com.yahoo.vespa.model.VespaModel; @@ -87,7 +87,7 @@ public class VespaModelTestCase { LogdConfig.Builder b = new LogdConfig.Builder(); b = (LogdConfig.Builder) model.getConfig(b, ""); LogdConfig c = new LogdConfig(b); - assertEquals(HostName.getLocalhost(), c.logserver().host()); + assertEquals(Hostnames.getLocalhost(), c.logserver().host()); SlobroksConfig.Builder sb = new SlobroksConfig.Builder(); sb = (com.yahoo.cloud.config.SlobroksConfig.Builder) model.getConfig(sb, ""); @@ -98,7 +98,7 @@ public class VespaModelTestCase { zb = (ZookeepersConfig.Builder) model.getConfig(zb, ""); ZookeepersConfig zc = new ZookeepersConfig(zb); assertEquals(zc.zookeeperserverlist().split(",").length, 2); - assertTrue(zc.zookeeperserverlist().startsWith(HostName.getLocalhost())); + assertTrue(zc.zookeeperserverlist().startsWith(Hostnames.getLocalhost())); ApplicationIdConfig.Builder appIdBuilder = new ApplicationIdConfig.Builder(); appIdBuilder = (ApplicationIdConfig.Builder) model.getConfig(appIdBuilder, ""); @@ -112,11 +112,11 @@ public class VespaModelTestCase { public void testHostsConfig() { VespaModel model = getVespaModel(TESTDIR + "app_qrserverandgw"); LogdConfig config = getLogdConfig(model, ""); - assertEquals(config.logserver().host(), HostName.getLocalhost()); + assertEquals(config.logserver().host(), Hostnames.getLocalhost()); assertNotNull(config); config = getLogdConfig(model, "hosts"); assertNotNull(config); - assertEquals(config.logserver().host(), HostName.getLocalhost()); + assertEquals(config.logserver().host(), Hostnames.getLocalhost()); } private static LogdConfig getLogdConfig(VespaModel model, String configId) { @@ -143,7 +143,7 @@ public class VespaModelTestCase { LogdConfig config = getLogdConfig(model, ""); assertNotNull(config); assertEquals(config.logserver().host(), "foo"); - config = getLogdConfig(model, "hosts/" + HostName.getLocalhost() + "/logd"); + config = getLogdConfig(model, "hosts/" + Hostnames.getLocalhost() + "/logd"); assertNotNull(config); assertEquals(config.logserver().host(), "foo"); } diff --git a/config-proxy/src/main/java/com/yahoo/vespa/config/proxy/filedistribution/FileDistributionRpcServer.java b/config-proxy/src/main/java/com/yahoo/vespa/config/proxy/filedistribution/FileDistributionRpcServer.java index d748ebb46e5..4e7390163a4 100644 --- a/config-proxy/src/main/java/com/yahoo/vespa/config/proxy/filedistribution/FileDistributionRpcServer.java +++ b/config-proxy/src/main/java/com/yahoo/vespa/config/proxy/filedistribution/FileDistributionRpcServer.java @@ -9,7 +9,7 @@ import com.yahoo.jrt.Request; import com.yahoo.jrt.StringArray; import com.yahoo.jrt.StringValue; import com.yahoo.jrt.Supervisor; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import com.yahoo.vespa.filedistribution.FileDownloader; import com.yahoo.vespa.filedistribution.FileReferenceDownload; @@ -103,7 +103,7 @@ class FileDistributionRpcServer { private void downloadFile(Request req) { FileReference fileReference = new FileReference(req.parameters().get(0).asString()); log.log(Level.FINE, () -> "getFile() called for file reference '" + fileReference.value() + "'"); - Optional<File> file = downloader.getFile(new FileReferenceDownload(fileReference, HostName.getLocalhost())); + Optional<File> file = downloader.getFile(new FileReferenceDownload(fileReference, Hostnames.getLocalhost())); if (file.isPresent()) { new RequestTracker().trackRequest(file.get().getParentFile()); req.returnValues().add(new StringValue(file.get().getAbsolutePath())); diff --git a/config/src/main/java/com/yahoo/vespa/config/util/ConfigUtils.java b/config/src/main/java/com/yahoo/vespa/config/util/ConfigUtils.java index ea92738cc26..eca87bd29b0 100644 --- a/config/src/main/java/com/yahoo/vespa/config/util/ConfigUtils.java +++ b/config/src/main/java/com/yahoo/vespa/config/util/ConfigUtils.java @@ -4,7 +4,7 @@ package com.yahoo.vespa.config.util; import com.yahoo.collections.Tuple2; import com.yahoo.io.HexDump; import com.yahoo.io.IOUtils; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import com.yahoo.slime.JsonFormat; import com.yahoo.text.AbstractUtf8Array; import com.yahoo.text.Utf8; @@ -351,7 +351,7 @@ public class ConfigUtils { } public static String getCanonicalHostName() { - return HostName.getLocalhost(); + return Hostnames.getLocalhost(); } /** diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/ApplicationRepository.java b/configserver/src/main/java/com/yahoo/vespa/config/server/ApplicationRepository.java index 0828c53a499..7a754dd84cd 100644 --- a/configserver/src/main/java/com/yahoo/vespa/config/server/ApplicationRepository.java +++ b/configserver/src/main/java/com/yahoo/vespa/config/server/ApplicationRepository.java @@ -1,7 +1,7 @@ // Copyright Yahoo. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. package com.yahoo.vespa.config.server; -import ai.vespa.validation.DomainName; +import com.yahoo.net.DomainName; import com.google.inject.Inject; import com.yahoo.cloud.config.ConfigserverConfig; import com.yahoo.component.Version; diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/filedistribution/FileDBRegistry.java b/configserver/src/main/java/com/yahoo/vespa/config/server/filedistribution/FileDBRegistry.java index 3eeaa6598d3..7040cd5e8a3 100644 --- a/configserver/src/main/java/com/yahoo/vespa/config/server/filedistribution/FileDBRegistry.java +++ b/configserver/src/main/java/com/yahoo/vespa/config/server/filedistribution/FileDBRegistry.java @@ -4,7 +4,7 @@ package com.yahoo.vespa.config.server.filedistribution; import com.google.common.collect.ImmutableMap; import com.yahoo.config.FileReference; import com.yahoo.config.application.api.FileRegistry; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import com.yahoo.text.Utf8; import net.jpountz.xxhash.XXHashFactory; @@ -130,7 +130,7 @@ public class FileDBRegistry implements FileRegistry { List<Entry> entries = registry.export(); StringBuilder builder = new StringBuilder(); - builder.append(HostName.getLocalhost()).append('\n'); + builder.append(Hostnames.getLocalhost()).append('\n'); for (FileRegistry.Entry entry : entries) { builder.append(entry.relativePath).append(entryDelimiter).append(entry.reference.value()).append('\n'); } diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/filedistribution/FileDistributionUtil.java b/configserver/src/main/java/com/yahoo/vespa/config/server/filedistribution/FileDistributionUtil.java index a1ddad7bfd4..cbaa87c36ac 100644 --- a/configserver/src/main/java/com/yahoo/vespa/config/server/filedistribution/FileDistributionUtil.java +++ b/configserver/src/main/java/com/yahoo/vespa/config/server/filedistribution/FileDistributionUtil.java @@ -3,7 +3,7 @@ package com.yahoo.vespa.config.server.filedistribution; import com.yahoo.cloud.config.ConfigserverConfig; import com.yahoo.config.FileReference; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import com.yahoo.vespa.config.server.ConfigServerSpec; import java.io.File; @@ -35,7 +35,7 @@ public class FileDistributionUtil { public static List<String> getOtherConfigServersInCluster(ConfigserverConfig configserverConfig) { return ConfigServerSpec.fromConfig(configserverConfig) .stream() - .filter(spec -> !spec.getHostName().equals(HostName.getLocalhost())) + .filter(spec -> !spec.getHostName().equals(Hostnames.getLocalhost())) .map(spec -> "tcp/" + spec.getHostName() + ":" + spec.getConfigServerPort()) .collect(Collectors.toList()); } diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/http/v2/ApplicationHandler.java b/configserver/src/main/java/com/yahoo/vespa/config/server/http/v2/ApplicationHandler.java index 79435dc61ae..0707261bb45 100644 --- a/configserver/src/main/java/com/yahoo/vespa/config/server/http/v2/ApplicationHandler.java +++ b/configserver/src/main/java/com/yahoo/vespa/config/server/http/v2/ApplicationHandler.java @@ -1,7 +1,7 @@ // Copyright Yahoo. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. package com.yahoo.vespa.config.server.http.v2; -import ai.vespa.validation.DomainName; +import com.yahoo.net.DomainName; import com.google.inject.Inject; import com.yahoo.component.Version; import com.yahoo.config.application.api.ApplicationFile; diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/rpc/GetConfigProcessor.java b/configserver/src/main/java/com/yahoo/vespa/config/server/rpc/GetConfigProcessor.java index 55645d83b3c..f63035cdd6e 100644 --- a/configserver/src/main/java/com/yahoo/vespa/config/server/rpc/GetConfigProcessor.java +++ b/configserver/src/main/java/com/yahoo/vespa/config/server/rpc/GetConfigProcessor.java @@ -7,7 +7,7 @@ import com.yahoo.component.Version; import com.yahoo.config.provision.TenantName; import com.yahoo.vespa.config.PayloadChecksums; import com.yahoo.jrt.Request; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import com.yahoo.vespa.config.ConfigPayload; import com.yahoo.vespa.config.ErrorCode; import com.yahoo.vespa.config.UnknownConfigIdException; @@ -32,7 +32,7 @@ import static com.yahoo.vespa.config.protocol.SlimeConfigResponse.fromConfigPayl class GetConfigProcessor implements Runnable { private static final Logger log = Logger.getLogger(GetConfigProcessor.class.getName()); - private static final String localHostName = HostName.getLocalhost(); + private static final String localHostName = Hostnames.getLocalhost(); private final JRTServerConfigRequest request; diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/session/SessionPreparer.java b/configserver/src/main/java/com/yahoo/vespa/config/server/session/SessionPreparer.java index 8293871335d..18dbdaccfe7 100644 --- a/configserver/src/main/java/com/yahoo/vespa/config/server/session/SessionPreparer.java +++ b/configserver/src/main/java/com/yahoo/vespa/config/server/session/SessionPreparer.java @@ -24,7 +24,7 @@ import com.yahoo.config.provision.DockerImage; import com.yahoo.config.provision.Zone; import com.yahoo.container.jdisc.secretstore.SecretStore; import com.yahoo.lang.SettableOptional; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import com.yahoo.path.Path; import com.yahoo.vespa.config.server.TimeoutBudget; import com.yahoo.vespa.config.server.application.ApplicationSet; @@ -236,7 +236,7 @@ public class SessionPreparer { applicationId + " (" + fileReference + ")"); properties.configServerSpecs() .stream() - .filter(spec -> !spec.getHostName().equals(HostName.getLocalhost())) + .filter(spec -> !spec.getHostName().equals(Hostnames.getLocalhost())) .forEach(spec -> fileDistribution.startDownload(spec.getHostName(), spec.getConfigServerPort(), Set.of(fileReference))); checkTimeout("startDistributionOfApplicationPackage"); diff --git a/configserver/src/test/java/com/yahoo/vespa/config/server/ApplicationRepositoryTest.java b/configserver/src/test/java/com/yahoo/vespa/config/server/ApplicationRepositoryTest.java index 13571f218ca..e3fc2345c68 100644 --- a/configserver/src/test/java/com/yahoo/vespa/config/server/ApplicationRepositoryTest.java +++ b/configserver/src/test/java/com/yahoo/vespa/config/server/ApplicationRepositoryTest.java @@ -1,7 +1,7 @@ // Copyright Yahoo. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. package com.yahoo.vespa.config.server; -import ai.vespa.validation.DomainName; +import com.yahoo.net.DomainName; import com.yahoo.cloud.config.ConfigserverConfig; import com.yahoo.component.Version; import com.yahoo.config.ConfigInstance; diff --git a/configserver/src/test/java/com/yahoo/vespa/config/server/filedistribution/FileServerTest.java b/configserver/src/test/java/com/yahoo/vespa/config/server/filedistribution/FileServerTest.java index 67c40f94b6a..2b6307f85b0 100644 --- a/configserver/src/test/java/com/yahoo/vespa/config/server/filedistribution/FileServerTest.java +++ b/configserver/src/test/java/com/yahoo/vespa/config/server/filedistribution/FileServerTest.java @@ -6,7 +6,7 @@ import com.yahoo.config.FileReference; import com.yahoo.io.IOUtils; import com.yahoo.jrt.Supervisor; import com.yahoo.jrt.Transport; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import com.yahoo.vespa.filedistribution.FileDownloader; import com.yahoo.vespa.filedistribution.FileReferenceData; import com.yahoo.vespa.filedistribution.FileReferenceDownload; @@ -91,7 +91,7 @@ public class FileServerTest { // Empty connection pool when only one server, no use in downloading from yourself List<ConfigserverConfig.Zookeeperserver.Builder> servers = new ArrayList<>(); ConfigserverConfig.Zookeeperserver.Builder serverBuilder = new ConfigserverConfig.Zookeeperserver.Builder(); - serverBuilder.hostname(HostName.getLocalhost()); + serverBuilder.hostname(Hostnames.getLocalhost()); serverBuilder.port(123456); servers.add(serverBuilder); builder.zookeeperserver(servers); diff --git a/configserver/src/test/java/com/yahoo/vespa/config/server/rpc/RpcTester.java b/configserver/src/test/java/com/yahoo/vespa/config/server/rpc/RpcTester.java index 40ed20b7969..198c646f6a7 100644 --- a/configserver/src/test/java/com/yahoo/vespa/config/server/rpc/RpcTester.java +++ b/configserver/src/test/java/com/yahoo/vespa/config/server/rpc/RpcTester.java @@ -10,7 +10,7 @@ import com.yahoo.jrt.Request; import com.yahoo.jrt.Spec; import com.yahoo.jrt.Supervisor; import com.yahoo.jrt.Transport; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import com.yahoo.test.ManualClock; import com.yahoo.vespa.config.server.ApplicationRepository; import com.yahoo.vespa.config.server.MemoryGenerationCounter; @@ -50,7 +50,7 @@ import static org.junit.Assert.assertTrue; public class RpcTester implements AutoCloseable { private final ManualClock clock = new ManualClock(Instant.ofEpochMilli(100)); - private final String myHostname = HostName.getLocalhost(); + private final String myHostname = Hostnames.getLocalhost(); private final HostLivenessTracker hostLivenessTracker = new ConfigRequestHostLivenessTracker(clock); private final Spec spec; diff --git a/container-disc/src/main/java/com/yahoo/container/jdisc/ConfiguredApplication.java b/container-disc/src/main/java/com/yahoo/container/jdisc/ConfiguredApplication.java index 2c25f38437a..98a3e72ab90 100644 --- a/container-disc/src/main/java/com/yahoo/container/jdisc/ConfiguredApplication.java +++ b/container-disc/src/main/java/com/yahoo/container/jdisc/ConfiguredApplication.java @@ -44,7 +44,7 @@ import com.yahoo.jrt.slobrok.api.Register; import com.yahoo.jrt.slobrok.api.SlobrokList; import com.yahoo.log.LogSetup; import com.yahoo.messagebus.network.rpc.SlobrokConfigSubscriber; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import com.yahoo.vespa.config.ConfigKey; import com.yahoo.yolean.Exceptions; import com.yahoo.yolean.UncheckedInterruptedException; @@ -195,7 +195,7 @@ public final class ConfiguredApplication implements Application { private Register registerInSlobrok(QrConfig qrConfig) { SlobrokList slobrokList = getSlobrokList(); - Spec mySpec = new Spec(HostName.getLocalhost(), acceptor.port()); + Spec mySpec = new Spec(Hostnames.getLocalhost(), acceptor.port()); Register slobrokRegistrator = new Register(supervisor, slobrokList, mySpec); slobrokRegistrator.registerName(qrConfig.rpc().slobrokId()); log.log(Level.INFO, "Registered name '" + qrConfig.rpc().slobrokId() + diff --git a/container-search/src/main/java/com/yahoo/search/dispatch/searchcluster/SearchCluster.java b/container-search/src/main/java/com/yahoo/search/dispatch/searchcluster/SearchCluster.java index f8c4627473d..f64a1be3251 100644 --- a/container-search/src/main/java/com/yahoo/search/dispatch/searchcluster/SearchCluster.java +++ b/container-search/src/main/java/com/yahoo/search/dispatch/searchcluster/SearchCluster.java @@ -4,7 +4,7 @@ package com.yahoo.search.dispatch.searchcluster; import com.google.common.collect.ImmutableMap; import com.google.common.math.Quantiles; import com.yahoo.container.handler.VipStatus; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import com.yahoo.prelude.Pong; import com.yahoo.search.cluster.ClusterMonitor; import com.yahoo.search.cluster.NodeManager; @@ -70,7 +70,7 @@ public class SearchCluster implements NodeManager<Node> { this.orderedGroups = List.copyOf(groupIntroductionOrder.values()); hitEstimator = new TopKEstimator(30.0, dispatchConfig.topKProbability(), SKEW_FACTOR); - this.localCorpusDispatchTarget = findLocalCorpusDispatchTarget(HostName.getLocalhost(), nodes, groups); + this.localCorpusDispatchTarget = findLocalCorpusDispatchTarget(Hostnames.getLocalhost(), nodes, groups); } @Override diff --git a/container-search/src/main/java/com/yahoo/search/handler/SearchHandler.java b/container-search/src/main/java/com/yahoo/search/handler/SearchHandler.java index af6374ba245..fdbcfdab56a 100644 --- a/container-search/src/main/java/com/yahoo/search/handler/SearchHandler.java +++ b/container-search/src/main/java/com/yahoo/search/handler/SearchHandler.java @@ -26,7 +26,7 @@ import com.yahoo.jdisc.Request; import com.yahoo.language.Linguistics; import com.yahoo.language.process.Embedder; import com.yahoo.language.provider.DefaultEmbedderProvider; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import com.yahoo.net.UriTools; import com.yahoo.prelude.query.parser.ParseException; import com.yahoo.processing.IllegalInputException; @@ -105,7 +105,7 @@ public class SearchHandler extends LoggingRequestHandler { /** If present, responses from this will set the HTTP response header with this key to the host name of this */ private final Optional<String> hostResponseHeaderKey; - private final String selfHostname = HostName.getLocalhost(); + private final String selfHostname = Hostnames.getLocalhost(); private final Map<String, Embedder> embedders; private final ExecutionFactory executionFactory; private final AtomicLong numRequestsLeftToTrace; diff --git a/container-search/src/test/java/com/yahoo/search/dispatch/searchcluster/SearchClusterTest.java b/container-search/src/test/java/com/yahoo/search/dispatch/searchcluster/SearchClusterTest.java index 5da7af9da1c..4dc85a4f3db 100644 --- a/container-search/src/test/java/com/yahoo/search/dispatch/searchcluster/SearchClusterTest.java +++ b/container-search/src/test/java/com/yahoo/search/dispatch/searchcluster/SearchClusterTest.java @@ -4,7 +4,7 @@ package com.yahoo.search.dispatch.searchcluster; import com.yahoo.container.QrSearchersConfig; import com.yahoo.container.handler.ClustersStatus; import com.yahoo.container.handler.VipStatus; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import com.yahoo.prelude.Pong; import com.yahoo.search.cluster.ClusterMonitor; import com.yahoo.search.dispatch.MockSearchCluster; @@ -181,7 +181,7 @@ public class SearchClusterTest { @Test public void requireThatVipStatusIsDefaultDownWithLocalDispatch() { - try (State test = new State("cluster.1", 1, HostName.getLocalhost(), "b")) { + try (State test = new State("cluster.1", 1, Hostnames.getLocalhost(), "b")) { assertTrue(test.searchCluster.localCorpusDispatchTarget().isPresent()); assertFalse(test.vipStatus.isInRotation()); @@ -192,7 +192,7 @@ public class SearchClusterTest { @Test public void requireThatVipStatusStaysUpWithLocalDispatchAndClusterSize1() { - try (State test = new State("cluster.1", 1, HostName.getLocalhost())) { + try (State test = new State("cluster.1", 1, Hostnames.getLocalhost())) { assertTrue(test.searchCluster.localCorpusDispatchTarget().isPresent()); assertFalse(test.vipStatus.isInRotation()); @@ -206,7 +206,7 @@ public class SearchClusterTest { @Test public void requireThatVipStatusIsDefaultDownWithLocalDispatchAndClusterSize2() { - try (State test = new State("cluster.1", 1, HostName.getLocalhost(), "otherhost")) { + try (State test = new State("cluster.1", 1, Hostnames.getLocalhost(), "otherhost")) { assertTrue(test.searchCluster.localCorpusDispatchTarget().isPresent()); assertFalse(test.vipStatus.isInRotation()); @@ -220,7 +220,7 @@ public class SearchClusterTest { @Test public void requireThatVipStatusDownWhenLocalIsDown() { - try (State test = new State("cluster.1",1,HostName.getLocalhost(), "b")) { + try (State test = new State("cluster.1", 1, Hostnames.getLocalhost(), "b")) { test.waitOneFullPingRound(); assertTrue(test.vipStatus.isInRotation()); diff --git a/container-search/src/test/java/com/yahoo/search/handler/SearchHandlerTest.java b/container-search/src/test/java/com/yahoo/search/handler/SearchHandlerTest.java index b1354106f19..c71c6abd511 100644 --- a/container-search/src/test/java/com/yahoo/search/handler/SearchHandlerTest.java +++ b/container-search/src/test/java/com/yahoo/search/handler/SearchHandlerTest.java @@ -11,7 +11,7 @@ import com.yahoo.io.IOUtils; import com.yahoo.jdisc.Request; import com.yahoo.jdisc.handler.RequestHandler; import com.yahoo.jdisc.test.MockMetric; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import com.yahoo.search.Query; import com.yahoo.search.Result; import com.yahoo.search.Searcher; @@ -52,7 +52,7 @@ public class SearchHandlerTest { private static final String testDir = "src/test/java/com/yahoo/search/handler/test/config"; private static final String myHostnameHeader = "my-hostname-header"; - private static final String selfHostname = HostName.getLocalhost(); + private static final String selfHostname = Hostnames.getLocalhost(); private static String tempDir = ""; diff --git a/container-search/src/test/java/com/yahoo/search/handler/test/JSONSearchHandlerTestCase.java b/container-search/src/test/java/com/yahoo/search/handler/test/JSONSearchHandlerTestCase.java index 7811336823d..02102251e59 100644 --- a/container-search/src/test/java/com/yahoo/search/handler/test/JSONSearchHandlerTestCase.java +++ b/container-search/src/test/java/com/yahoo/search/handler/test/JSONSearchHandlerTestCase.java @@ -11,7 +11,7 @@ import com.yahoo.container.jdisc.HttpRequest; import com.yahoo.container.jdisc.RequestHandlerTestDriver; import com.yahoo.container.protect.Error; import com.yahoo.io.IOUtils; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import com.yahoo.search.handler.SearchHandler; import com.yahoo.search.searchchain.config.test.SearchChainConfigurerTestCase; import com.yahoo.slime.Inspector; @@ -48,7 +48,7 @@ public class JSONSearchHandlerTestCase { private static final String testDir = "src/test/java/com/yahoo/search/handler/test/config"; private static final String myHostnameHeader = "my-hostname-header"; - private static final String selfHostname = HostName.getLocalhost(); + private static final String selfHostname = Hostnames.getLocalhost(); private static String tempDir = ""; private static String configId = null; diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/Controller.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/Controller.java index 16e1a2db2e4..8d3b0505603 100644 --- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/Controller.java +++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/Controller.java @@ -12,6 +12,7 @@ import com.yahoo.config.provision.SystemName; import com.yahoo.config.provision.zone.ZoneApi; import com.yahoo.container.jdisc.secretstore.SecretStore; import com.yahoo.jdisc.Metric; +import com.yahoo.net.Hostnames; import com.yahoo.vespa.curator.Lock; import com.yahoo.vespa.flags.FlagSource; import com.yahoo.vespa.hosted.controller.api.integration.ServiceRegistry; @@ -98,7 +99,7 @@ public class Controller extends AbstractComponent { public Controller(CuratorDb curator, RotationsConfig rotationsConfig, AccessControl accessControl, FlagSource flagSource, MavenRepository mavenRepository, ServiceRegistry serviceRegistry, Metric metric, SecretStore secretStore, ControllerConfig controllerConfig) { - this(curator, rotationsConfig, accessControl, com.yahoo.net.HostName::getLocalhost, flagSource, + this(curator, rotationsConfig, accessControl, Hostnames::getLocalhost, flagSource, mavenRepository, serviceRegistry, metric, secretStore, controllerConfig, Sleeper.DEFAULT); } diff --git a/jdisc_core/src/main/java/com/yahoo/jdisc/core/ConsoleLogListener.java b/jdisc_core/src/main/java/com/yahoo/jdisc/core/ConsoleLogListener.java index 2cfa604109b..b506da39731 100644 --- a/jdisc_core/src/main/java/com/yahoo/jdisc/core/ConsoleLogListener.java +++ b/jdisc_core/src/main/java/com/yahoo/jdisc/core/ConsoleLogListener.java @@ -1,7 +1,7 @@ // Copyright Yahoo. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. package com.yahoo.jdisc.core; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import org.osgi.service.log.LogEntry; import org.osgi.service.log.LogListener; @@ -72,6 +72,6 @@ class ConsoleLogListener implements LogListener { } static String getHostname() { - return HostName.getLocalhost(); + return Hostnames.getLocalhost(); } } diff --git a/messagebus/src/main/java/com/yahoo/messagebus/network/Identity.java b/messagebus/src/main/java/com/yahoo/messagebus/network/Identity.java index 1362052996a..9beb2deb909 100644 --- a/messagebus/src/main/java/com/yahoo/messagebus/network/Identity.java +++ b/messagebus/src/main/java/com/yahoo/messagebus/network/Identity.java @@ -1,7 +1,7 @@ // Copyright Yahoo. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. package com.yahoo.messagebus.network; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; /** * This class encapsulates the identity of the application that uses this instance of message bus. This identity @@ -23,7 +23,7 @@ public class Identity { * @param configId The config identifier for the application. */ public Identity(String configId) { - hostname = HostName.getLocalhost(); // ... but fallback to hostname if we get an IPv6 address + hostname = Hostnames.getLocalhost(); // ... but fallback to hostname if we get an IPv6 address servicePrefix = configId; } diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/NodeAllocation.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/NodeAllocation.java index 67d89835c0d..32f2c985880 100644 --- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/NodeAllocation.java +++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/provisioning/NodeAllocation.java @@ -8,6 +8,7 @@ import com.yahoo.config.provision.Flavor; import com.yahoo.config.provision.NodeResources; import com.yahoo.config.provision.NodeType; import com.yahoo.config.provision.SystemName; +import com.yahoo.net.Hostnames; import com.yahoo.vespa.hosted.provision.Node; import com.yahoo.vespa.hosted.provision.NodeList; import com.yahoo.vespa.hosted.provision.NodeRepository; @@ -346,7 +347,7 @@ class NodeAllocation { // - cfg1 is starting and redeploys its infrastructure application during bootstrap. A deficit is detected // (itself, because cfg1 is only added to the repository *after* it is up) // - cfg1 tries to provision a new host for itself - Integer myIndex = parseIndex(com.yahoo.net.HostName.getLocalhost()); + Integer myIndex = parseIndex(Hostnames.getLocalhost()); indices.remove(myIndex); return indices; } diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/restapi/LocksResponse.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/restapi/LocksResponse.java index 8a66a9394b2..166b65e08ca 100644 --- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/restapi/LocksResponse.java +++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/restapi/LocksResponse.java @@ -2,7 +2,7 @@ package com.yahoo.vespa.hosted.provision.restapi; import com.yahoo.container.jdisc.HttpResponse; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import com.yahoo.slime.Cursor; import com.yahoo.slime.JsonFormat; import com.yahoo.slime.Slime; @@ -33,7 +33,7 @@ public class LocksResponse extends HttpResponse { private final Slime slime = new Slime(); public LocksResponse() { - this(HostName.getLocalhost(), + this(Hostnames.getLocalhost(), new TreeMap<>(LockStats.getGlobal().getLockMetricsByPath()), LockStats.getGlobal().getThreadLockStats(), LockStats.getGlobal().getLockAttemptSamples()); diff --git a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/DynamicProvisioningMaintainerTest.java b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/DynamicProvisioningMaintainerTest.java index 68d75db8a4c..326d4999bbe 100644 --- a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/DynamicProvisioningMaintainerTest.java +++ b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/maintenance/DynamicProvisioningMaintainerTest.java @@ -15,7 +15,7 @@ import com.yahoo.config.provision.NodeType; import com.yahoo.config.provision.RegionName; import com.yahoo.config.provision.SystemName; import com.yahoo.config.provision.Zone; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import com.yahoo.vespa.flags.InMemoryFlagSource; import com.yahoo.vespa.flags.PermanentFlags; import com.yahoo.vespa.flags.custom.ClusterCapacity; @@ -503,12 +503,12 @@ public class DynamicProvisioningMaintainerTest { assertEquals(2, allNodes.nodeType(hostType.childNodeType()).size()); // Deployment by the removed host has no effect - HostName.setHostNameForTestingOnly("cfg2.example.com"); + Hostnames.setHostNameForTestingOnly("cfg2.example.com"); tester.prepareAndActivateInfraApplication(configSrvApp, hostType.childNodeType()); assertEquals(List.of(), dynamicProvisioningTester.hostProvisioner.provisionedHosts()); // Deployment on another config server starts provisioning a new host and child - HostName.setHostNameForTestingOnly("cfg3.example.com"); + Hostnames.setHostNameForTestingOnly("cfg3.example.com"); assertEquals(0, tester.nodeRepository().nodes().list(Node.State.reserved).nodeType(hostType.childNodeType()).size()); assertEquals(2, tester.prepareAndActivateInfraApplication(configSrvApp, hostType.childNodeType()).size()); assertEquals(1, tester.nodeRepository().nodes().list(Node.State.reserved).nodeType(hostType.childNodeType()).size()); diff --git a/vespaclient-container-plugin/src/main/java/com/yahoo/vespa/http/server/ClientFeederV3.java b/vespaclient-container-plugin/src/main/java/com/yahoo/vespa/http/server/ClientFeederV3.java index 66955be2325..ad91dc8c7b1 100644 --- a/vespaclient-container-plugin/src/main/java/com/yahoo/vespa/http/server/ClientFeederV3.java +++ b/vespaclient-container-plugin/src/main/java/com/yahoo/vespa/http/server/ClientFeederV3.java @@ -13,7 +13,7 @@ import com.yahoo.messagebus.Message; import com.yahoo.messagebus.ReplyHandler; import com.yahoo.messagebus.Result; import com.yahoo.messagebus.shared.SharedSourceSession; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import com.yahoo.vespa.http.client.core.ErrorCode; import com.yahoo.vespa.http.client.core.Headers; import com.yahoo.vespa.http.client.core.OperationStatus; @@ -69,7 +69,7 @@ class ClientFeederV3 { this.feedReplyHandler = feedReplyHandler; this.metric = metric; this.streamReaderV3 = new StreamReaderV3(feedReaderFactory, docTypeManager); - this.hostName = HostName.getLocalhost(); + this.hostName = Hostnames.getLocalhost(); } boolean timedOut() { diff --git a/vespaclient-java/src/main/java/com/yahoo/vespavisit/VdsVisitTarget.java b/vespaclient-java/src/main/java/com/yahoo/vespavisit/VdsVisitTarget.java index d1fbde7dd42..0138900046c 100644 --- a/vespaclient-java/src/main/java/com/yahoo/vespavisit/VdsVisitTarget.java +++ b/vespaclient-java/src/main/java/com/yahoo/vespavisit/VdsVisitTarget.java @@ -12,7 +12,7 @@ import com.yahoo.documentapi.messagebus.loadtypes.LoadTypeSet; import java.util.logging.Level; import com.yahoo.log.LogSetup; import com.yahoo.messagebus.network.Identity; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import org.apache.commons.cli.CommandLine; import org.apache.commons.cli.CommandLineParser; import org.apache.commons.cli.DefaultParser; @@ -201,7 +201,7 @@ public class VdsVisitTarget { } private String getTcpAddress() { - String hostname = HostName.getLocalhost(); + String hostname = Hostnames.getLocalhost(); return "tcp/" + hostname + ":" + port + "/visit-destination"; } diff --git a/vespajlib/src/main/java/com/yahoo/concurrent/maintenance/Maintainer.java b/vespajlib/src/main/java/com/yahoo/concurrent/maintenance/Maintainer.java index 1edf8e4edbe..5421bd770bf 100644 --- a/vespajlib/src/main/java/com/yahoo/concurrent/maintenance/Maintainer.java +++ b/vespajlib/src/main/java/com/yahoo/concurrent/maintenance/Maintainer.java @@ -2,7 +2,7 @@ package com.yahoo.concurrent.maintenance; import com.yahoo.concurrent.UncheckedTimeoutException; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import java.math.BigDecimal; import java.math.RoundingMode; @@ -45,7 +45,7 @@ public abstract class Maintainer implements Runnable { this.ignoreCollision = ignoreCollision; Objects.requireNonNull(startedAt); Objects.requireNonNull(clusterHostnames); - Duration initialDelay = staggeredDelay(interval, startedAt, HostName.getLocalhost(), clusterHostnames) + Duration initialDelay = staggeredDelay(interval, startedAt, Hostnames.getLocalhost(), clusterHostnames) .plus(Duration.ofSeconds(30)); // Let the system stabilize before maintenance service = new ScheduledThreadPoolExecutor(1, r -> new Thread(r, name() + "-worker")); service.scheduleAtFixedRate(this, initialDelay.toMillis(), interval.toMillis(), TimeUnit.MILLISECONDS); diff --git a/vespajlib/src/main/java/ai/vespa/validation/DomainName.java b/vespajlib/src/main/java/com/yahoo/net/DomainName.java index 52d9acc822c..5ac13d0306c 100644 --- a/vespajlib/src/main/java/ai/vespa/validation/DomainName.java +++ b/vespajlib/src/main/java/com/yahoo/net/DomainName.java @@ -1,5 +1,7 @@ // Copyright Yahoo. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. -package ai.vespa.validation; +package com.yahoo.net; + +import ai.vespa.validation.StringWrapper; import java.util.regex.Pattern; diff --git a/vespajlib/src/main/java/ai/vespa/validation/Hostname.java b/vespajlib/src/main/java/com/yahoo/net/Hostname.java index e53dd501c45..37dc41cbe14 100644 --- a/vespajlib/src/main/java/ai/vespa/validation/Hostname.java +++ b/vespajlib/src/main/java/com/yahoo/net/Hostname.java @@ -1,5 +1,7 @@ // Copyright Yahoo. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. -package ai.vespa.validation; +package com.yahoo.net; + +import ai.vespa.validation.StringWrapper; import static ai.vespa.validation.Validation.requireInRange; import static ai.vespa.validation.Validation.requireMatch; diff --git a/vespajlib/src/main/java/com/yahoo/net/HostName.java b/vespajlib/src/main/java/com/yahoo/net/Hostnames.java index 5b820cd996e..c6e78d778e3 100644 --- a/vespajlib/src/main/java/com/yahoo/net/HostName.java +++ b/vespajlib/src/main/java/com/yahoo/net/Hostnames.java @@ -11,7 +11,7 @@ import java.util.Optional; * * @author arnej */ -public class HostName { +public class Hostnames { private static String preferredHostName = null; diff --git a/vespajlib/src/test/java/ai/vespa/validation/DomainNameTest.java b/vespajlib/src/test/java/com/yahoo/net/DomainNameTest.java index dad2086d6ea..d8e76b71d7e 100644 --- a/vespajlib/src/test/java/ai/vespa/validation/DomainNameTest.java +++ b/vespajlib/src/test/java/com/yahoo/net/DomainNameTest.java @@ -1,6 +1,7 @@ // Copyright Yahoo. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. -package ai.vespa.validation; +package com.yahoo.net; +import com.yahoo.net.DomainName; import org.junit.jupiter.api.Test; import static org.junit.jupiter.api.Assertions.assertThrows; diff --git a/vespajlib/src/test/java/ai/vespa/validation/HostnameTest.java b/vespajlib/src/test/java/com/yahoo/net/HostnameTest.java index 7969c2a5f78..ef502aea2cd 100644 --- a/vespajlib/src/test/java/ai/vespa/validation/HostnameTest.java +++ b/vespajlib/src/test/java/com/yahoo/net/HostnameTest.java @@ -1,6 +1,7 @@ // Copyright Yahoo. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. -package ai.vespa.validation; +package com.yahoo.net; +import com.yahoo.net.Hostname; import org.junit.jupiter.api.Test; import static org.junit.jupiter.api.Assertions.assertThrows; diff --git a/vespajlib/src/test/java/com/yahoo/net/HostNameTestCase.java b/vespajlib/src/test/java/com/yahoo/net/HostnamesTestCase.java index ee26b357db8..2e942605e3e 100644 --- a/vespajlib/src/test/java/com/yahoo/net/HostNameTestCase.java +++ b/vespajlib/src/test/java/com/yahoo/net/HostnamesTestCase.java @@ -8,11 +8,11 @@ import static org.junit.Assert.assertFalse; /** * @author Ulf Lilleengen */ -public class HostNameTestCase { +public class HostnamesTestCase { @Test public void testHostnameIsFound() { - assertFalse(HostName.getLocalhost().isEmpty()); + assertFalse(Hostnames.getLocalhost().isEmpty()); } } diff --git a/zkfacade/src/main/java/com/yahoo/vespa/curator/ConnectionSpec.java b/zkfacade/src/main/java/com/yahoo/vespa/curator/ConnectionSpec.java index 022932bef89..73a46981c9b 100644 --- a/zkfacade/src/main/java/com/yahoo/vespa/curator/ConnectionSpec.java +++ b/zkfacade/src/main/java/com/yahoo/vespa/curator/ConnectionSpec.java @@ -1,7 +1,7 @@ // Copyright Yahoo. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. package com.yahoo.vespa.curator; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import java.util.List; import java.util.Objects; @@ -75,7 +75,7 @@ class ConnectionSpec { Function<T, String> hostnameGetter, Function<T, Integer> portGetter, boolean localhostAffinity) { - String thisServer = HostName.getLocalhost(); + String thisServer = Hostnames.getLocalhost(); StringBuilder connectionSpec = new StringBuilder(); for (var server : servers) { if (localhostAffinity && !thisServer.equals(hostnameGetter.apply(server))) continue; diff --git a/zkfacade/src/test/java/com/yahoo/vespa/curator/ConnectionSpecTest.java b/zkfacade/src/test/java/com/yahoo/vespa/curator/ConnectionSpecTest.java index 6e833088f0d..b4bf8bc1e73 100644 --- a/zkfacade/src/test/java/com/yahoo/vespa/curator/ConnectionSpecTest.java +++ b/zkfacade/src/test/java/com/yahoo/vespa/curator/ConnectionSpecTest.java @@ -1,7 +1,7 @@ // Copyright Yahoo. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. package com.yahoo.vespa.curator; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import org.junit.Test; import java.util.List; @@ -15,7 +15,7 @@ public class ConnectionSpecTest { @Test public void create() { - HostName.setHostNameForTestingOnly("host2"); + Hostnames.setHostNameForTestingOnly("host2"); Config config = new Config(List.of(new Config.Server("host1", 10001), new Config.Server("host2", 10002), new Config.Server("host3", 10003))); diff --git a/zkfacade/src/test/java/com/yahoo/vespa/curator/CuratorTest.java b/zkfacade/src/test/java/com/yahoo/vespa/curator/CuratorTest.java index 71eae121313..dbefda6282e 100644 --- a/zkfacade/src/test/java/com/yahoo/vespa/curator/CuratorTest.java +++ b/zkfacade/src/test/java/com/yahoo/vespa/curator/CuratorTest.java @@ -2,7 +2,7 @@ package com.yahoo.vespa.curator; import com.yahoo.cloud.config.CuratorConfig; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import com.yahoo.path.Path; import com.yahoo.text.Utf8; import org.junit.Test; @@ -19,7 +19,7 @@ import static org.junit.Assert.fail; */ public class CuratorTest { - private static final String localhost = HostName.getLocalhost(); + private static final String localhost = Hostnames.getLocalhost(); private static final int port1 = 1; private static final int port2 = 2; diff --git a/zookeeper-server/zookeeper-server-3.7.0/src/test/java/com/yahoo/vespa/zookeper/VespaZooKeeperTest.java b/zookeeper-server/zookeeper-server-3.7.0/src/test/java/com/yahoo/vespa/zookeper/VespaZooKeeperTest.java index db643d76e0d..123059a8a1f 100644 --- a/zookeeper-server/zookeeper-server-3.7.0/src/test/java/com/yahoo/vespa/zookeper/VespaZooKeeperTest.java +++ b/zookeeper-server/zookeeper-server-3.7.0/src/test/java/com/yahoo/vespa/zookeper/VespaZooKeeperTest.java @@ -2,7 +2,7 @@ package com.yahoo.vespa.zookeper; import com.yahoo.cloud.config.ZookeeperServerConfig; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import com.yahoo.vespa.zookeeper.ReconfigurableVespaZooKeeperServer; import com.yahoo.vespa.zookeeper.Reconfigurer; import com.yahoo.vespa.zookeeper.VespaZooKeeperAdminImpl; @@ -147,7 +147,7 @@ public class VespaZooKeeperTest { } static ZooKeeperAdmin createAdmin(ZookeeperServerConfig config) throws IOException { - return new ZooKeeperAdmin(HostName.getLocalhost() + ":" + config.clientPort(), + return new ZooKeeperAdmin(Hostnames.getLocalhost() + ":" + config.clientPort(), 10_000, System.err::println, new ZkClientConfigBuilder().toConfig()); diff --git a/zookeeper-server/zookeeper-server-common/src/main/java/com/yahoo/vespa/zookeeper/Reconfigurer.java b/zookeeper-server/zookeeper-server-common/src/main/java/com/yahoo/vespa/zookeeper/Reconfigurer.java index 389df340ca7..229290c2f8b 100644 --- a/zookeeper-server/zookeeper-server-common/src/main/java/com/yahoo/vespa/zookeeper/Reconfigurer.java +++ b/zookeeper-server/zookeeper-server-common/src/main/java/com/yahoo/vespa/zookeeper/Reconfigurer.java @@ -4,7 +4,7 @@ package com.yahoo.vespa.zookeeper; import com.google.inject.Inject; import com.yahoo.cloud.config.ZookeeperServerConfig; import com.yahoo.component.AbstractComponent; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import com.yahoo.protect.Process; import com.yahoo.yolean.Exceptions; @@ -135,7 +135,7 @@ public class Reconfigurer extends AbstractComponent { } private static String localConnectionSpec(ZookeeperServerConfig config) { - return HostName.getLocalhost() + ":" + config.clientPort(); + return Hostnames.getLocalhost() + ":" + config.clientPort(); } private static List<String> servers(ZookeeperServerConfig config) { diff --git a/zookeeper-server/zookeeper-server-common/src/test/java/com/yahoo/vespa/zookeeper/ReconfigurerTest.java b/zookeeper-server/zookeeper-server-common/src/test/java/com/yahoo/vespa/zookeeper/ReconfigurerTest.java index 760c326cf5d..f31d742884d 100644 --- a/zookeeper-server/zookeeper-server-common/src/test/java/com/yahoo/vespa/zookeeper/ReconfigurerTest.java +++ b/zookeeper-server/zookeeper-server-common/src/test/java/com/yahoo/vespa/zookeeper/ReconfigurerTest.java @@ -2,7 +2,7 @@ package com.yahoo.vespa.zookeeper; import com.yahoo.cloud.config.ZookeeperServerConfig; -import com.yahoo.net.HostName; +import com.yahoo.net.Hostnames; import org.junit.After; import org.junit.Before; import org.junit.Rule; @@ -150,7 +150,7 @@ public class ReconfigurerTest { } }); this.zooKeeperAdmin = zooKeeperAdmin; - HostName.setHostNameForTestingOnly("node1"); + Hostnames.setHostNameForTestingOnly("node1"); } void startOrReconfigure(ZookeeperServerConfig newConfig) { |