Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[8.18] Abort pending deletion on IndicesService stop (#123569) #123669

Merged
merged 1 commit into from
Feb 28, 2025

Conversation

ywangd
Copy link
Member

@ywangd ywangd commented Feb 28, 2025

Backport

This will backport the following commits from main to 8.18:

Questions ?

Please refer to the Backport tool documentation

When IndicesService is closed, the pending deletion may still be in
progress due to indices removed before IndicesService gets closed. If
the deletion stucks for some reason, it can stall the node shutdown.
This PR aborts the pending deletion more promptly by not retry after
IndicesService is stopped.

Resolves: elastic#121717 Resolves: elastic#121716  Resolves: elastic#122119
(cherry picked from commit c7e7dbe)

# Conflicts:
#	muted-tests.yml
@ywangd ywangd added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Feb 28, 2025
@elasticsearchmachine elasticsearchmachine merged commit 5ce6331 into elastic:8.18 Feb 28, 2025
15 checks passed
@ywangd ywangd deleted the backport/8.18/pr-123569 branch February 28, 2025 02: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 v8.18.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants