Skip to content

Fix ConcurrentSnapshotsIT#testBackToBackQueuedDeletes#144461

Merged
DaveCTurner merged 1 commit intoelastic:mainfrom
DaveCTurner:2026/03/18/ConcurrentSnapshotsIT-testBackToBackQueuedDeletes
Mar 19, 2026
Merged

Fix ConcurrentSnapshotsIT#testBackToBackQueuedDeletes#144461
DaveCTurner merged 1 commit intoelastic:mainfrom
DaveCTurner:2026/03/18/ConcurrentSnapshotsIT-testBackToBackQueuedDeletes

Conversation

@DaveCTurner
Copy link
Copy Markdown
Member

This test fails if it only has a single snapshot thread and that thread
gets blocked on the repository before the second snapshot has been
enqueued: since #141998 landed, enqueuing batches of deletes requires a
snapshot thread. This commit fixes the problem by adding a second
snapshot thread.

Closes #143387

This test fails if it only has a single snapshot thread and that thread
gets blocked on the repository before the second snapshot has been
enqueued: since elastic#141998 landed, enqueuing batches of deletes requires a
snapshot thread. This commit fixes the problem by adding a second
snapshot thread.

Closes elastic#143387
@DaveCTurner DaveCTurner requested a review from blerer March 18, 2026 07:14
@DaveCTurner DaveCTurner added >test Issues or PRs that are addressing/adding tests :Distributed/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs v9.4.0 labels Mar 18, 2026
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-distributed (Team:Distributed)

Copy link
Copy Markdown
Member

@ywangd ywangd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@DaveCTurner DaveCTurner merged commit 53f0516 into elastic:main Mar 19, 2026
36 checks passed
@DaveCTurner DaveCTurner deleted the 2026/03/18/ConcurrentSnapshotsIT-testBackToBackQueuedDeletes branch March 19, 2026 06:04
michalborek pushed a commit to michalborek/elasticsearch that referenced this pull request Mar 23, 2026
This test fails if it only has a single snapshot thread and that thread
gets blocked on the repository before the second snapshot has been
enqueued: since elastic#141998 landed, enqueuing batches of deletes requires a
snapshot thread. This commit fixes the problem by adding a second
snapshot thread.

Closes elastic#143387
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs Team:Distributed Meta label for distributed team. >test Issues or PRs that are addressing/adding tests v9.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] ConcurrentSnapshotsIT testBackToBackQueuedDeletes failing

3 participants