Skip to content

Conversation

@lcawl
Copy link
Contributor

@lcawl lcawl commented May 7, 2020

Related to elastic/logstash#11843, elastic/kibana#64661, https://github.com/elastic/elastic-stack-testing/pull/549

This PR synchs Elasticsearch monitoring documentation with terminology changes across the stack (i.e. refer to "internal" collection as "legacy" collection). It also groups all of the pages related to the legacy collection methods under a "Legacy collection methods" section in the navigation.

Preview:
http://elasticsearch_56395.docs-preview.app.elstc.co/diff
For example: http://elasticsearch_56395.docs-preview.app.elstc.co/guide/en/elasticsearch/reference/master/collecting-monitoring-data.html

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-docs (>docs)

@elasticmachine elasticmachine added the Team:Docs Meta label for docs team label May 7, 2020
@elasticmachine
Copy link
Collaborator

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

@elasticmachine elasticmachine added the Team:Data Management Meta label for data/management team label May 7, 2020
@lcawl lcawl marked this pull request as ready for review May 7, 2020 22:18
@lcawl lcawl requested a review from chrisronline May 7, 2020 22:18
@lcawl lcawl requested review from CBlaisure and jakelandis and removed request for CBlaisure and chrisronline May 15, 2020 16:17
@jakelandis
Copy link
Contributor

jakelandis commented May 27, 2020

@lcawl - apologizes for the delay this slipped off my radar. The text changes look good, but have a bit of a concern about rename.

@ycombinator @chrisronline - I worry that changing this to legacy in our public docs is a bit premature. Given that cloud uses these collectors and if you want to export monitoring data from cloud to on prem or to another cloud instance these collectors are the only way. I think we need to be careful calling something "legacy" without an alternative across deployment scenarios.

@chrisronline
Copy link
Contributor

cc @ravikesarwani

@chrisronline
Copy link
Contributor

@jakelandis I'm going to assume that you aren't going to be comfortable marking internal/legacy collectors as deprecated in the docs either? We're planning on doing that across the board in 7.9 but I want to make sure we think through your concerns before committing to anything on the Elasticsearch side.

@jakelandis
Copy link
Contributor

I'm going to assume that you aren't going to be comfortable marking internal/legacy collectors as deprecated in the docs either?

@chrisronline I checked out the documentation for ESS/ECE where monitoring is implemented as abstraction through the platform (i.e. not requiring the user to setting settings these directly) and their doc reflects that. So I don't have strong objections, but did want to raise the point. I will defer to whatever decision is made, (which sounds like it already been made).

@lcawl - LGTM

@lcawl lcawl force-pushed the legacy-monitoring branch from 99eaadf to 228fc1b Compare June 11, 2020 00:51
@lcawl lcawl merged commit e9d13ec into elastic:master Jun 11, 2020
@lcawl lcawl deleted the legacy-monitoring branch June 11, 2020 17:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Data Management/Monitoring >docs General docs changes Team:Data Management Meta label for data/management team Team:Docs Meta label for docs team v7.7.2 v7.8.0 v7.9.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants