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

[SLOs] [Synthetics availability] Link location from SLO overview #178138

Closed
dominiqueclarke opened this issue Mar 6, 2024 · 1 comment
Closed
Assignees
Labels
Feature:SLO Team:obs-ux-management Observability Management User Experience Team

Comments

@dominiqueclarke
Copy link
Contributor

dominiqueclarke commented Mar 6, 2024

Kibana version: 8.14.0

Describe the feature:

Synthetics Availability SLOs are automatically grouped by name and location.

From the SLO details page, when displaying the name of the monitor we link back to the individual monitor's detail page.

It would be great to hyperlink the location name as well, and navigate to the Synthetics overview page filtered for that location.

To do so, a new overview locator will need to be added to the Synthetics app, to be consumed by SLOs

@dominiqueclarke dominiqueclarke added Feature:SLO Team:obs-ux-management Observability Management User Experience Team v8.14.0 labels Mar 6, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-management-team (Team:obs-ux-management)

@baileycash-elastic baileycash-elastic self-assigned this Feb 10, 2025
baileycash-elastic added a commit that referenced this issue Feb 12, 2025
## Summary

- Implements #178138 
- Fixes a bug where clicking on the existing link to the monitor via the
name badge led failed to load any data.

## Release Notes
- Fixes a bug where clicking on the name badge for a synthetics monitor
on an SLO details page would lead to a page that failed to load monitor
details.
- Adds a working link to the location badge on synthetics SLOs that will
route the user to the monitors page with a filter applied that matches
the location of the origin SLO.

![Screenshot 2025-02-11 at 3 31
15 PM](https://github.com/user-attachments/assets/1df39069-fc42-4c33-a7e5-8395b2730f43)
![Screenshot 2025-02-11 at 3 31
34 PM](https://github.com/user-attachments/assets/f1b3180f-eb9c-4f3b-9ff6-66bd4d1f8d5b)
kibanamachine pushed a commit to kibanamachine/kibana that referenced this issue Feb 12, 2025
## Summary

- Implements elastic#178138
- Fixes a bug where clicking on the existing link to the monitor via the
name badge led failed to load any data.

## Release Notes
- Fixes a bug where clicking on the name badge for a synthetics monitor
on an SLO details page would lead to a page that failed to load monitor
details.
- Adds a working link to the location badge on synthetics SLOs that will
route the user to the monitors page with a filter applied that matches
the location of the origin SLO.

![Screenshot 2025-02-11 at 3 31
15 PM](https://github.com/user-attachments/assets/1df39069-fc42-4c33-a7e5-8395b2730f43)
![Screenshot 2025-02-11 at 3 31
34 PM](https://github.com/user-attachments/assets/f1b3180f-eb9c-4f3b-9ff6-66bd4d1f8d5b)

(cherry picked from commit 4d3cf33)
kibanamachine added a commit that referenced this issue Feb 18, 2025
#210930)

# Backport

This will backport the following commits from `main` to `9.0`:
- [Add location, fix name badge links for Synthetics SLOs
(#210695)](#210695)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Bailey
Cash","email":"bailey.cash@elastic.co"},"sourceCommit":{"committedDate":"2025-02-12T20:08:13Z","message":"Add
location, fix name badge links for Synthetics SLOs (#210695)\n\n##
Summary\r\n\r\n- Implements #178138 \r\n- Fixes a bug where clicking on
the existing link to the monitor via the\r\nname badge led failed to
load any data.\r\n\r\n## Release Notes\r\n- Fixes a bug where clicking
on the name badge for a synthetics monitor\r\non an SLO details page
would lead to a page that failed to load monitor\r\ndetails.\r\n- Adds a
working link to the location badge on synthetics SLOs that will\r\nroute
the user to the monitors page with a filter applied that matches\r\nthe
location of the origin SLO.\r\n\r\n![Screenshot 2025-02-11 at 3
31\r\n15 PM](https://github.com/user-attachments/assets/1df39069-fc42-4c33-a7e5-8395b2730f43)\r\n![Screenshot
2025-02-11 at 3
31\r\n34 PM](https://github.com/user-attachments/assets/f1b3180f-eb9c-4f3b-9ff6-66bd4d1f8d5b)","sha":"4d3cf33ffd6d561cb73c1bccd53c643f3846cc1e","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:enhancement","release_note:fix","v9.0.0","backport:prev-minor","Team:obs-ux-management","v9.1.0"],"title":"Add
location, fix name badge links for Synthetics
SLOs","number":210695,"url":"https://github.com/elastic/kibana/pull/210695","mergeCommit":{"message":"Add
location, fix name badge links for Synthetics SLOs (#210695)\n\n##
Summary\r\n\r\n- Implements #178138 \r\n- Fixes a bug where clicking on
the existing link to the monitor via the\r\nname badge led failed to
load any data.\r\n\r\n## Release Notes\r\n- Fixes a bug where clicking
on the name badge for a synthetics monitor\r\non an SLO details page
would lead to a page that failed to load monitor\r\ndetails.\r\n- Adds a
working link to the location badge on synthetics SLOs that will\r\nroute
the user to the monitors page with a filter applied that matches\r\nthe
location of the origin SLO.\r\n\r\n![Screenshot 2025-02-11 at 3
31\r\n15 PM](https://github.com/user-attachments/assets/1df39069-fc42-4c33-a7e5-8395b2730f43)\r\n![Screenshot
2025-02-11 at 3
31\r\n34 PM](https://github.com/user-attachments/assets/f1b3180f-eb9c-4f3b-9ff6-66bd4d1f8d5b)","sha":"4d3cf33ffd6d561cb73c1bccd53c643f3846cc1e"}},"sourceBranch":"main","suggestedTargetBranches":["9.0"],"targetPullRequestStates":[{"branch":"9.0","label":"v9.0.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/210695","number":210695,"mergeCommit":{"message":"Add
location, fix name badge links for Synthetics SLOs (#210695)\n\n##
Summary\r\n\r\n- Implements #178138 \r\n- Fixes a bug where clicking on
the existing link to the monitor via the\r\nname badge led failed to
load any data.\r\n\r\n## Release Notes\r\n- Fixes a bug where clicking
on the name badge for a synthetics monitor\r\non an SLO details page
would lead to a page that failed to load monitor\r\ndetails.\r\n- Adds a
working link to the location badge on synthetics SLOs that will\r\nroute
the user to the monitors page with a filter applied that matches\r\nthe
location of the origin SLO.\r\n\r\n![Screenshot 2025-02-11 at 3
31\r\n15 PM](https://github.com/user-attachments/assets/1df39069-fc42-4c33-a7e5-8395b2730f43)\r\n![Screenshot
2025-02-11 at 3
31\r\n34 PM](https://github.com/user-attachments/assets/f1b3180f-eb9c-4f3b-9ff6-66bd4d1f8d5b)","sha":"4d3cf33ffd6d561cb73c1bccd53c643f3846cc1e"}}]}]
BACKPORT-->

Co-authored-by: Bailey Cash <bailey.cash@elastic.co>
Co-authored-by: Shahzad <shahzad31comp@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:SLO Team:obs-ux-management Observability Management User Experience Team
Projects
None yet
Development

No branches or pull requests

4 participants