Skip to content

Commit 6042942

Browse files
authored
[7.17] Fix Netty4TransportMultiPortIntegrationIT#testThatTransportClientCanConnect (#108112)
#103894 increased the range of available client ports on Windows for a node, but didn't update the range of ports that the transport client test iterates through in trying to connect to the node. Both test failures reported in #107757 happened on Windows machines. See #103894 Resolves #107757
1 parent 8474043 commit 6042942

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

modules/transport-netty4/src/internalClusterTest/java/org/elasticsearch/transport/netty4/Netty4TransportMultiPortIntegrationIT.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ public void testThatTransportClientCanConnect() throws Exception {
7272
.build();
7373
// we have to test all the ports that the data node might be bound to
7474
try (TransportClient transportClient = new MockTransportClient(settings, Arrays.asList(Netty4Plugin.class))) {
75-
for (int i = 0; i <= 10; i++) {
75+
for (int i = 0; i <= NUMBER_OF_CLIENT_PORTS; i++) {
7676
transportClient.addTransportAddress(new TransportAddress(InetAddress.getByName("127.0.0.1"), randomPort + i));
7777
}
7878
ClusterHealthResponse response = transportClient.admin().cluster().prepareHealth().get();

0 commit comments

Comments
 (0)