-
Notifications
You must be signed in to change notification settings - Fork 159
Cluster monitoring #926
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
Cluster monitoring #926
Conversation
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.
Thanks @stefnestor ! I think as a process test, this is looking very much like we'd expect a docs contribution PR to look 👍
... but because so much content is still in flux this week and most of these changes aren't essential, I'd suggest keeping only the ECK/ECE typo fix and reverting the rest. If that works for you, I'll create an issue to follow up on the other changes right after 9.0 is out the door.
(And I'll open a separate PR for the troubleshooting-to-monitoring link and send it your way for review)
Thanks, @marciw ! Updated per feedback 🙏 |
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.
Thanks for scaling this down @stefnestor :)
Approving but it looks like there are some (unrelated) build failures ... those tend to get fixed quickly, so we can probably just keep an eye on this 🤞
oh didn't realize the branch needed updating; that might fix it |
👋 howdy, team!
This is a process test first PR to contribute to new docs content from Support 🙂. The main goal was to link Troubleshooting ES to Monitoring as a common Support recommendation to user, but then I got distracted fixing a typo and then cross-linking guides. So stopping randomly.
TIA! Stef