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

[DOCS] Add deprecation admon to unfreeze index API #78886

Merged
merged 2 commits into from
Oct 11, 2021
Merged

Conversation

stefnestor
Copy link
Contributor

Hey, team! Copying over deprecation warning from Freeze Index API page for parity.

Hey, team! Copying over deprecation warning from [Freeze Index API](https://www.elastic.co/guide/en/elasticsearch/reference/current/freeze-index-api.html) page for parity.
@stefnestor stefnestor added >docs General docs changes Team:Docs Meta label for docs team labels Oct 9, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-docs (Team:Docs)

@stefnestor stefnestor requested a review from jrodewig October 9, 2021 21:01
@elasticsearchmachine elasticsearchmachine added external-contributor Pull request authored by a developer outside the Elasticsearch team v7.15.1 labels Oct 9, 2021
@jrodewig jrodewig added the :Data Management/Indices APIs APIs to create and manage indices and templates label Oct 11, 2021
@elasticmachine elasticmachine added the Team:Data Management Meta label for data/management team label Oct 11, 2021
@jrodewig jrodewig removed their request for review October 11, 2021 13:19
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-data-management (Team:Data Management)

@jrodewig jrodewig removed the v7.15.1 label Oct 11, 2021
@jrodewig
Copy link
Contributor

Hey @stefnestor

We intentionally omitted a deprecation admon here. While we deprecated the freeze endpoints/actions in 7.15+, we're delaying deprecation of the unfreeze API until 8.x. Although freezing new indices is discouraged, users may still need to unfreeze older frozen indices as an "escape hatch."

You can see the whole deprecation plan here:
#70192

@jrodewig
Copy link
Contributor

Nevermind what I said. @danhermann pointed out that this API is in fact deprecated. However, we do still plan to remove the unfreeze API after other freeze endpoints/actions are removed.

I'll make a few small tweaks and get this merged in. Thanks @stefnestor.

@jrodewig jrodewig reopened this Oct 11, 2021
@jrodewig jrodewig changed the title +Deprecation warning onto Unfreeze Index [DOCS] Add deprecation admin to unfreeze index API Oct 11, 2021
@jrodewig jrodewig changed the title [DOCS] Add deprecation admin to unfreeze index API [DOCS] Add deprecation admon to unfreeze index API Oct 11, 2021
@jrodewig jrodewig merged commit 82431e3 into 7.15 Oct 11, 2021
@jrodewig jrodewig deleted the stefnestor-patch-5 branch October 11, 2021 14:16
jrodewig added a commit that referenced this pull request Oct 11, 2021
Adds a missing deprecation admonition to the unfreeze index API.

Co-authored-by: James Rodewig <[email protected]>

Co-authored-by: Stef Nestor <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Data Management/Indices APIs APIs to create and manage indices and templates >docs General docs changes external-contributor Pull request authored by a developer outside the Elasticsearch team Team:Data Management Meta label for data/management team Team:Docs Meta label for docs team v7.15.2 v7.16.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants