[DOCS] Merge monitoring sections in Elasticsearch book #46885
Conversation
lcawl
added
>docs
|
Pinging @elastic/es-core-features |
|
Pinging @elastic/es-docs |
debadair
left a comment
debadair
left a comment
There was a problem hiding this comment.
A couple minor comments, but LGTM
| @@ -0,0 +1,39 @@ | |||
| [role="xpack"] | |||
There was a problem hiding this comment.
Since this file is new, I'd align the filename with the top-level anchor.
| [testenv="basic"] | ||
| [[es-monitoring]] | ||
| = Monitoring {es} | ||
| [[monitor-cluster]] |
There was a problem hiding this comment.
Since this is a top-level heading, I'd be inclined to be redundant and include "elasticsearch" in the anchor: [monitor-elasticsearch-cluster]. Similarly, I'd insert {es} before cluster in the paragraph below.
|
@lcawl I can no longer see the preview from the link you posted, I'm afraid. |
Sorry, the URL changed. See http://elasticsearch_46885.docs-preview.app.elstc.co/guide/en/elasticsearch/reference/master/monitor-elasticsearch-cluster.html |
7 participants
Related to #46718
This PR creates a "Monitor a cluster" part at the top level of the Elasticsearch book, which contains the following existing content:
https://www.elastic.co/guide/en/elasticsearch/reference/master/es-monitoring.html
https://www.elastic.co/guide/en/elasticsearch/reference/master/configuring-monitoring.html
It also adds a brief introduction, which is copied from https://www.elastic.co/guide/en/elastic-stack-overview/master/xpack-monitoring.html
Additional content might be moved from the Stack Overview in subsequent PRs.
PREVIEW: http://elasticsearch_46885.docs-preview.app.elstc.co/guide/en/elasticsearch/reference/master/monitor-elasticsearch-cluster.html