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 supported ESS settings to ES docs #57953

Merged

Conversation

lockewritesdocs
Copy link
Contributor

@lockewritesdocs lockewritesdocs commented Jun 10, 2020

This PR includes the following changes:

  • Adds an ESS icon to supported settings in the ES docs.
  • Defines ES settings supported by ESS that were previously mentioned but not defined.

7.x backport #57953
7.8 backport #57953

Closes #57486

@lockewritesdocs lockewritesdocs self-assigned this Jun 10, 2020
@lockewritesdocs
Copy link
Contributor Author

@elasticmachine update branch

@elasticmachine
Copy link
Collaborator

merge conflict between base and head

@lockewritesdocs lockewritesdocs marked this pull request as ready for review June 24, 2020 12:33
@lockewritesdocs lockewritesdocs requested a review from debadair June 24, 2020 12:34
@lockewritesdocs lockewritesdocs added the >docs General docs changes label Jun 24, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-docs (>docs)

@elasticmachine elasticmachine added the Team:Docs Meta label for docs team label Jun 24, 2020
docs/reference/docs/reindex.asciidoc Outdated Show resolved Hide resolved
@lockewritesdocs lockewritesdocs requested a review from gtback June 26, 2020 18:13
Copy link
Member

@gtback gtback left a comment

Choose a reason for hiding this comment

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

LGTM, but I'd want to make sure another ES writer gave this a 👍 too.

@@ -0,0 +1,46 @@
== Cloud warehouse
Copy link
Member

Choose a reason for hiding this comment

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

Do we still want to keep this file?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@gtback, I intended to keep this file because these settings were not surfaced anywhere specifically in the ES Reference.

Copy link
Contributor

Choose a reason for hiding this comment

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

This are not cloud-specific settings, so for maintenance purposes I'd prefer to add them to the appropriate settings/modules files and use anchors that do not reference cloud.

Copy link
Contributor Author

@lockewritesdocs lockewritesdocs Jun 30, 2020

Choose a reason for hiding this comment

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

@debadair, I moved those settings into a new Index management settings page to house some of these orphan settings that didn't fit well into existing categories. The page is somewhat of a miscellaneous index bucket, but I didn't want to bury these settings or shoehorn them into pages where they don't fit.

Another consideration is adding them to the ILM settings page as Miscellaneous index settings (like in the Cluster-level settings page.

Copy link
Contributor

Choose a reason for hiding this comment

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

I think having them on a separate page is the right call--they aren't really related to ILM, they are more akin to the cluster-level settings. (In fact, the one is a cluster setting.) Since they affect how you work with indices, though, grouping them under Index management seems reasonable.

Copy link
Contributor

@debadair debadair left a comment

Choose a reason for hiding this comment

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

Left some suggestions. In particular, I don't think we should label the missing settings with the cloud prefix, as they are not specific to cloud. Rather than adding a cloud warehouse file, I'd embed them in the relevant existing settings files.

docs/reference/docs/reindex.asciidoc Outdated Show resolved Hide resolved
docs/reference/docs/reindex.asciidoc Outdated Show resolved Hide resolved
docs/reference/modules/http.asciidoc Outdated Show resolved Hide resolved
docs/reference/modules/http.asciidoc Outdated Show resolved Hide resolved
@@ -0,0 +1,46 @@
== Cloud warehouse
Copy link
Contributor

Choose a reason for hiding this comment

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

This are not cloud-specific settings, so for maintenance purposes I'd prefer to add them to the appropriate settings/modules files and use anchors that do not reference cloud.

docs/reference/warehouse/wh_cloud.asciidoc Outdated Show resolved Hide resolved
docs/reference/warehouse/wh_cloud.asciidoc Outdated Show resolved Hide resolved
docs/reference/warehouse/wh_cloud.asciidoc Outdated Show resolved Hide resolved
docs/reference/warehouse/wh_cloud.asciidoc Outdated Show resolved Hide resolved
docs/reference/warehouse/wh_cloud.asciidoc Outdated Show resolved Hide resolved
@lockewritesdocs lockewritesdocs merged commit 3a1258f into elastic:master Jul 2, 2020
@lockewritesdocs lockewritesdocs deleted the docs__add-ess-icon-support branch July 2, 2020 18:13
lockewritesdocs pushed a commit to lockewritesdocs/elasticsearch that referenced this pull request Jul 2, 2020
* Adding ESS icons to supported ES settings.

* Adding new file for supported ESS settings.

* Adding supported ESS settings for HTTP and disk-based shard allocation.

* Adding more supported settings for ESS.

* Adding descriptions for each Cloud section, plus additional settings.

* Adding new warehouse file for Cloud, plus additional settings.

* Adding node settings for Cloud.

* Adding audit settings for Cloud.

* Resolving merge conflict.

* Adding SAML settings (part 1).

* Adding SAML realm encryption and signing settings.

* Adding SAML SSL settings.

* Adding Kerberos realm settings.

* Adding OpenID Connect Realm settings.

* Adding OpenID Connect SSL settings.

* Resolving leftover Git merge markers.

* Removing Cloud settings page and link to it.

* Add link to mapping source

* Update docs/reference/docs/reindex.asciidoc

* Incorporate edit of HTTP settings

* Remove "cloud" from tag and ID

* Remove "cloud" from tag and update description

* Remove "cloud" from tag and ID

* Change "whitelists" to "specifies"

* Remove "cloud" from end tag

* Removing cloud from IDs and tags.

* Changing link reference to fix build issue.

* Adding index management page for missing settings.

* Removing warehouse file for Cloud and moving settings elsewhere.

* Clarifying true/false usage of http.detailed_errors.enabled.

* Changing underscore to dash in link to fix ci build.
lockewritesdocs pushed a commit that referenced this pull request Jul 2, 2020
* Adding ESS icons to supported ES settings.

* Adding new file for supported ESS settings.

* Adding supported ESS settings for HTTP and disk-based shard allocation.

* Adding more supported settings for ESS.

* Adding descriptions for each Cloud section, plus additional settings.

* Adding new warehouse file for Cloud, plus additional settings.

* Adding node settings for Cloud.

* Adding audit settings for Cloud.

* Resolving merge conflict.

* Adding SAML settings (part 1).

* Adding SAML realm encryption and signing settings.

* Adding SAML SSL settings.

* Adding Kerberos realm settings.

* Adding OpenID Connect Realm settings.

* Adding OpenID Connect SSL settings.

* Resolving leftover Git merge markers.

* Removing Cloud settings page and link to it.

* Add link to mapping source

* Update docs/reference/docs/reindex.asciidoc

* Incorporate edit of HTTP settings

* Remove "cloud" from tag and ID

* Remove "cloud" from tag and update description

* Remove "cloud" from tag and ID

* Change "whitelists" to "specifies"

* Remove "cloud" from end tag

* Removing cloud from IDs and tags.

* Changing link reference to fix build issue.

* Adding index management page for missing settings.

* Removing warehouse file for Cloud and moving settings elsewhere.

* Clarifying true/false usage of http.detailed_errors.enabled.

* Changing underscore to dash in link to fix ci build.
lockewritesdocs pushed a commit to lockewritesdocs/elasticsearch that referenced this pull request Jul 2, 2020
* Adding ESS icons to supported ES settings.

* Adding new file for supported ESS settings.

* Adding supported ESS settings for HTTP and disk-based shard allocation.

* Adding more supported settings for ESS.

* Adding descriptions for each Cloud section, plus additional settings.

* Adding new warehouse file for Cloud, plus additional settings.

* Adding node settings for Cloud.

* Adding audit settings for Cloud.

* Resolving merge conflict.

* Adding SAML settings (part 1).

* Adding SAML realm encryption and signing settings.

* Adding SAML SSL settings.

* Adding Kerberos realm settings.

* Adding OpenID Connect Realm settings.

* Adding OpenID Connect SSL settings.

* Resolving leftover Git merge markers.

* Removing Cloud settings page and link to it.

* Add link to mapping source

* Update docs/reference/docs/reindex.asciidoc

* Incorporate edit of HTTP settings

* Remove "cloud" from tag and ID

* Remove "cloud" from tag and update description

* Remove "cloud" from tag and ID

* Change "whitelists" to "specifies"

* Remove "cloud" from end tag

* Removing cloud from IDs and tags.

* Changing link reference to fix build issue.

* Adding index management page for missing settings.

* Removing warehouse file for Cloud and moving settings elsewhere.

* Clarifying true/false usage of http.detailed_errors.enabled.

* Changing underscore to dash in link to fix ci build.
lockewritesdocs pushed a commit that referenced this pull request Jul 3, 2020
* Adding ESS icons to supported ES settings.

* Adding new file for supported ESS settings.

* Adding supported ESS settings for HTTP and disk-based shard allocation.

* Adding more supported settings for ESS.

* Adding descriptions for each Cloud section, plus additional settings.

* Adding new warehouse file for Cloud, plus additional settings.

* Adding node settings for Cloud.

* Adding audit settings for Cloud.

* Resolving merge conflict.

* Adding SAML settings (part 1).

* Adding SAML realm encryption and signing settings.

* Adding SAML SSL settings.

* Adding Kerberos realm settings.

* Adding OpenID Connect Realm settings.

* Adding OpenID Connect SSL settings.

* Resolving leftover Git merge markers.

* Removing Cloud settings page and link to it.

* Add link to mapping source

* Update docs/reference/docs/reindex.asciidoc

* Incorporate edit of HTTP settings

* Remove "cloud" from tag and ID

* Remove "cloud" from tag and update description

* Remove "cloud" from tag and ID

* Change "whitelists" to "specifies"

* Remove "cloud" from end tag

* Removing cloud from IDs and tags.

* Changing link reference to fix build issue.

* Adding index management page for missing settings.

* Removing warehouse file for Cloud and moving settings elsewhere.

* Clarifying true/false usage of http.detailed_errors.enabled.

* Changing underscore to dash in link to fix ci build.
lockewritesdocs pushed a commit that referenced this pull request Oct 11, 2021
#57953 introduced changes that added ESS icons to many Elasticsearch settings. As part of those changes, the default value for `cluster.indices.close.enable` was indicated as `false`, when it should be `true`. This PR updates the default value to `true`. 

Closes #78877
lockewritesdocs pushed a commit that referenced this pull request Oct 13, 2021
* [DOCS] Fix default value for closed indices

#57953 introduced changes that added ESS icons to many Elasticsearch settings. As part of those changes, the default value for `cluster.indices.close.enable` was indicated as `false`, when it should be `true`. This PR updates the default value to `true`. 

Closes #78877

* Update description

* Update note to remove outdated claims
lockewritesdocs pushed a commit to lockewritesdocs/elasticsearch that referenced this pull request Oct 13, 2021
* [DOCS] Fix default value for closed indices

elastic#57953 introduced changes that added ESS icons to many Elasticsearch settings. As part of those changes, the default value for `cluster.indices.close.enable` was indicated as `false`, when it should be `true`. This PR updates the default value to `true`. 

Closes elastic#78877

* Update description

* Update note to remove outdated claims
lockewritesdocs pushed a commit to lockewritesdocs/elasticsearch that referenced this pull request Oct 13, 2021
* [DOCS] Fix default value for closed indices

elastic#57953 introduced changes that added ESS icons to many Elasticsearch settings. As part of those changes, the default value for `cluster.indices.close.enable` was indicated as `false`, when it should be `true`. This PR updates the default value to `true`. 

Closes elastic#78877

* Update description

* Update note to remove outdated claims
lockewritesdocs pushed a commit to lockewritesdocs/elasticsearch that referenced this pull request Oct 13, 2021
* [DOCS] Fix default value for closed indices

elastic#57953 introduced changes that added ESS icons to many Elasticsearch settings. As part of those changes, the default value for `cluster.indices.close.enable` was indicated as `false`, when it should be `true`. This PR updates the default value to `true`. 

Closes elastic#78877

* Update description

* Update note to remove outdated claims
lockewritesdocs pushed a commit to lockewritesdocs/elasticsearch that referenced this pull request Oct 13, 2021
* [DOCS] Fix default value for closed indices

elastic#57953 introduced changes that added ESS icons to many Elasticsearch settings. As part of those changes, the default value for `cluster.indices.close.enable` was indicated as `false`, when it should be `true`. This PR updates the default value to `true`. 

Closes elastic#78877

* Update description

* Update note to remove outdated claims
lockewritesdocs pushed a commit to lockewritesdocs/elasticsearch that referenced this pull request Oct 13, 2021
* [DOCS] Fix default value for closed indices

elastic#57953 introduced changes that added ESS icons to many Elasticsearch settings. As part of those changes, the default value for `cluster.indices.close.enable` was indicated as `false`, when it should be `true`. This PR updates the default value to `true`. 

Closes elastic#78877

* Update description

* Update note to remove outdated claims
lockewritesdocs pushed a commit to lockewritesdocs/elasticsearch that referenced this pull request Oct 13, 2021
* [DOCS] Fix default value for closed indices

elastic#57953 introduced changes that added ESS icons to many Elasticsearch settings. As part of those changes, the default value for `cluster.indices.close.enable` was indicated as `false`, when it should be `true`. This PR updates the default value to `true`. 

Closes elastic#78877

* Update description

* Update note to remove outdated claims
lockewritesdocs pushed a commit to lockewritesdocs/elasticsearch that referenced this pull request Oct 13, 2021
* [DOCS] Fix default value for closed indices

elastic#57953 introduced changes that added ESS icons to many Elasticsearch settings. As part of those changes, the default value for `cluster.indices.close.enable` was indicated as `false`, when it should be `true`. This PR updates the default value to `true`. 

Closes elastic#78877

* Update description

* Update note to remove outdated claims
lockewritesdocs pushed a commit to lockewritesdocs/elasticsearch that referenced this pull request Oct 13, 2021
* [DOCS] Fix default value for closed indices

elastic#57953 introduced changes that added ESS icons to many Elasticsearch settings. As part of those changes, the default value for `cluster.indices.close.enable` was indicated as `false`, when it should be `true`. This PR updates the default value to `true`. 

Closes elastic#78877

* Update description

* Update note to remove outdated claims
elasticsearchmachine pushed a commit that referenced this pull request Oct 13, 2021
* [DOCS] Fix default value for closed indices

#57953 introduced changes that added ESS icons to many Elasticsearch settings. As part of those changes, the default value for `cluster.indices.close.enable` was indicated as `false`, when it should be `true`. This PR updates the default value to `true`. 

Closes #78877

* Update description

* Update note to remove outdated claims
elasticsearchmachine pushed a commit that referenced this pull request Oct 13, 2021
* [DOCS] Fix default value for closed indices

#57953 introduced changes that added ESS icons to many Elasticsearch settings. As part of those changes, the default value for `cluster.indices.close.enable` was indicated as `false`, when it should be `true`. This PR updates the default value to `true`. 

Closes #78877

* Update description

* Update note to remove outdated claims
elasticsearchmachine pushed a commit that referenced this pull request Oct 13, 2021
* [DOCS] Fix default value for closed indices

#57953 introduced changes that added ESS icons to many Elasticsearch settings. As part of those changes, the default value for `cluster.indices.close.enable` was indicated as `false`, when it should be `true`. This PR updates the default value to `true`. 

Closes #78877

* Update description

* Update note to remove outdated claims
elasticsearchmachine pushed a commit that referenced this pull request Oct 13, 2021
* [DOCS] Fix default value for closed indices

#57953 introduced changes that added ESS icons to many Elasticsearch settings. As part of those changes, the default value for `cluster.indices.close.enable` was indicated as `false`, when it should be `true`. This PR updates the default value to `true`. 

Closes #78877

* Update description

* Update note to remove outdated claims
elasticsearchmachine pushed a commit that referenced this pull request Oct 13, 2021
* [DOCS] Fix default value for closed indices

#57953 introduced changes that added ESS icons to many Elasticsearch settings. As part of those changes, the default value for `cluster.indices.close.enable` was indicated as `false`, when it should be `true`. This PR updates the default value to `true`. 

Closes #78877

* Update description

* Update note to remove outdated claims
elasticsearchmachine pushed a commit that referenced this pull request Oct 13, 2021
* [DOCS] Fix default value for closed indices

#57953 introduced changes that added ESS icons to many Elasticsearch settings. As part of those changes, the default value for `cluster.indices.close.enable` was indicated as `false`, when it should be `true`. This PR updates the default value to `true`. 

Closes #78877

* Update description

* Update note to remove outdated claims
elasticsearchmachine pushed a commit that referenced this pull request Oct 13, 2021
* [DOCS] Fix default value for closed indices

#57953 introduced changes that added ESS icons to many Elasticsearch settings. As part of those changes, the default value for `cluster.indices.close.enable` was indicated as `false`, when it should be `true`. This PR updates the default value to `true`. 

Closes #78877

* Update description

* Update note to remove outdated claims
elasticsearchmachine pushed a commit that referenced this pull request Oct 13, 2021
* [DOCS] Fix default value for closed indices

#57953 introduced changes that added ESS icons to many Elasticsearch settings. As part of those changes, the default value for `cluster.indices.close.enable` was indicated as `false`, when it should be `true`. This PR updates the default value to `true`. 

Closes #78877

* Update description

* Update note to remove outdated claims
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>docs General docs changes Team:Docs Meta label for docs team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add ESS icons to Elasticsearch docs for supported Cloud settings
4 participants