-
Notifications
You must be signed in to change notification settings - Fork 25.6k
[DOCS] Archived settings block other setting updates #82069
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] Archived settings block other setting updates #82069
Conversation
|
Pinging @elastic/es-docs (Team:Docs) |
|
Pinging @elastic/es-core-infra (Team:Core/Infra) |
j-bennet
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! 👍 I really hope the inconsistent behavior can be fixed in the future though, besides only being clarified in the documentation.
| Removing archived index settings requires a reindex after the upgrade. However, | ||
| reindexing can be resource intensive. Because {es} ignores archived settings, | ||
| you can safely leave them in place if wanted. | ||
| Removing an index's archived index settings requires a <<docs-reindex,reindex>>. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This clarification is great.
|
@DaveCTurner @gwbrown Do one of you mind taking a look at this PR at your convenience? Thank you! |
|
@elasticmachine update branch |
|
@elasticmachine update branch |
gwbrown
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks! And again, apologies for the delay!
|
Thanks @gwbrown! |
💚 All backports created successfully
Questions ?Please refer to the Backport tool documentation |
💚 All backports created successfully
Questions ?Please refer to the Backport tool documentation |
**Changes:** * Notes that archived cluster settings block cluster setting updates. Previously, the docs stated that ES ignored archived cluster settings. * Notes that archived index settings can block index settings updates. For example, it blocks `index.hidden` but not `number_of_replicas`. Previously, the docs stated that you could safely ignore archived index settings. Relates #78351 Closes #61175 (cherry picked from commit 47d45d0)
**Changes:** * Notes that archived cluster settings block cluster setting updates. Previously, the docs stated that ES ignored archived cluster settings. * Notes that archived index settings can block index settings updates. For example, it blocks `index.hidden` but not `number_of_replicas`. Previously, the docs stated that you could safely ignore archived index settings. Relates #78351 Closes #61175 (cherry picked from commit 47d45d0)
**Changes:** * Notes that archived cluster settings block cluster setting updates. Previously, the docs stated that ES ignored archived cluster settings. * Notes that archived index settings can block index settings updates. For example, it blocks `index.hidden` but not `number_of_replicas`. Previously, the docs stated that you could safely ignore archived index settings. Relates #78351 Closes #61175 (cherry picked from commit 47d45d0)
**Changes:** * Notes that archived cluster settings block cluster setting updates. Previously, the docs stated that ES ignored archived cluster settings. * Notes that archived index settings can block index settings updates. For example, it blocks `index.hidden` but not `number_of_replicas`. Previously, the docs stated that you could safely ignore archived index settings. Relates #78351 Closes #61175 (cherry picked from commit 47d45d0)
Changes:
index.hiddenbut notnumber_of_replicas. Previously, the docs stated that you could safely ignore archived index settings.Preview
https://elasticsearch_82069.docs-preview.app.elstc.co/guide/en/elasticsearch/reference/master/archived-settings.html
Relates #78351
Closes #61175