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

Add docs for per-integration health reporting #263

Merged
merged 3 commits into from
Jun 14, 2023

Conversation

kilfoyle
Copy link
Contributor

@kilfoyle kilfoyle commented Jun 13, 2023

This updates the Monitor Elastic Agents docs page to include the new per-integration health reporting. I also added bit of detail to other sections, and renamed the sections so it's more clear what each section is about.

@jillguyonnet thanks for your really excellent recorded demo in the UI sync! Please let me know if I've missed anything. Also, if the UI has changed since that demo please let me know and I'll try to get a new screen capture for the docs.

Docs preview

Closes: #209

@kilfoyle kilfoyle marked this pull request as draft June 13, 2023 15:02
@kilfoyle kilfoyle requested a review from jillguyonnet June 14, 2023 13:54
@kilfoyle kilfoyle marked this pull request as ready for review June 14, 2023 13:54
@jillguyonnet
Copy link
Contributor

@kilfoyle The new section looks great, thank you! 🎉

I would suggest the below two screenshots instead of the current ones. The latter don't have the latest UX changes and there is an inconsistency in the Overview pane (agent status should be Unhealthy, this might be a consequence of the bug I was experiencing at the time of the demo). The new ones should also be of better quality.

Screenshot 2023-06-14 at 16 28 37 Screenshot 2023-06-14 at 16 35 54

@kilfoyle
Copy link
Contributor Author

Thanks @jillguyonnet! The new screencaps are perfect. I've added them in place of the originals. The changes will show up in the docs preview in about 20 minutes or so, after the elasticsearch-ci/docs job completes.

Copy link
Contributor

@jillguyonnet jillguyonnet left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@kilfoyle kilfoyle merged commit fcbe765 into elastic:main Jun 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Fleet] Document our per-integration health reporting
2 participants