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

[APM] Investigate alternative Alerting API approach #73148

Closed
sorenlouv opened this issue Jul 23, 2020 · 2 comments
Closed

[APM] Investigate alternative Alerting API approach #73148

sorenlouv opened this issue Jul 23, 2020 · 2 comments
Labels
Team:APM All issues that need APM UI Team support v7.11.0

Comments

@sorenlouv
Copy link
Member

We have created an issue where we outline some requirements for the alerting API.

We were hoping to have this available in 7.10 but it's been moved to "Long Term".
Instead an alternative approach has been suggested for the short term (7.10). We should investigate whether this workaround is sufficient for our use cases (Service Maps alert indicators and Observability Landing Page).
Important: we should not jump through hoops to make the integration work (we've incurred too much tech debt and introduced too many bugs doing this in the past). Either the API is applicable to the use case or it's not.

@sorenlouv sorenlouv added [zube]: (7.10) Planned for release Team:APM All issues that need APM UI Team support v7.10.0 labels Jul 23, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/apm-ui (Team:apm)

@sorenlouv sorenlouv changed the title [APM] Investigate Alerting API [APM] Investigate alternative Alerting API approach Jul 23, 2020
@ogupte
Copy link
Contributor

ogupte commented Jul 28, 2020

I believe the workaround that has been suggested was similar to the approach I mentioned in my comment. It works by fetching the task state for each matching alert returned in the find call. In setups with few alerts configured, it will add a few requests to the page load. But what if we're trying to load a service map with 10, 20, or more services where each has an alert configured? Are we OK adding an x number of single requests to our initial page load?

From the alerting plugin context, it might be possible to obtain multiple alerts states in a single request, but it would require querying the task manager index filtered by job ids obtained in the initial find. This would result in the initial page load adding a constant 2 additional requests instead of the suggested 1 + x requests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:APM All issues that need APM UI Team support v7.11.0
Projects
None yet
Development

No branches or pull requests

4 participants