Skip to content

[8.19] [SLO] Check for unique SLO ids across spaces (#214496)#220267

Merged
baileycash-elastic merged 5 commits intoelastic:8.19from
baileycash-elastic:backport/8.19/pr-214496
May 7, 2025
Merged

[8.19] [SLO] Check for unique SLO ids across spaces (#214496)#220267
baileycash-elastic merged 5 commits intoelastic:8.19from
baileycash-elastic:backport/8.19/pr-214496

Conversation

@baileycash-elastic
Copy link
Contributor

Backport

This will backport the following commits from main to 8.19:

Questions ?

Please refer to the Backport tool documentation

## Summary
Resolves elastic#212784
Ensure that when an SLO is created, the id is verified across all
spaces.

## Release Notes
Ensure that when an SLO is created, the id is verified across all
spaces.

## Testing
1. Create an SLO and save the id returned in the response in a space "A"
2. Create a second SLO with the id saved from the first SLO in the
request in a different space "B"
3. User should receive a 409 error from the SLO API.

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
(cherry picked from commit 56f1ebf)

# Conflicts:
#	x-pack/solutions/observability/plugins/slo/server/routes/slo/create_slo.ts
#	x-pack/solutions/observability/plugins/slo/server/routes/slo/inspect_slo.ts
#	x-pack/solutions/observability/plugins/slo/server/services/create_slo.test.ts
#	x-pack/solutions/observability/plugins/slo/server/services/create_slo.ts
@baileycash-elastic baileycash-elastic added the backport This PR is a backport of another PR label May 6, 2025
@baileycash-elastic baileycash-elastic enabled auto-merge (squash) May 6, 2025 15:27
@botelastic botelastic bot added the Team:actionable-obs Formerly "obs-ux-management", responsible for SLO, o11y alerting, significant events, & synthetics. label May 6, 2025
@elasticmachine
Copy link
Contributor

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

@elasticmachine
Copy link
Contributor

elasticmachine commented May 6, 2025

💔 Build Failed

Failed CI Steps

History

@baileycash-elastic baileycash-elastic merged commit 86e5923 into elastic:8.19 May 7, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR Team:actionable-obs Formerly "obs-ux-management", responsible for SLO, o11y alerting, significant events, & synthetics.

4 participants