diff options
author | Andreas Eriksen <andreer@pvv.ntnu.no> | 2019-08-26 23:28:20 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-08-26 23:28:20 +0200 |
commit | 807a29cc8dcccda799c7f35bfd1debcc686b4db1 (patch) | |
tree | 1e5c1fb220a3b624026a5b1d722f6e9c297e49cc /node-repository | |
parent | b04a95077d8831a3c0d3606df80c5857b72db75f (diff) |
Revert "Use real port 4443 for NLBs"
Diffstat (limited to 'node-repository')
4 files changed, 10 insertions, 9 deletions
diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/lb/Real.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/lb/Real.java index 3c237a9a8a0..196a7ff2d05 100644 --- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/lb/Real.java +++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/lb/Real.java @@ -13,7 +13,8 @@ import java.util.Objects; */ public class Real implements Comparable<Real> { - private static final int defaultPort = 4443; + // TODO: Change to 4443 when moving to HTTPS + private static final int defaultPort = 4080; private final HostName hostname; private final String ipAddress; diff --git a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/LoadBalancerProvisionerTest.java b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/LoadBalancerProvisionerTest.java index 97acdc5bfaf..08112f9b055 100644 --- a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/LoadBalancerProvisionerTest.java +++ b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/LoadBalancerProvisionerTest.java @@ -67,9 +67,9 @@ public class LoadBalancerProvisionerTest { assertEquals(containerCluster1, lbApp1.get().get(0).id().cluster()); assertEquals(Collections.singleton(4443), lbApp1.get().get(0).instance().ports()); assertEquals("127.0.0.1", get(lbApp1.get().get(0).instance().reals(), 0).ipAddress()); - assertEquals(4443, get(lbApp1.get().get(0).instance().reals(), 0).port()); + assertEquals(4080, get(lbApp1.get().get(0).instance().reals(), 0).port()); assertEquals("127.0.0.2", get(lbApp1.get().get(0).instance().reals(), 1).ipAddress()); - assertEquals(4443, get(lbApp1.get().get(0).instance().reals(), 1).port()); + assertEquals(4080, get(lbApp1.get().get(0).instance().reals(), 1).port()); // A container is failed Supplier<List<Node>> containers = () -> tester.getNodes(app1).type(ClusterSpec.Type.container).asList(); diff --git a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/v2/responses/load-balancers-single.json b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/v2/responses/load-balancers-single.json index a9a728bab15..19e65c2fc25 100644 --- a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/v2/responses/load-balancers-single.json +++ b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/v2/responses/load-balancers-single.json @@ -21,12 +21,12 @@ { "hostname": "host13.yahoo.com", "ipAddress": "127.0.13.1", - "port": 4443 + "port": 4080 }, { "hostname": "host14.yahoo.com", "ipAddress": "127.0.14.1", - "port": 4443 + "port": 4080 } ] } diff --git a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/v2/responses/load-balancers.json b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/v2/responses/load-balancers.json index 515081bcb8e..0b05a41af0a 100644 --- a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/v2/responses/load-balancers.json +++ b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/restapi/v2/responses/load-balancers.json @@ -21,12 +21,12 @@ { "hostname": "host1.yahoo.com", "ipAddress": "127.0.1.1", - "port": 4443 + "port": 4080 }, { "hostname": "host10.yahoo.com", "ipAddress": "127.0.10.1", - "port": 4443 + "port": 4080 } ] }, @@ -51,12 +51,12 @@ { "hostname": "host13.yahoo.com", "ipAddress": "127.0.13.1", - "port": 4443 + "port": 4080 }, { "hostname": "host14.yahoo.com", "ipAddress": "127.0.14.1", - "port": 4443 + "port": 4080 } ] } |