Skip to content

Conversation

@ywangd
Copy link
Member

@ywangd ywangd commented Jun 2, 2025

Backport

This will backport the following commits from main to 9.0:

Questions ?

Please refer to the Backport tool documentation

…stic#128650)

If an index is deleted after a snapshot has written its shardGenerations
file but before the snapshot is finalized, we exclude this index from the
snapshot because its indexMetadata is no longer available. However,
the shardGenerations file is still valid in that it is the latest copy with all
necessary information despite it containing an extra snapshot entry.
This is OK. Instead of dropping this shardGenerations file, this PR
changes to carry it forward by updating RepositoryData and relevant
in-progress snapshots so that the next finalization builds on top of this one.

Co-authored-by: David Turner <[email protected]>
(cherry picked from commit aa0397f)

# Conflicts:
#	server/src/main/java/org/elasticsearch/snapshots/SnapshotsService.java
@ywangd ywangd added the backport label Jun 2, 2025
@ywangd ywangd added >enhancement :Distributed Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) labels Jun 2, 2025
@elasticsearchmachine elasticsearchmachine merged commit e4340ec into elastic:9.0 Jun 2, 2025
16 checks passed
@ywangd ywangd deleted the backport/9.0/pr-128650 branch June 2, 2025 06:25
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 Coordination/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs >enhancement v9.0.3

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants