Skip to content

[8.19] [Security Solution][Detection Engine] removes FTR "deletes the underlying migration task" test (#233995)#234429

Closed
kibanamachine wants to merge 1 commit intoelastic:8.19from
kibanamachine:backport/8.19/pr-233995
Closed

[8.19] [Security Solution][Detection Engine] removes FTR "deletes the underlying migration task" test (#233995)#234429
kibanamachine wants to merge 1 commit intoelastic:8.19from
kibanamachine:backport/8.19/pr-233995

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.19:

Questions ?

Please refer to the Backport tool documentation

…ying migration task" test (elastic#233995)

## Summary

- addresses elastic#179593
- removes FTR "deletes the underlying migration task" test
- test was skipped from the moment it was introduced, 5 yers ago -
https://github.com/elastic/kibana/pull/85690/files#diff-d3ab7ab5d42b0e7dfe7a3162188693eb420f0a819444aa36380148f2a450a6dbR166-R185,
so was never functional
- While looked at code of [finalize migration
util](https://github.com/elastic/kibana/blob/main/x-pack/solutions/security/plugins/security_solution/server/lib/detection_engine/migrations/finalize_migration.ts),
I did not see any indications, it actually deletes underlying migration
task
- since this API is deprecated anyway, the most reasonable approach
seems just to remove that test

(cherry picked from commit 980469a)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Sep 9, 2025
@kibanamachine kibanamachine enabled auto-merge (squash) September 9, 2025 12:10
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

cc @vitaliidm

@vitaliidm vitaliidm closed this Sep 10, 2025
auto-merge was automatically disabled September 10, 2025 09:02

Pull request was closed

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants