Skip to content

[APM] Add ability to view, create & manage SLOs in Service Views#253285

Merged
rmyz merged 18 commits intoelastic:mainfrom
rmyz:5101-apm-service-overview-slos
Feb 20, 2026
Merged

[APM] Add ability to view, create & manage SLOs in Service Views#253285
rmyz merged 18 commits intoelastic:mainfrom
rmyz:5101-apm-service-overview-slos

Conversation

@rmyz
Copy link
Copy Markdown
Contributor

@rmyz rmyz commented Feb 16, 2026

Summary

This PR adds the ability to view, create & manage SLOs in Service Views.

Changes:

  • Fixed a bug where SLO and alert burn rules created with service.environment: * weren't retrieved with service.environment: <Whatever>.
  • Added an icon to the SLO status badge
  • Added status "NO SLO" for the status badge, which we will show in Service Inventory if other services have SLOs and in the Service Overview
  • Refactored code that was used in multiple places to use shared context/hooks
  • Improved SLO callout design in Service Overview
  • Added "Actions" contextual menu next to Environment selector on Service Overview
  • Added SLO Overview and SLO Create Flyouts to the Service Overview
  • Added Service header to show the count of alerts and highest critical SLO status
  • Removed the alert count in Alerts tab for the Service Overview
  • Add empty state for the SLO Overview flyout to be able to create an SLO

Demo

General Changes

Kapture.2026-02-19.at.11.54.28.mp4

Creation functionality inside Service Overview

Kapture.2026-02-19.at.11.59.25.mp4

@rmyz rmyz self-assigned this Feb 16, 2026
@rmyz rmyz added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting ci:beta-faster-pr-build Team:obs-presentation Focus: APM UI, Infra UI, Hosts UI, Universal Profiling, Obs Overview and left Navigation labels Feb 16, 2026
@rmyz rmyz changed the title wip slos in service overview [APM] Add ability to view, create & manage SLOs in Service Views Feb 17, 2026
@rmyz rmyz added the ci:project-deploy-observability Create an Observability project label Feb 17, 2026
@github-actions
Copy link
Copy Markdown
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!)

@rmyz
Copy link
Copy Markdown
Contributor Author

rmyz commented Feb 18, 2026

/ci

@rmyz
Copy link
Copy Markdown
Contributor Author

rmyz commented Feb 18, 2026

/ci

@rmyz
Copy link
Copy Markdown
Contributor Author

rmyz commented Feb 18, 2026

/ci

@rmyz
Copy link
Copy Markdown
Contributor Author

rmyz commented Feb 19, 2026

/ci

@rmyz
Copy link
Copy Markdown
Contributor Author

rmyz commented Feb 19, 2026

/ci

@rmyz
Copy link
Copy Markdown
Contributor Author

rmyz commented Feb 19, 2026

/ci

@rmyz rmyz marked this pull request as ready for review February 19, 2026 13:03
@rmyz rmyz requested review from a team as code owners February 19, 2026 13:03
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/obs-presentation-team (Team:obs-presentation)

Copy link
Copy Markdown
Contributor

@kdelemme kdelemme left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

actionable-obs code change lgtm

@elasticmachine
Copy link
Copy Markdown
Contributor

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

rmyz added 3 commits February 20, 2026 10:34
# Conflicts:
#	x-pack/solutions/observability/plugins/apm/public/components/routing/templates/apm_service_template/index.tsx
aria-labelledby={flyoutTitleId}
// we need this hardcoded size as S is too small and M is too large
size="550px"
size={620}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NIT: as the size has change might now M work?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sadly, we have to hardcode this, the sizes used in most flyouts is hardcoded to 620, and this lives in different plugins (alerts, SLO, APM...) so let's keep it for now as it is and try to refactor once we have more guidelines from the flyout working group

Copy link
Copy Markdown
Contributor

@MiriamAparicio MiriamAparicio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code only review LGTM

@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Feb 20, 2026

💚 Build Succeeded

  • Buildkite Build
  • Commit: 837fd53
  • Kibana Serverless Image: docker.elastic.co/kibana-ci/kibana-serverless:pr-253285-837fd536e65a

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
apm 2208 2215 +7

Async chunks

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

id before after diff
apm 2.7MB 2.7MB +12.3KB
slo 1.1MB 1.1MB +16.0B
total +12.3KB

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
apm 22.5KB 22.5KB -10.0B
Unknown metric groups

async chunk count

id before after diff
apm 88 87 -1

History

cc @rmyz

@rmyz rmyz merged commit 2e4cfb9 into elastic:main Feb 20, 2026
16 checks passed
@rmyz rmyz deleted the 5101-apm-service-overview-slos branch February 20, 2026 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting ci:project-deploy-observability Create an Observability project release_note:skip Skip the PR/issue when compiling release notes Team:obs-presentation Focus: APM UI, Infra UI, Hosts UI, Universal Profiling, Obs Overview and left Navigation Team:obs-ux-management v9.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants