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] [uptime] add {{state.firstTriggeredAt}} timezone support #90568

Closed
jagjag opened this issue Feb 8, 2021 · 8 comments
Closed

[alerting] [uptime] add {{state.firstTriggeredAt}} timezone support #90568

jagjag opened this issue Feb 8, 2021 · 8 comments
Labels
💝community estimate:needs-research Estimated as too large and requires research to break down into workable issues Feature:Alerting/RuleActions Issues related to the Actions attached to Rules on the Alerting Framework Feature:Alerting NeededFor:Uptime Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability UX

Comments

@jagjag
Copy link

jagjag commented Feb 8, 2021

Describe the feature:
hi, the alerting message template variant {{state.firstTriggeredAt}} and {{state.lastTriggeredAt}} currently recorded in UTC time zone.
please add a option to convert to local time zone

Describe a specific use case for the feature:
the alerting message triggered time is different from the local timezone which makes people confused about the exact date time the alert generated.

@jagjag jagjag changed the title uptime and alerting add {{state.firstTriggeredAt}} timezone support [alerting] [uptime] add {{state.firstTriggeredAt}} timezone support Feb 9, 2021
@nickpeihl nickpeihl added the Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) label Feb 12, 2021
@elasticmachine
Copy link
Contributor

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

@pmuellr
Copy link
Member

pmuellr commented Feb 13, 2021

Thanks for feature request.

We have two related issues already open:

The first is a specific implementation request for more mustache helpers, that might be able to help with this.

The second is more general, and I think points to alerting needing to support painless and/or Kibana expressions to transform alert data into action parameters. Presumably, it would be general enough to cover this use case, but there's a lot of design and work to do here.

@mikecote mikecote added the Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability label Feb 16, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/uptime (Team:uptime)

@mikecote
Copy link
Contributor

Pinging Uptime for awareness seems like something the alerting framework should solve.

@shahzad31
Copy link
Contributor

@mikecote @pmuellr do we also have any plans to add more formatting options like styling bold/color, icons etc to messages? those are especially useful in terms of slack etc

@mikecote
Copy link
Contributor

@shahzad31 this would be done when moving text areas to EUI Markdown, we are tracking the Email one with #66723 but I don't recall issues for others (ex: slack).

@pmuellr
Copy link
Member

pmuellr commented Feb 18, 2021

The templates themselves can include format-specific markup, but there's not a ton you can do formatting-wise w/markdown. bold/italic works for slack and email, but they're different **bold**, __bold__, *italic* , _italic_ for markdown/email, *bold*, _italic_ for slack - the only common one there is italic :-(.

Eventually we should support "Block Kit" for Slack - issue #88832 - not clear how we would present this to users, editing that kind of JSON is the stuff of nightmares. But it's not hard to imagine that you could take a structure like that and also produce some semantically similar HTML, for emails.

@gmmorris gmmorris added Feature:Alerting Feature:Alerting/RuleActions Issues related to the Actions attached to Rules on the Alerting Framework NeededFor:Uptime 💝community labels Jul 2, 2021
@gmmorris gmmorris added the loe:needs-research This issue requires some research before it can be worked on or estimated label Jul 14, 2021
@gmmorris gmmorris added UX estimate:needs-research Estimated as too large and requires research to break down into workable issues labels Aug 16, 2021
@gmmorris gmmorris removed the loe:needs-research This issue requires some research before it can be worked on or estimated label Sep 2, 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
@mikecote
Copy link
Contributor

mikecote commented Aug 4, 2022

Closing in favour of #84217.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💝community estimate:needs-research Estimated as too large and requires research to break down into workable issues Feature:Alerting/RuleActions Issues related to the Actions attached to Rules on the Alerting Framework Feature:Alerting NeededFor:Uptime Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability UX
Projects
None yet
Development

No branches or pull requests

8 participants