[Doc]Restructure monitoring to support new and legacy internal collectors for 7.x#11787
[Doc]Restructure monitoring to support new and legacy internal collectors for 7.x#11787karenzone wants to merge 1 commit intoelastic:7.16from
Conversation
…ollectors (elastic#11714) * [Doc] added description of xpack.monitoring.collection.write_direct.enabled setting * Added page to mark as deprecated the legacy internal collector and fixed all the `xpack.monitoring.*` references * Included legacy collector file into monitoring overview * Restructure monitoring docs * Incorporate review comments Co-authored-by: andsel <selva.andre@gmail.com>
|
@elasticmachine, run elasticsearch-ci/docs |
…tors (#11714) * [Doc] added description of xpack.monitoring.collection.write_direct.enabled setting * Added page to mark as deprecated the legacy internal collector and fixed all the `xpack.monitoring.*` references * Included legacy collector file into monitoring overview * Restructure monitoring docs * Incorporate review comments Co-authored-by: andsel <selva.andre@gmail.com> Fixes #11787
…tors (#11714) * [Doc] added description of xpack.monitoring.collection.write_direct.enabled setting * Added page to mark as deprecated the legacy internal collector and fixed all the `xpack.monitoring.*` references * Included legacy collector file into monitoring overview * Restructure monitoring docs * Incorporate review comments Co-authored-by: andsel <selva.andre@gmail.com> Fixes #11787
kaisecheng
left a comment
kaisecheng
left a comment
There was a problem hiding this comment.
@karenzone do you still want to merge to 7.16?
It is a clean backport, once the conflicts are solved, I can approve it.
There was a problem hiding this comment.
Jarvis merged these changes into 7.7 and 7.x, as indicated here: #11787 (comment). So that means that these changes are present in every branch from 7.7 and beyond. And yes, I verified this statement by checking github. A sample for reference: https://github.com/elastic/logstash/blame/7.16/docs/static/monitoring/collectors-legacy.asciidoc
Jarvis had a weird quirk whereby it wouldn't close a PR that was based out of any branch other than main. All that to say, we can safely close this PR as it has been merged. I will do that now.
Thanks for circling back around on this, @kaisecheng ❤️
|
Closing per this comment: #11787 (review). |
4 participants
Backports #11714 to 7.x
[Doc]Restructure monitoring docs to support new and legacy internal collectors #11714
xpack.monitoring.*referencesCo-authored-by: andsel selva.andre@gmail.com