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] Fix <index> param def for delete index API #74674

Merged
merged 4 commits into from
Jul 1, 2021
Merged

[DOCS] Fix <index> param def for delete index API #74674

merged 4 commits into from
Jul 1, 2021

Conversation

LiamMacP
Copy link
Contributor

The current documentation leads to assume that you cannot delete an index if there is an alias that exists and links to it. This appears not to be the case and what it means is that it cannot be deleted through the alias name.

The current documentation leads to assume that you cannot delete an index if there is an alias that exists and links to it. This appears not to be the case and what it means is that it cannot be deleted through the alias name.
@elasticsearchmachine elasticsearchmachine added v8.0.0 external-contributor Pull request authored by a developer outside the Elasticsearch team labels Jun 29, 2021
@jrodewig jrodewig added :Data Management/Indices APIs APIs to create and manage indices and templates v7.13.4 v7.14.1 v7.15.0 labels Jul 1, 2021
@elasticmachine elasticmachine added the Team:Data Management Meta label for data/management team label Jul 1, 2021
@elasticmachine
Copy link
Collaborator

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

Copy link
Contributor

@jrodewig jrodewig left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing this @LiamMacP. The original wording was indeed unclear.

I've pushed some changes to fix some other errors and reorganize the info for this parameter. With those changes, this LGTM.

Thanks again!

@jrodewig
Copy link
Contributor

jrodewig commented Jul 1, 2021

@elasticmachine ok to test

@jrodewig jrodewig changed the title Update delete-index.asciidoc [DOCS] Fix <index> param def for delete index API Jul 1, 2021
@jrodewig jrodewig merged commit ba227f9 into elastic:master Jul 1, 2021
@LiamMacP LiamMacP deleted the patch-2 branch July 1, 2021 14:18
jrodewig added a commit that referenced this pull request Jul 1, 2021
Clarifies that you cannot specify an alias in the delete index API. You _can_ delete indices with an alias.

Co-authored-by: James Rodewig <40268737+jrodewig@users.noreply.github.com>
Co-authored-by: Liam MacPherson <11508628+LiamMacP@users.noreply.github.com>
jrodewig added a commit that referenced this pull request Jul 1, 2021
Clarifies that you cannot specify an alias in the delete index API. You _can_ delete indices with an alias.

Co-authored-by: James Rodewig <40268737+jrodewig@users.noreply.github.com>

Co-authored-by: Liam MacPherson <11508628+LiamMacP@users.noreply.github.com>
elasticsearchmachine pushed a commit that referenced this pull request Jul 1, 2021
Clarifies that you cannot specify an alias in the delete index API. You _can_ delete indices with an alias.

Co-authored-by: James Rodewig <40268737+jrodewig@users.noreply.github.com>
# Conflicts:
#	docs/reference/indices/delete-index.asciidoc

Co-authored-by: Liam MacPherson <11508628+LiamMacP@users.noreply.github.com>
jrodewig added a commit that referenced this pull request Aug 25, 2021
In 7.x versions, the `action.destructive_requires_name` cluster setting defaults to `false`. 
The 7.x backports of #74674 introduced a docs bug that indicate the setting defaults to `true`.
This reverts the related changes.
jrodewig added a commit that referenced this pull request Aug 25, 2021
…#76926)

In 7.x versions, the `action.destructive_requires_name` cluster setting defaults to `false`. 
The 7.x backports of #74674 introduced a docs bug that indicate the setting defaults to `true`.
This reverts the related changes.
jrodewig added a commit that referenced this pull request Aug 25, 2021
…#76927)

In 7.x versions, the `action.destructive_requires_name` cluster setting defaults to `false`. 
The 7.x backports of #74674 introduced a docs bug that indicate the setting defaults to `true`.
This reverts the related changes.
jrodewig added a commit that referenced this pull request Aug 25, 2021
…#76928)

In 7.x versions, the `action.destructive_requires_name` cluster setting defaults to `false`. 
The 7.x backports of #74674 introduced a docs bug that indicate the setting defaults to `true`.
This reverts the related changes.
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 external-contributor Pull request authored by a developer outside the Elasticsearch team Team:Data Management Meta label for data/management team v7.13.4 v7.14.0 v7.15.0 v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants