Skip to content

[8.5] Execute async cleanup tasks in CoordinatorTests (#91794)#91807

Merged
elasticsearchmachine merged 1 commit intoelastic:8.5from
DaveCTurner:backport/8.5/pr-91794
Nov 22, 2022
Merged

[8.5] Execute async cleanup tasks in CoordinatorTests (#91794)#91807
elasticsearchmachine merged 1 commit intoelastic:8.5from
DaveCTurner:backport/8.5/pr-91794

Conversation

@DaveCTurner
Copy link
Copy Markdown
Member

Backports the following commits to 8.5:

Closing the `JoinValidationService` will enqueue a cache-cleaning task
which must be executed to release any pages held by the cache.

This commit also introduces a deterministic leak detector to replace the
one based on garbage collection, because in these tests we know exactly
the expected lifecycle of all allocated pages.

Closes elastic#91599
Closes elastic#91379
Closes elastic#90576
@DaveCTurner DaveCTurner added :Distributed/Cluster Coordination Cluster formation and cluster state publication, including cluster membership and fault detection. >test Issues or PRs that are addressing/adding tests auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Distributed Meta label for distributed team. labels Nov 22, 2022
@elasticsearchmachine elasticsearchmachine merged commit 957bf16 into elastic:8.5 Nov 22, 2022
@DaveCTurner DaveCTurner deleted the backport/8.5/pr-91794 branch November 22, 2022 15: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 :Distributed/Cluster Coordination Cluster formation and cluster state publication, including cluster membership and fault detection. Team:Distributed Meta label for distributed team. >test Issues or PRs that are addressing/adding tests v8.5.3

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants