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

[AO] Use kibana.alert.url instead of context.alertDetailsUrl #158359

Closed
maryam-saeidi opened this issue May 24, 2023 · 2 comments
Closed

[AO] Use kibana.alert.url instead of context.alertDetailsUrl #158359

maryam-saeidi opened this issue May 24, 2023 · 2 comments
Labels
Team: Actionable Observability - DEPRECATED For Observability Alerting and SLOs use "Team:obs-ux-management", for AIops "Team:obs-knowledge" v8.9.0

Comments

@maryam-saeidi
Copy link
Member

📝 Summary

Previously, we introduced context.alertDetailsUrl in observability rules to add the URL of observability alert details there. After adding this parameter, responseOps team added a new variable to the alert document called kibana.alert.url.
In this ticket, we need to use kibana.alert.url instead of context.alertDetailsUrl and think about how to deprecate already used context.alertDetailsUrl in APM latency actions

✅ Acceptance Criteria

  • Use kibana.alert.url instead of context.alertDetailsUrl
  • Come up with a plan about how to deprecate context.alertDetailsUrl in APM Latency
@maryam-saeidi maryam-saeidi added Team: Actionable Observability - DEPRECATED For Observability Alerting and SLOs use "Team:obs-ux-management", for AIops "Team:obs-knowledge" v8.9.0 labels May 24, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/actionable-observability (Team: Actionable Observability)

maryam-saeidi added a commit that referenced this issue May 26, 2023
Closes #156534

## Summary

This PR adds the alertDetailsUrl to the infra rules. The value of this
variable is a link to the `observability > alerts` page filtered for
this instance of alert.


![image](https://github.com/elastic/kibana/assets/12370520/409bea90-5d2b-4e60-ae4c-61223cccd41a)

Here is an example of this action variable:

|alertDetailsUrl as action variable|Result of action|
|---|---|

|![image](https://github.com/elastic/kibana/assets/12370520/4d920710-357a-4557-b871-36f0f8ecf0be)|![image](https://github.com/elastic/kibana/assets/12370520/4f800c6d-f15f-481e-b7fc-4f85aa1085a7)|

**Note**
- I will change this field to `kibana.alert.url` in another
[ticket](#158359)

## 🧪 How to test
- Ensure that `server.publicBaseUrl` is configured in kibana.dev.yml
- Create a metric threshold/inventory/logs rule and use the
`context.alertDetailsUrl` in action for this rule
- After an alert is triggered, open the link provided by alertDetailsUrl
and make sure that the alert is filtered correctly
- Check the time range, it should be set for 5 mins before the alert
start time

---------

Co-authored-by: kibanamachine <[email protected]>
Co-authored-by: Carlos Crespo <[email protected]>
@maryam-saeidi
Copy link
Member Author

We decided to set both kibana.alert.url and context.alertDetailsUrl for now (158391) and move to using kibana.alert.url when #158183 is implemented.

@maryam-saeidi maryam-saeidi closed this as not planned Won't fix, can't repro, duplicate, stale Jun 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team: Actionable Observability - DEPRECATED For Observability Alerting and SLOs use "Team:obs-ux-management", for AIops "Team:obs-knowledge" v8.9.0
Projects
None yet
Development

No branches or pull requests

2 participants