Skip to content

[8.2] Reduce port range re-use in tests (#85777)#85858

Merged
elasticsearchmachine merged 1 commit intoelastic:8.2from
DaveCTurner:backport/8.2/pr-85777
Apr 13, 2022
Merged

[8.2] Reduce port range re-use in tests (#85777)#85858
elasticsearchmachine merged 1 commit intoelastic:8.2from
DaveCTurner:backport/8.2/pr-85777

Conversation

@DaveCTurner
Copy link
Copy Markdown
Member

Backports the following commits to 8.2:

Today we select the port range to use in tests by taking the Gradle
worker ID mod 223. We now use significantly more than 223 Gradle workers
in each test run which means port ranges are re-used, resulting in
spurious failures.

This commit replaces the magic number 223 with a computation defined in
terms of the available ports and the number of ports to allocate per
worker. It also reduces the number of ports per worker to 30 to try and
reduce the frequency of port clashes.
@DaveCTurner DaveCTurner added auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport labels Apr 13, 2022
@elasticsearchmachine elasticsearchmachine merged commit 60d8398 into elastic:8.2 Apr 13, 2022
@DaveCTurner DaveCTurner deleted the backport/8.2/pr-85777 branch April 13, 2022 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport v8.2.1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants