Skip to content

Reduce channels in AbstractSimpleTransportTestCase (#34863)#34880

Merged
Tim-Brooks merged 1 commit intoelastic:6.xfrom
Tim-Brooks:backport_reduce_connections
Oct 26, 2018
Merged

Reduce channels in AbstractSimpleTransportTestCase (#34863)#34880
Tim-Brooks merged 1 commit intoelastic:6.xfrom
Tim-Brooks:backport_reduce_connections

Conversation

@Tim-Brooks
Copy link
Copy Markdown
Contributor

This is related to #30876. The AbstractSimpleTransportTestCase initiates
many tcp connections. There are normally over 1,000 connections in
TIME_WAIT at the end of the test. This is because every test opens at
least two different transports that connect to each other with 13
channel connection profiles. This commit modifies the default
connection profile used by this test to 6. One connection for each
type, except for REG which gets 2 connections.

This is related to elastic#30876. The AbstractSimpleTransportTestCase initiates
many tcp connections. There are normally over 1,000 connections in
TIME_WAIT at the end of the test. This is because every test opens at
least two different transports that connect to each other with 13
channel connection profiles. This commit modifies the default
connection profile used by this test to 6. One connection for each
type, except for REG which gets 2 connections.
@Tim-Brooks Tim-Brooks added >test Issues or PRs that are addressing/adding tests :Distributed/Network Http and internode communication implementations v6.0.3 v6.1.5 v6.2.5 v6.3.3 v6.6.0 v6.4.4 v6.5.1 labels Oct 25, 2018
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-distributed

@Tim-Brooks
Copy link
Copy Markdown
Contributor Author

This is just a back port.

@Tim-Brooks Tim-Brooks merged commit 419ad45 into elastic:6.x Oct 26, 2018
Tim-Brooks added a commit that referenced this pull request Oct 26, 2018
This is related to #30876. The AbstractSimpleTransportTestCase initiates
many tcp connections. There are normally over 1,000 connections in
TIME_WAIT at the end of the test. This is because every test opens at
least two different transports that connect to each other with 13
channel connection profiles. This commit modifies the default
connection profile used by this test to 6. One connection for each
type, except for REG which gets 2 connections.
Tim-Brooks added a commit to Tim-Brooks/elasticsearch that referenced this pull request Oct 26, 2018
…lastic#34880)

This is related to elastic#30876. The AbstractSimpleTransportTestCase initiates
many tcp connections. There are normally over 1,000 connections in
TIME_WAIT at the end of the test. This is because every test opens at
least two different transports that connect to each other with 13
channel connection profiles. This commit modifies the default
connection profile used by this test to 6. One connection for each
type, except for REG which gets 2 connections.
Tim-Brooks added a commit that referenced this pull request Oct 26, 2018
…#34915)

This is related to #30876. The AbstractSimpleTransportTestCase initiates
many tcp connections. There are normally over 1,000 connections in
TIME_WAIT at the end of the test. This is because every test opens at
least two different transports that connect to each other with 13
channel connection profiles. This commit modifies the default
connection profile used by this test to 6. One connection for each
type, except for REG which gets 2 connections.
@Tim-Brooks Tim-Brooks deleted the backport_reduce_connections branch December 18, 2019 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed/Network Http and internode communication implementations >test Issues or PRs that are addressing/adding tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants