Skip to content

[7.x] Restrict ILM frozen phase to searchable snapshot actions only (#70158)#70170

Merged
dakrone merged 2 commits intoelastic:7.xfrom
dakrone:backport/7.x/pr-70158
Mar 9, 2021
Merged

[7.x] Restrict ILM frozen phase to searchable snapshot actions only (#70158)#70170
dakrone merged 2 commits intoelastic:7.xfrom
dakrone:backport/7.x/pr-70158

Conversation

@dakrone
Copy link
Copy Markdown
Member

@dakrone dakrone commented Mar 9, 2021

Backports the following commits to 7.x:

…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 backport :Data Management/ILM+SLM DO NOT USE. Use ":StorageEngine/ILM" or ":Distributed Coordination/SLM" instead. v7.13.0 labels Mar 9, 2021
@elasticmachine elasticmachine added the Team:Data Management (obsolete) DO NOT USE. This team no longer exists. label Mar 9, 2021
@elasticmachine
Copy link
Copy Markdown
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.
The action was recently removed from the list of acceptable actions (elastic#70158), the test shouldn't use
it any more.
@dakrone dakrone merged commit f69cb27 into elastic:7.x Mar 9, 2021
@dakrone dakrone deleted the backport/7.x/pr-70158 branch March 9, 2021 20:28
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 DO NOT USE. Use ":StorageEngine/ILM" or ":Distributed Coordination/SLM" instead. Team:Data Management (obsolete) DO NOT USE. This team no longer exists. v7.13.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants