-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Rules migration][Integration test] Get migration rules API (#11232) #210867
[Rules migration][Integration test] Get migration rules API (#11232) #210867
Conversation
Pinging @elastic/security-threat-hunting (Team:Threat Hunting) |
Pinging @elastic/security-solution (Team: SecuritySolution) |
💚 Build Succeeded
Metrics [docs]
cc @e40pud |
get: async ({ | ||
migrationId, | ||
page, | ||
perPage, | ||
sortField, | ||
sortDirection, | ||
filters, | ||
expectStatusCode = 200, | ||
}: GetRuleMigrationParams): Promise<{ body: GetRuleMigrationResponse }> => { | ||
const response = await supertest | ||
.get(replaceParams(SIEM_RULE_MIGRATION_PATH, { migration_id: migrationId })) | ||
.query({ | ||
page, | ||
per_page: perPage, | ||
sort_field: sortField, | ||
sort_direction: sortDirection, | ||
search_term: filters?.searchTerm, | ||
ids: filters?.ids, | ||
is_prebuilt: filters?.prebuilt, | ||
is_installed: filters?.installed, | ||
is_fully_translated: filters?.fullyTranslated, | ||
is_partially_translated: filters?.partiallyTranslated, | ||
is_untranslatable: filters?.untranslatable, | ||
is_failed: filters?.failed, | ||
}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
NIT: Consider using the real parameter format from the GetRuleMigrationRequestQuery
type instead of doing this conversion.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds great! I will update it in my next PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
addressed in #210999
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome! LGTM
Starting backport for target branches: 8.18, 8.x, 9.0 |
…11232) (elastic#210867) ## Summary [Internal link](elastic/security-team#10820) to the feature details Part of elastic/security-team#11232 This PR covers SIEM Migrations GET API (route: `/internal/siem_migrations/rules/{migration_id}`) integration test: * Basic calls * Filtering * Sorting * Pagination (cherry picked from commit 526ff05)
…11232) (elastic#210867) ## Summary [Internal link](elastic/security-team#10820) to the feature details Part of elastic/security-team#11232 This PR covers SIEM Migrations GET API (route: `/internal/siem_migrations/rules/{migration_id}`) integration test: * Basic calls * Filtering * Sorting * Pagination (cherry picked from commit 526ff05)
…11232) (elastic#210867) ## Summary [Internal link](elastic/security-team#10820) to the feature details Part of elastic/security-team#11232 This PR covers SIEM Migrations GET API (route: `/internal/siem_migrations/rules/{migration_id}`) integration test: * Basic calls * Filtering * Sorting * Pagination (cherry picked from commit 526ff05)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
…1232) (#210867) (#210998) # Backport This will backport the following commits from `main` to `9.0`: - [[Rules migration][Integration test] Get migration rules API (#11232) (#210867)](#210867) <!--- Backport version: 9.4.3 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Ievgen Sorokopud","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-02-13T12:44:08Z","message":"[Rules migration][Integration test] Get migration rules API (#11232) (#210867)\n\n## Summary\r\n\r\n[Internal link](https://github.com/elastic/security-team/issues/10820)\r\nto the feature details\r\n\r\nPart of https://github.com/elastic/security-team/issues/11232\r\n\r\nThis PR covers SIEM Migrations GET API (route:\r\n`/internal/siem_migrations/rules/{migration_id}`) integration test:\r\n* Basic calls\r\n* Filtering\r\n* Sorting\r\n* Pagination","sha":"526ff0516e701ee1e1db8a248fe8f7c86587d257","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:Threat Hunting","Team: SecuritySolution","backport:version","v8.18.0","v9.1.0","v8.19.0"],"title":"[Rules migration][Integration test] Get migration rules API (#11232)","number":210867,"url":"https://github.com/elastic/kibana/pull/210867","mergeCommit":{"message":"[Rules migration][Integration test] Get migration rules API (#11232) (#210867)\n\n## Summary\r\n\r\n[Internal link](https://github.com/elastic/security-team/issues/10820)\r\nto the feature details\r\n\r\nPart of https://github.com/elastic/security-team/issues/11232\r\n\r\nThis PR covers SIEM Migrations GET API (route:\r\n`/internal/siem_migrations/rules/{migration_id}`) integration test:\r\n* Basic calls\r\n* Filtering\r\n* Sorting\r\n* Pagination","sha":"526ff0516e701ee1e1db8a248fe8f7c86587d257"}},"sourceBranch":"main","suggestedTargetBranches":["9.0","8.18","8.x"],"targetPullRequestStates":[{"branch":"9.0","label":"v9.0.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.18","label":"v8.18.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/210867","number":210867,"mergeCommit":{"message":"[Rules migration][Integration test] Get migration rules API (#11232) (#210867)\n\n## Summary\r\n\r\n[Internal link](https://github.com/elastic/security-team/issues/10820)\r\nto the feature details\r\n\r\nPart of https://github.com/elastic/security-team/issues/11232\r\n\r\nThis PR covers SIEM Migrations GET API (route:\r\n`/internal/siem_migrations/rules/{migration_id}`) integration test:\r\n* Basic calls\r\n* Filtering\r\n* Sorting\r\n* Pagination","sha":"526ff0516e701ee1e1db8a248fe8f7c86587d257"}},{"branch":"8.x","label":"v8.19.0","branchLabelMappingKey":"^v8.19.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}] BACKPORT--> Co-authored-by: Ievgen Sorokopud <[email protected]>
…11232) (#210867) (#210996) # Backport This will backport the following commits from `main` to `8.18`: - [[Rules migration][Integration test] Get migration rules API (#11232) (#210867)](#210867) <!--- Backport version: 9.4.3 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Ievgen Sorokopud","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-02-13T12:44:08Z","message":"[Rules migration][Integration test] Get migration rules API (#11232) (#210867)\n\n## Summary\r\n\r\n[Internal link](https://github.com/elastic/security-team/issues/10820)\r\nto the feature details\r\n\r\nPart of https://github.com/elastic/security-team/issues/11232\r\n\r\nThis PR covers SIEM Migrations GET API (route:\r\n`/internal/siem_migrations/rules/{migration_id}`) integration test:\r\n* Basic calls\r\n* Filtering\r\n* Sorting\r\n* Pagination","sha":"526ff0516e701ee1e1db8a248fe8f7c86587d257","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:Threat Hunting","Team: SecuritySolution","backport:version","v8.18.0","v9.1.0","v8.19.0"],"title":"[Rules migration][Integration test] Get migration rules API (#11232)","number":210867,"url":"https://github.com/elastic/kibana/pull/210867","mergeCommit":{"message":"[Rules migration][Integration test] Get migration rules API (#11232) (#210867)\n\n## Summary\r\n\r\n[Internal link](https://github.com/elastic/security-team/issues/10820)\r\nto the feature details\r\n\r\nPart of https://github.com/elastic/security-team/issues/11232\r\n\r\nThis PR covers SIEM Migrations GET API (route:\r\n`/internal/siem_migrations/rules/{migration_id}`) integration test:\r\n* Basic calls\r\n* Filtering\r\n* Sorting\r\n* Pagination","sha":"526ff0516e701ee1e1db8a248fe8f7c86587d257"}},"sourceBranch":"main","suggestedTargetBranches":["9.0","8.18","8.x"],"targetPullRequestStates":[{"branch":"9.0","label":"v9.0.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.18","label":"v8.18.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/210867","number":210867,"mergeCommit":{"message":"[Rules migration][Integration test] Get migration rules API (#11232) (#210867)\n\n## Summary\r\n\r\n[Internal link](https://github.com/elastic/security-team/issues/10820)\r\nto the feature details\r\n\r\nPart of https://github.com/elastic/security-team/issues/11232\r\n\r\nThis PR covers SIEM Migrations GET API (route:\r\n`/internal/siem_migrations/rules/{migration_id}`) integration test:\r\n* Basic calls\r\n* Filtering\r\n* Sorting\r\n* Pagination","sha":"526ff0516e701ee1e1db8a248fe8f7c86587d257"}},{"branch":"8.x","label":"v8.19.0","branchLabelMappingKey":"^v8.19.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}] BACKPORT--> Co-authored-by: Ievgen Sorokopud <[email protected]>
…1232) (#210867) (#210997) # Backport This will backport the following commits from `main` to `8.x`: - [[Rules migration][Integration test] Get migration rules API (#11232) (#210867)](#210867) <!--- Backport version: 9.4.3 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Ievgen Sorokopud","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-02-13T12:44:08Z","message":"[Rules migration][Integration test] Get migration rules API (#11232) (#210867)\n\n## Summary\r\n\r\n[Internal link](https://github.com/elastic/security-team/issues/10820)\r\nto the feature details\r\n\r\nPart of https://github.com/elastic/security-team/issues/11232\r\n\r\nThis PR covers SIEM Migrations GET API (route:\r\n`/internal/siem_migrations/rules/{migration_id}`) integration test:\r\n* Basic calls\r\n* Filtering\r\n* Sorting\r\n* Pagination","sha":"526ff0516e701ee1e1db8a248fe8f7c86587d257","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:Threat Hunting","Team: SecuritySolution","backport:version","v8.18.0","v9.1.0","v8.19.0"],"title":"[Rules migration][Integration test] Get migration rules API (#11232)","number":210867,"url":"https://github.com/elastic/kibana/pull/210867","mergeCommit":{"message":"[Rules migration][Integration test] Get migration rules API (#11232) (#210867)\n\n## Summary\r\n\r\n[Internal link](https://github.com/elastic/security-team/issues/10820)\r\nto the feature details\r\n\r\nPart of https://github.com/elastic/security-team/issues/11232\r\n\r\nThis PR covers SIEM Migrations GET API (route:\r\n`/internal/siem_migrations/rules/{migration_id}`) integration test:\r\n* Basic calls\r\n* Filtering\r\n* Sorting\r\n* Pagination","sha":"526ff0516e701ee1e1db8a248fe8f7c86587d257"}},"sourceBranch":"main","suggestedTargetBranches":["9.0","8.18","8.x"],"targetPullRequestStates":[{"branch":"9.0","label":"v9.0.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.18","label":"v8.18.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/210867","number":210867,"mergeCommit":{"message":"[Rules migration][Integration test] Get migration rules API (#11232) (#210867)\n\n## Summary\r\n\r\n[Internal link](https://github.com/elastic/security-team/issues/10820)\r\nto the feature details\r\n\r\nPart of https://github.com/elastic/security-team/issues/11232\r\n\r\nThis PR covers SIEM Migrations GET API (route:\r\n`/internal/siem_migrations/rules/{migration_id}`) integration test:\r\n* Basic calls\r\n* Filtering\r\n* Sorting\r\n* Pagination","sha":"526ff0516e701ee1e1db8a248fe8f7c86587d257"}},{"branch":"8.x","label":"v8.19.0","branchLabelMappingKey":"^v8.19.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}] BACKPORT--> Co-authored-by: Ievgen Sorokopud <[email protected]>
…10999) ## Summary [Internal link](elastic/security-team#10820) to the feature details Part of elastic/security-team#11232 This PR covers SIEM Migrations CREATE API (route: `/internal/siem_migrations/rules/{migration_id?}`) integration test: * Create migration with provided ID * Create migration without provided ID * Create migration with rules that have resources * Error handling ("no content") when no rules provided Also, as part of this PR, I addressed this comment to my previous changes #210867 (comment)
…) (elastic#210999) ## Summary [Internal link](elastic/security-team#10820) to the feature details Part of elastic/security-team#11232 This PR covers SIEM Migrations CREATE API (route: `/internal/siem_migrations/rules/{migration_id?}`) integration test: * Create migration with provided ID * Create migration without provided ID * Create migration with rules that have resources * Error handling ("no content") when no rules provided Also, as part of this PR, I addressed this comment to my previous changes elastic#210867 (comment) (cherry picked from commit 44fdf81)
…) (elastic#210999) ## Summary [Internal link](elastic/security-team#10820) to the feature details Part of elastic/security-team#11232 This PR covers SIEM Migrations CREATE API (route: `/internal/siem_migrations/rules/{migration_id?}`) integration test: * Create migration with provided ID * Create migration without provided ID * Create migration with rules that have resources * Error handling ("no content") when no rules provided Also, as part of this PR, I addressed this comment to my previous changes elastic#210867 (comment) (cherry picked from commit 44fdf81)
…) (elastic#210999) ## Summary [Internal link](elastic/security-team#10820) to the feature details Part of elastic/security-team#11232 This PR covers SIEM Migrations CREATE API (route: `/internal/siem_migrations/rules/{migration_id?}`) integration test: * Create migration with provided ID * Create migration without provided ID * Create migration with rules that have resources * Error handling ("no content") when no rules provided Also, as part of this PR, I addressed this comment to my previous changes elastic#210867 (comment) (cherry picked from commit 44fdf81)
…) (#210999) (#211083) # Backport This will backport the following commits from `main` to `8.18`: - [[Rules migration][Integration test] Create migration API (#11232) (#210999)](#210999) <!--- Backport version: 9.4.3 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Ievgen Sorokopud","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-02-13T19:20:06Z","message":"[Rules migration][Integration test] Create migration API (#11232) (#210999)\n\n## Summary\r\n\r\n[Internal link](https://github.com/elastic/security-team/issues/10820)\r\nto the feature details\r\n\r\nPart of https://github.com/elastic/security-team/issues/11232\r\n\r\nThis PR covers SIEM Migrations CREATE API (route:\r\n`/internal/siem_migrations/rules/{migration_id?}`) integration test:\r\n* Create migration with provided ID\r\n* Create migration without provided ID\r\n* Create migration with rules that have resources\r\n* Error handling (\"no content\") when no rules provided\r\n\r\nAlso, as part of this PR, I addressed this comment to my previous\r\nchanges\r\nhttps://github.com//pull/210867#discussion_r1954344990","sha":"44fdf81bbec5c053ebccfbdbb062d39de0d7358c","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:Threat Hunting","Team: SecuritySolution","backport:version","v8.18.0","v9.1.0","v8.19.0"],"title":"[Rules migration][Integration test] Create migration API (#11232)","number":210999,"url":"https://github.com/elastic/kibana/pull/210999","mergeCommit":{"message":"[Rules migration][Integration test] Create migration API (#11232) (#210999)\n\n## Summary\r\n\r\n[Internal link](https://github.com/elastic/security-team/issues/10820)\r\nto the feature details\r\n\r\nPart of https://github.com/elastic/security-team/issues/11232\r\n\r\nThis PR covers SIEM Migrations CREATE API (route:\r\n`/internal/siem_migrations/rules/{migration_id?}`) integration test:\r\n* Create migration with provided ID\r\n* Create migration without provided ID\r\n* Create migration with rules that have resources\r\n* Error handling (\"no content\") when no rules provided\r\n\r\nAlso, as part of this PR, I addressed this comment to my previous\r\nchanges\r\nhttps://github.com//pull/210867#discussion_r1954344990","sha":"44fdf81bbec5c053ebccfbdbb062d39de0d7358c"}},"sourceBranch":"main","suggestedTargetBranches":["9.0","8.18","8.x"],"targetPullRequestStates":[{"branch":"9.0","label":"v9.0.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.18","label":"v8.18.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/210999","number":210999,"mergeCommit":{"message":"[Rules migration][Integration test] Create migration API (#11232) (#210999)\n\n## Summary\r\n\r\n[Internal link](https://github.com/elastic/security-team/issues/10820)\r\nto the feature details\r\n\r\nPart of https://github.com/elastic/security-team/issues/11232\r\n\r\nThis PR covers SIEM Migrations CREATE API (route:\r\n`/internal/siem_migrations/rules/{migration_id?}`) integration test:\r\n* Create migration with provided ID\r\n* Create migration without provided ID\r\n* Create migration with rules that have resources\r\n* Error handling (\"no content\") when no rules provided\r\n\r\nAlso, as part of this PR, I addressed this comment to my previous\r\nchanges\r\nhttps://github.com//pull/210867#discussion_r1954344990","sha":"44fdf81bbec5c053ebccfbdbb062d39de0d7358c"}},{"branch":"8.x","label":"v8.19.0","branchLabelMappingKey":"^v8.19.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}] BACKPORT--> Co-authored-by: Ievgen Sorokopud <[email protected]>
… (#210999) (#211085) # Backport This will backport the following commits from `main` to `9.0`: - [[Rules migration][Integration test] Create migration API (#11232) (#210999)](#210999) <!--- Backport version: 9.4.3 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Ievgen Sorokopud","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-02-13T19:20:06Z","message":"[Rules migration][Integration test] Create migration API (#11232) (#210999)\n\n## Summary\r\n\r\n[Internal link](https://github.com/elastic/security-team/issues/10820)\r\nto the feature details\r\n\r\nPart of https://github.com/elastic/security-team/issues/11232\r\n\r\nThis PR covers SIEM Migrations CREATE API (route:\r\n`/internal/siem_migrations/rules/{migration_id?}`) integration test:\r\n* Create migration with provided ID\r\n* Create migration without provided ID\r\n* Create migration with rules that have resources\r\n* Error handling (\"no content\") when no rules provided\r\n\r\nAlso, as part of this PR, I addressed this comment to my previous\r\nchanges\r\nhttps://github.com//pull/210867#discussion_r1954344990","sha":"44fdf81bbec5c053ebccfbdbb062d39de0d7358c","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:Threat Hunting","Team: SecuritySolution","backport:version","v8.18.0","v9.1.0","v8.19.0"],"title":"[Rules migration][Integration test] Create migration API (#11232)","number":210999,"url":"https://github.com/elastic/kibana/pull/210999","mergeCommit":{"message":"[Rules migration][Integration test] Create migration API (#11232) (#210999)\n\n## Summary\r\n\r\n[Internal link](https://github.com/elastic/security-team/issues/10820)\r\nto the feature details\r\n\r\nPart of https://github.com/elastic/security-team/issues/11232\r\n\r\nThis PR covers SIEM Migrations CREATE API (route:\r\n`/internal/siem_migrations/rules/{migration_id?}`) integration test:\r\n* Create migration with provided ID\r\n* Create migration without provided ID\r\n* Create migration with rules that have resources\r\n* Error handling (\"no content\") when no rules provided\r\n\r\nAlso, as part of this PR, I addressed this comment to my previous\r\nchanges\r\nhttps://github.com//pull/210867#discussion_r1954344990","sha":"44fdf81bbec5c053ebccfbdbb062d39de0d7358c"}},"sourceBranch":"main","suggestedTargetBranches":["9.0","8.18","8.x"],"targetPullRequestStates":[{"branch":"9.0","label":"v9.0.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.18","label":"v8.18.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/210999","number":210999,"mergeCommit":{"message":"[Rules migration][Integration test] Create migration API (#11232) (#210999)\n\n## Summary\r\n\r\n[Internal link](https://github.com/elastic/security-team/issues/10820)\r\nto the feature details\r\n\r\nPart of https://github.com/elastic/security-team/issues/11232\r\n\r\nThis PR covers SIEM Migrations CREATE API (route:\r\n`/internal/siem_migrations/rules/{migration_id?}`) integration test:\r\n* Create migration with provided ID\r\n* Create migration without provided ID\r\n* Create migration with rules that have resources\r\n* Error handling (\"no content\") when no rules provided\r\n\r\nAlso, as part of this PR, I addressed this comment to my previous\r\nchanges\r\nhttps://github.com//pull/210867#discussion_r1954344990","sha":"44fdf81bbec5c053ebccfbdbb062d39de0d7358c"}},{"branch":"8.x","label":"v8.19.0","branchLabelMappingKey":"^v8.19.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}] BACKPORT--> Co-authored-by: Ievgen Sorokopud <[email protected]>
… (#210999) (#211084) # Backport This will backport the following commits from `main` to `8.x`: - [[Rules migration][Integration test] Create migration API (#11232) (#210999)](#210999) <!--- Backport version: 9.4.3 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Ievgen Sorokopud","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-02-13T19:20:06Z","message":"[Rules migration][Integration test] Create migration API (#11232) (#210999)\n\n## Summary\r\n\r\n[Internal link](https://github.com/elastic/security-team/issues/10820)\r\nto the feature details\r\n\r\nPart of https://github.com/elastic/security-team/issues/11232\r\n\r\nThis PR covers SIEM Migrations CREATE API (route:\r\n`/internal/siem_migrations/rules/{migration_id?}`) integration test:\r\n* Create migration with provided ID\r\n* Create migration without provided ID\r\n* Create migration with rules that have resources\r\n* Error handling (\"no content\") when no rules provided\r\n\r\nAlso, as part of this PR, I addressed this comment to my previous\r\nchanges\r\nhttps://github.com//pull/210867#discussion_r1954344990","sha":"44fdf81bbec5c053ebccfbdbb062d39de0d7358c","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:Threat Hunting","Team: SecuritySolution","backport:version","v8.18.0","v9.1.0","v8.19.0"],"title":"[Rules migration][Integration test] Create migration API (#11232)","number":210999,"url":"https://github.com/elastic/kibana/pull/210999","mergeCommit":{"message":"[Rules migration][Integration test] Create migration API (#11232) (#210999)\n\n## Summary\r\n\r\n[Internal link](https://github.com/elastic/security-team/issues/10820)\r\nto the feature details\r\n\r\nPart of https://github.com/elastic/security-team/issues/11232\r\n\r\nThis PR covers SIEM Migrations CREATE API (route:\r\n`/internal/siem_migrations/rules/{migration_id?}`) integration test:\r\n* Create migration with provided ID\r\n* Create migration without provided ID\r\n* Create migration with rules that have resources\r\n* Error handling (\"no content\") when no rules provided\r\n\r\nAlso, as part of this PR, I addressed this comment to my previous\r\nchanges\r\nhttps://github.com//pull/210867#discussion_r1954344990","sha":"44fdf81bbec5c053ebccfbdbb062d39de0d7358c"}},"sourceBranch":"main","suggestedTargetBranches":["9.0","8.18","8.x"],"targetPullRequestStates":[{"branch":"9.0","label":"v9.0.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.18","label":"v8.18.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/210999","number":210999,"mergeCommit":{"message":"[Rules migration][Integration test] Create migration API (#11232) (#210999)\n\n## Summary\r\n\r\n[Internal link](https://github.com/elastic/security-team/issues/10820)\r\nto the feature details\r\n\r\nPart of https://github.com/elastic/security-team/issues/11232\r\n\r\nThis PR covers SIEM Migrations CREATE API (route:\r\n`/internal/siem_migrations/rules/{migration_id?}`) integration test:\r\n* Create migration with provided ID\r\n* Create migration without provided ID\r\n* Create migration with rules that have resources\r\n* Error handling (\"no content\") when no rules provided\r\n\r\nAlso, as part of this PR, I addressed this comment to my previous\r\nchanges\r\nhttps://github.com//pull/210867#discussion_r1954344990","sha":"44fdf81bbec5c053ebccfbdbb062d39de0d7358c"}},{"branch":"8.x","label":"v8.19.0","branchLabelMappingKey":"^v8.19.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}] BACKPORT--> Co-authored-by: Ievgen Sorokopud <[email protected]>
Summary
Internal link to the feature details
Part of https://github.com/elastic/security-team/issues/11232
This PR covers SIEM Migrations GET API (route:
/internal/siem_migrations/rules/{migration_id}
) integration test: