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][Discuss] meta issue on mustache templating and action parameter processing #79786

Open
pmuellr opened this issue Oct 6, 2020 · 2 comments
Labels
discuss Feature:Alerting Meta Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams)

Comments

@pmuellr
Copy link
Member

pmuellr commented Oct 6, 2020

Our current story of having replaceable parameters in action parameters via mustache templates is not living up to the needs of our customers. Collecting some of the related issues here, want to noodle on this for a bit and we can come up with a plan for improvements.

closed:

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

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

@pmuellr
Copy link
Member Author

pmuellr commented Nov 24, 2020

Looking at these with a view on priority:

#83780 - Ability to declare unescaped messageVariables

This one seems simple enough (haven't done a lot of research, but it's only UI, no server-side code), and has enough value I think we could try to squeeze this into 7.11.0

#66095 - allow alerts to create default action parameters

This one is probably going to be extremely useful, hopefully being able to provide a lot of the default templating we need across various actions, action groups, etc. I think the first iteration of this can be UI-side only, but long term we'll likely want this on the server as well.

Guessing it's too late at this stage for this to be useful in 7.11.0, as each alert will have to provide a function with default templates across many axes, for this to be useful. Feels like something we should get into master as soon as the 7.11 branch is cut though.

#84148 - should alert types have a top-level action param variable for conditions

I feel like this will have a lot of value as well, but also requires buy-in from all the alerts to be useful, and so is also not a great candidate for 7.11, but good one to start early for 7.12.

@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
discuss Feature:Alerting Meta Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams)
Projects
None yet
Development

No branches or pull requests

4 participants