Skip to content

fix(slo): alert deletion#219876

Merged
kdelemme merged 1 commit intoelastic:mainfrom
kdelemme:fix/bulk-delete-alert
May 1, 2025
Merged

fix(slo): alert deletion#219876
kdelemme merged 1 commit intoelastic:mainfrom
kdelemme:fix/bulk-delete-alert

Conversation

@kdelemme
Copy link
Copy Markdown
Contributor

@kdelemme kdelemme commented May 1, 2025

Summary

Properly delete the rules when bulk deleting SLOs.

@kdelemme kdelemme requested a review from a team as a code owner May 1, 2025 19:02
@botelastic botelastic bot added the Team:actionable-obs Formerly "obs-ux-management", responsible for SLO, o11y alerting, significant events, & synthetics. label May 1, 2025
@elasticmachine
Copy link
Copy Markdown
Contributor

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

@kdelemme kdelemme added release_note:skip Skip the PR/issue when compiling release notes backport:version Backport to applied version labels v9.1.0 v8.19.0 labels May 1, 2025
@prodsecmachine
Copy link
Copy Markdown
Collaborator

prodsecmachine commented May 1, 2025

🎉 Snyk checks have passed. No issues have been found so far.

security/snyk check is complete. No issues have been found. (View Details)

license/snyk check is complete. No issues have been found. (View Details)

@kdelemme kdelemme enabled auto-merge (squash) May 1, 2025 19:10
@kdelemme kdelemme merged commit cf7bd01 into elastic:main May 1, 2025
21 checks passed
@kibanamachine
Copy link
Copy Markdown
Contributor

Starting backport for target branches: 8.19

https://github.com/elastic/kibana/actions/runs/14783344410

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request May 1, 2025
## Summary

Properly delete the rules when bulk deleting SLOs.

(cherry picked from commit cf7bd01)
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 All backports created successfully

Status Branch Result
8.19

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kapral18 added a commit to kapral18/kibana that referenced this pull request May 4, 2025
…ends-crash

* main: (111 commits)
  [ResponseOps][Rules] Cases action title length too long (elastic#219226)
  [main] Sync bundled packages with Package Storage (elastic#219839)
  Fix ignored dynamic templates (elastic#219875)
  Enforce dependency review by kibana-security workflow (elastic#219262)
  [Security Solution] [Detections] Removes tech preview text from eql seq suppression ui (elastic#219870)
  [Security Solution] Fix alerts table potentially not applying alert assignees (elastic#219460)
  fix(slo): alert deletion (elastic#219876)
  [AI4DSOC] fix styling to address cutoff when screen is narrow (elastic#219306)
  [Security Solution][Endpoint] Response action create and history log API updates in of space awareness (elastic#218674)
  Update publish_oas_docs.sh to deploy Kibana Serverless API docs (elastic#219867)
  feat(slo): lock resource installation (elastic#219747)
  [AI4DSOC] Alert flyout code cleanup (elastic#219810)
  [fleet] fixing `isAgentlessDefault` config usage and readability improvements to `isAgentlessSetupDefault` (elastic#219423)
  feat(slo): Bulk delete UI (elastic#219634)
  m1 demo prep (elastic#219588)
  [Security Solution] Replace sourcerer in EQL tab with dataview picker (elastic#218897)
  [AI4DSOC] Attack discovery widget follow up follow up (elastic#219849)
  [AI Assistant] Fix some OpenAI models not accepting temperature for Inference service (elastic#218887)
  Update dependency msw to ~2.7.5 (main) (elastic#219289)
  Use new client URLs in doc link service (elastic#219600)
  ...
kibanamachine added a commit that referenced this pull request May 5, 2025
# Backport

This will backport the following commits from `main` to `8.19`:
- [fix(slo): alert deletion
(#219876)](#219876)

<!--- Backport version: 9.6.6 -->

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

<!--BACKPORT [{"author":{"name":"Kevin
Delemme","email":"kevin.delemme@elastic.co"},"sourceCommit":{"committedDate":"2025-05-01T20:53:48Z","message":"fix(slo):
alert deletion (#219876)\n\n## Summary\n\nProperly delete the rules when
bulk deleting
SLOs.","sha":"cf7bd011ba758a747bfcc88681cc4b8d0be852f6","branchLabelMapping":{"^v9.1.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:obs-ux-management","backport:version","v9.1.0","v8.19.0"],"title":"fix(slo):
alert
deletion","number":219876,"url":"https://github.com/elastic/kibana/pull/219876","mergeCommit":{"message":"fix(slo):
alert deletion (#219876)\n\n## Summary\n\nProperly delete the rules when
bulk deleting
SLOs.","sha":"cf7bd011ba758a747bfcc88681cc4b8d0be852f6"}},"sourceBranch":"main","suggestedTargetBranches":["8.19"],"targetPullRequestStates":[{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/219876","number":219876,"mergeCommit":{"message":"fix(slo):
alert deletion (#219876)\n\n## Summary\n\nProperly delete the rules when
bulk deleting
SLOs.","sha":"cf7bd011ba758a747bfcc88681cc4b8d0be852f6"}},{"branch":"8.19","label":"v8.19.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Kevin Delemme <kevin.delemme@elastic.co>
akowalska622 pushed a commit to akowalska622/kibana that referenced this pull request May 29, 2025
## Summary

Properly delete the rules when bulk deleting SLOs.
qn895 pushed a commit to qn895/kibana that referenced this pull request Jun 3, 2025
## Summary

Properly delete the rules when bulk deleting SLOs.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:version Backport to applied version labels release_note:skip Skip the PR/issue when compiling release notes Team:actionable-obs Formerly "obs-ux-management", responsible for SLO, o11y alerting, significant events, & synthetics. v8.19.0 v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants