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

[alerting]: research generic way of handling background tasks like requests #55648

Closed
pmuellr opened this issue Jan 23, 2020 · 3 comments
Closed
Labels
Feature:Alerting Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams)

Comments

@pmuellr
Copy link
Member

pmuellr commented Jan 23, 2020

Most Kibana plugins use platform-y app-architecty-y ways of accessing things like ES. Eg, http request handlers are automagically scoped to the user that made the request, in terms of ES, SO access, etc. There's some NP "context" stuff to deal with this.

Alerting and actions have a problem though - they run things in the background, not scoped to an http request, but to an API key bound to the alert/action. We've already handled existing cases like callCluster, internally, but we recently got a request for adding support for UISettings (and/or it's NP equivalent) in alert executors: #55052

So, it's easy to add one-off things like UISettiings support to alert/action executors, as another property an alert/action function can use, the question is whether there is a more general need for more support for similar services, and how we can accommodate that. Eg, what happens when we need 10 more of these user-scoped services?

@pmuellr pmuellr added Feature:Alerting Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) labels Jan 23, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-alerting-services (Team:Alerting Services)

@pmuellr
Copy link
Member Author

pmuellr commented Jan 27, 2020

some related discussion in issue #39430

@ymao1
Copy link
Contributor

ymao1 commented Apr 23, 2021

Closing in favor of #56773

@ymao1 ymao1 closed this as completed Apr 23, 2021
@kobelb kobelb added the needs-team Issues missing a team label label Jan 31, 2022
@botelastic botelastic bot removed the needs-team Issues missing a team label label Jan 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Alerting Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams)
Projects
None yet
Development

No branches or pull requests

4 participants