Skip to content

[9.2] Fix testAbortSnapshotWhileRemovingNode (#142852)#143000

Merged
elasticsearchmachine merged 2 commits intoelastic:9.2from
ywangd:backport/9.2/pr-142852
Feb 24, 2026
Merged

[9.2] Fix testAbortSnapshotWhileRemovingNode (#142852)#143000
elasticsearchmachine merged 2 commits intoelastic:9.2from
ywangd:backport/9.2/pr-142852

Conversation

@ywangd
Copy link
Copy Markdown
Member

@ywangd ywangd commented Feb 24, 2026

Backport

This will backport the following commits from main to 9.2:

Questions ?

Please refer to the Backport tool documentation

Clear the mock transport rule in testAbortSnapshotWhileRemovingNode
after releasing the single update_snapshot_status request we
coordinate on, so any further requests are handled normally and
complete before teardown. This prevents the test from failing in
assertAfterTest() with "All incoming requests on node [X] should
have finished".

Closes elastic#142805

* Fix testAbortSnapshotWhileRemovingNode

Clear the mock transport rule in testAbortSnapshotWhileRemovingNode
after releasing the single update_snapshot_status request we
coordinate on, so any further requests are handled normally and
complete before teardown. This prevents the test from failing in
assertAfterTest() with "All incoming requests on node [X] should
have finished".

Closes elastic#142805

* Update comment

* Remove second masterTransportService.clearAllRules();

* Use two CountDownLatches

* Add extra masterTransportService.clearAllRules();

* [CI] Auto commit changes from spotless

---------

Co-authored-by: elasticsearchmachine <infra-root+elasticsearchmachine@elastic.co>
(cherry picked from commit c8d36f0)
@ywangd ywangd added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Feb 24, 2026
@elasticsearchmachine elasticsearchmachine merged commit 1ddee86 into elastic:9.2 Feb 24, 2026
35 checks passed
@ywangd ywangd deleted the backport/9.2/pr-142852 branch February 24, 2026 23:22
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 v9.2.7

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants