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

[Logs UI] Assess "noise" from the /summary endpoint #152434

Open
Kerry350 opened this issue Mar 1, 2023 · 3 comments
Open

[Logs UI] Assess "noise" from the /summary endpoint #152434

Kerry350 opened this issue Mar 1, 2023 · 3 comments
Labels
Feature:Logs UI Logs UI feature Team:obs-ux-logs Observability Logs User Experience Team

Comments

@Kerry350
Copy link
Contributor

Kerry350 commented Mar 1, 2023

Summary

The /api/log_entries/summary API endpoint frequently results in a 502 response (BACKEND_CLOSED_CONNECTION), most likely timeouts communicating with Elasticsearch. When looking at a specific case of this the cluster was under-resourced, which aligns with that theory. The issue is the frequency of these responses, which is polluting certain statistics.

Whilst the API could possibly use a more accurate status code than 502 the response behaviour isn't necessarily broken.

What we should investigate is why this endpoint differently to others; is there broken retry logic? Or cancellation logic? (etc).

@Kerry350 Kerry350 added Feature:Logs UI Logs UI feature Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services labels Mar 1, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/infra-monitoring-ui (Team:Infra Monitoring UI)

@weltenwort
Copy link
Member

💭 Could this be related to #148670? If the summary was requested often via sync search it could both cause cluster load but also be negatively affected in a feedback loop. Would be interesting to see if the noise subsides with 8.6.1 and 8.7.0. Changing this to use async search could improve it further.

@gbamparop gbamparop added Team:obs-ux-logs Observability Logs User Experience Team and removed Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services labels Nov 9, 2023
@botelastic botelastic bot added the needs-team Issues missing a team label label Nov 9, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-logs-team (Team:obs-ux-logs)

@botelastic botelastic bot removed the needs-team Issues missing a team label label Nov 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Logs UI Logs UI feature Team:obs-ux-logs Observability Logs User Experience Team
Projects
None yet
Development

No branches or pull requests

4 participants