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

[7.12] Restrict ILM frozen phase to searchable snapshot actions only (#70158) #70171

Merged
merged 2 commits into from
Mar 9, 2021

Conversation

dakrone
Copy link
Member

@dakrone dakrone commented Mar 9, 2021

This commit changes the frozen phase within ILM in the following ways:

  • The searchable_snapshot action now no longer takes a storage parameter. The storage type is
    determined by the phase within which it is invoked (shared cache for frozen and full copy for
    everything else).
  • The frozen phase in ILM now no longer allows any actions other than searchable_snapshot
  • If a frozen phase is provided, it must include a searchable_snapshot action.

These changes may seem breaking, but since they are intended to go back to 7.12 which has not been
released yet, they are not truly breaking changes.

Backport of #70158

…c#70158)

This commit changes the frozen phase within ILM in the following ways:

- The `searchable_snapshot` action now no longer takes a `storage` parameter. The storage type is
determined by the phase within which it is invoked (shared cache for frozen and full copy for
everything else).
- The frozen phase in ILM now no longer allows *any* actions other than `searchable_snapshot`
- If a frozen phase is provided, it *must* include a `searchable_snapshot` action.

These changes may seem breaking, but since they are intended to go back to 7.12 which has not been
released yet, they are not truly breaking changes.
@dakrone dakrone added :Data Management/ILM+SLM Index and Snapshot lifecycle management backport v7.12.0 labels Mar 9, 2021
@elasticmachine elasticmachine added the Team:Data Management Meta label for data/management team label Mar 9, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features (Team:Core/Features)

dakrone added a commit to dakrone/elasticsearch that referenced this pull request Mar 9, 2021
With elastic#70158 (master), elastic#70170 (7.x), and elastic#70171 (7.12) merged, we can now re-enable backwards
compatibilty tests on master and 7.x.
@dakrone dakrone merged commit 3df17b6 into elastic:7.12 Mar 9, 2021
@dakrone dakrone deleted the backport/7.12/pr-70158 branch March 9, 2021 19:45
dakrone added a commit that referenced this pull request Mar 9, 2021
With #70158 (master), #70170 (7.x), and #70171 (7.12) merged, we can now re-enable backwards
compatibilty tests on master and 7.x.
dakrone added a commit to dakrone/elasticsearch that referenced this pull request Mar 9, 2021
…ic#70179)

With elastic#70158 (master), elastic#70170 (7.x), and elastic#70171 (7.12) merged, we can now re-enable backwards
compatibilty tests on master and 7.x.
dakrone added a commit that referenced this pull request Mar 10, 2021
…70179) (#70188)

With #70158 (master), #70170 (7.x), and #70171 (7.12) merged, we can now re-enable backwards
compatibilty tests on master and 7.x.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport :Data Management/ILM+SLM Index and Snapshot lifecycle management Team:Data Management Meta label for data/management team v7.12.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants