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

[8.x] [APM] Show all alerts in service views & service inventory for consistency with entity inventory (#202025) #202347

Merged
merged 1 commit into from
Nov 29, 2024

Conversation

kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

…stency with entity inventory (elastic#202025)

## Summary

Closes elastic#201565

This update ensures that the service inventory and service views display
all alerts grouped by `service.name`, regardless of index pattern. This
provides consistent alert visibility across services.

>[!NOTE]
>The presence of two services with alerts in the "before" state, rather
than one, is not due to any changes made in this PR, but rather because
the alerts stopped being active while doing the screenshots.

Screen|Before|After
-|-|-
Entity inventory|![Screenshot 2024-11-27 at 17 04
01](https://github.com/user-attachments/assets/5ec7aa5a-0872-43bb-b4ae-1dcc68ce8e34)|![Screenshot
2024-11-27 at 17 13
58](https://github.com/user-attachments/assets/382ce562-d81b-4cc9-8869-8cf88597b231)
Service inventory|![Screenshot 2024-11-27 at 17 04
54](https://github.com/user-attachments/assets/6a4ff0b0-839f-4004-b240-13e8ccd804a3)|![Screenshot
2024-11-27 at 17 14
14](https://github.com/user-attachments/assets/62a28c6b-be3e-4ff2-bb95-29bc747db83e)
Service detail overview|![Screenshot 2024-11-27 at 17 04
34](https://github.com/user-attachments/assets/53129b4b-cdc6-4bb1-849a-401b82c57327)|![Screenshot
2024-11-27 at 17 05
27](https://github.com/user-attachments/assets/17ee297f-7b6a-46f1-bf98-b38593fa2abe)
Service detail alerts|![Screenshot 2024-11-27 at 17 04
45](https://github.com/user-attachments/assets/ae4ff74e-7d14-48cf-b1f0-76cc3f9e9e1c)|![Screenshot
2024-11-27 at 17 05
49](https://github.com/user-attachments/assets/507d7d6c-0ac8-455b-8aa6-c0ce8f1c756c)

(cherry picked from commit 8084bd6)
@elasticmachine
Copy link
Contributor

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

Copy link
Contributor

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@kibanamachine kibanamachine merged commit 8963e68 into elastic:8.x Nov 29, 2024
15 checks passed
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
apm 1881 1882 +1

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
apm 3.5MB 3.5MB +51.0B
inventory 236.6KB 236.4KB -153.0B
total -102.0B

cc @iblancof

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport ci:project-deploy-observability Create an Observability project Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants