Skip to content

Conversation

@shainaraskas
Copy link
Collaborator

@shainaraskas shainaraskas commented Mar 12, 2025

This PR:

  • reorganizes more of the monitoring section:

^ these three sections are reviewable

todo:

  • finish the overview
  • clean up monitoring orchestrators, kib task manager monitoring, and logging config
  • find a home for vcpu boosting

@shainaraskas shainaraskas changed the title drafty [D+M] Monitoring pt II Mar 12, 2025
@shainaraskas shainaraskas requested a review from eedugon March 17, 2025 20:57
@shainaraskas shainaraskas marked this pull request as ready for review March 17, 2025 20:57
@eedugon
Copy link
Contributor

eedugon commented Mar 18, 2025

Quick comment about this box in Access monitoring data in Kibana:

image

The links offered by the second bullet are not stack monitoring data, are about the out of the box monitoring that ECE performs (regardless of the stack monitoring that a user can configure in logs and metrics). The opposite explanation has been added in ECE docs here:

image

I think we have to clarify that stack monitoring in ECE allows users to send logs and metrics from deployments to deployments for monitoring purposes, but the platform also monitors all the deployments by default, offering also Proxy access logs (which are not offered by stack monitoring).

The platform / system implemented monitoring is described at /deploy-manage/monitor/orchestrators/ece-platform-monitoring.md and /deploy-manage/monitor/orchestrators/ece-monitoring-ece-access.md

Copy link
Contributor

@eedugon eedugon left a comment

Choose a reason for hiding this comment

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

I've added a few small and picky comments. The updated sections feel really great!!

For the Autoops refinement consider asking Arianna for a review, as she has been applying some changes requested by PMs, to ensure we don't undo anything by mistake :)

@shainaraskas
Copy link
Collaborator Author

@alaudazzi I took a quick cleanup pass of autoops - you can also see in the monitoring overview how we're positioning autoops in comparison to stack monitoring. let me know if you have any concerns!

@shainaraskas shainaraskas requested a review from alaudazzi March 18, 2025 15:57
@alaudazzi
Copy link
Contributor

@shainaraskas
In the AutoOps docset, it makes sense to combine the different views under the same TOC entry "Views" 👍

@alaudazzi
Copy link
Contributor

@shainaraskas @eedugon
FYI -- Today the PM has requested an update on the https://docs-v3-preview.elastic.dev/elastic/docs-content/pull/750/deploy-manage/monitor/autoops/ec-autoops-deployment-view page. I need to make this change in the Cloud repo as well, because the PM wants to link this page from a report to Gartner this Thursday.
Let me know if you want to first to merge this PR, or wait my quick update.

@shainaraskas
Copy link
Collaborator Author

@shainaraskas @eedugon FYI -- Today the PM has requested an update on the docs-v3-preview.elastic.dev/elastic/docs-content/pull/750/deploy-manage/monitor/autoops/ec-autoops-deployment-view page. I need to make this change in the Cloud repo as well, because the PM wants to link this page from a report to Gartner this Thursday. Let me know if you want to first to merge this PR, or wait my quick update.

@alaudazzi if you can do it today, go ahead. but otherwise my thing might get in first (I need to get the file moves merged) :)

@shainaraskas shainaraskas merged commit a360b15 into main Mar 18, 2025
4 checks passed
@shainaraskas shainaraskas deleted the monitoring-pt2 branch March 18, 2025 22:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants