Skip to content

[7.14] Properly apply system flag on data streams when restoring a snapshot #75854

Merged
elasticsearchmachine merged 2 commits intoelastic:7.14from
AthenaEryma:backport/7.14/system-data-stream-restore-fix
Jul 29, 2021
Merged

[7.14] Properly apply system flag on data streams when restoring a snapshot #75854
elasticsearchmachine merged 2 commits intoelastic:7.14from
AthenaEryma:backport/7.14/system-data-stream-restore-fix

Conversation

@AthenaEryma
Copy link
Copy Markdown
Contributor

@AthenaEryma AthenaEryma commented Jul 29, 2021

This commit modifies the restore process to ensure that the system
flag is properly applied to restored data streams. Otherwise, this
flag is lost when restoring system data streams, which causes errors
and/or assertion failures as the backing indices are properly marked
as system indices, but the restored data stream is no longer a
system data stream.

Also adds a test to ensure this flag survives a round trip through
the snapshot/restore process.

Backport of #75819

elastic#75819)

This commit modifies the restore process to ensure that the `system`
flag is properly applied to restored data streams. Otherwise, this
flag is lost when restoring system data streams, which causes errors
and/or assertion failures as the backing indices are properly marked
as system indices, but the restored data stream is no longer a
system data stream.

Also adds a test to ensure this flag survives a round trip through
the snapshot/restore process.
@AthenaEryma AthenaEryma added backport v7.14.1 auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) labels Jul 29, 2021
@elasticsearchmachine elasticsearchmachine merged commit dd5a0a2 into elastic:7.14 Jul 29, 2021
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 v7.14.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants