[Security Solution] Turns on feature flag for deprecated prebuilt rules workflow#263560
Conversation
|
Pinging @elastic/security-detections-response (Team:Detections and Resp) |
|
Pinging @elastic/security-solution (Team: SecuritySolution) |
|
Pinging @elastic/security-detection-rule-management (Team:Detection Rule Management) |
nikitaindik
left a comment
There was a problem hiding this comment.
Hey, @dplumlee! I reviewed and left a few comments. Please take a look.
f743f59 to
2e3450a
Compare
| values={{ | ||
| docsLink: ( | ||
| <EuiLink | ||
| href="https://www.elastic.co/docs/solutions/security/detect-and-alert/manage-detection-rules#deprecated-prebuilt-rules" |
There was a problem hiding this comment.
nit: We can use an existing manageDetectionRules doc link here. Here's an example: https://github.com/elastic/kibana/blob/main/x-pack/solutions/security/plugins/security_solution/public/detection_engine/rule_creation/components/step_rule_actions/translations.tsx#L30-L37
There was a problem hiding this comment.
Thanks for the fixes, Davis! I re-reviewed, re-tested and left a couple of minor comments for you to consider. Approving in advance.
Side note: I tested on 9.4.0-beta.1 package by building it from source and adding this to the config:
xpack.fleet.developer:
bundledPackageLocation: /Users/nondemic/Work/detection-rules/dist/bundled
disableRegistryVersionCheck: true
xpack.securitySolution.prebuiltRulesPackageVersion: 9.4.0-beta.1
I haven't tested on a real package, since it's not yet in the EPR.
⏳ Build in-progress, with failures
Failed CI StepsTest Failures
History
cc @dplumlee |
|
Starting backport for target branches: 9.4 https://github.com/elastic/kibana/actions/runs/24801949169 |
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
…lt rules workflow (#263560) (#265154) # Backport This will backport the following commits from `main` to `9.4`: - [[Security Solution] Turns on feature flag for deprecated prebuilt rules workflow (#263560)](#263560) <!--- Backport version: 9.6.6 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sorenlouv/backport) <!--BACKPORT [{"author":{"name":"Davis Plumlee","email":"56367316+dplumlee@users.noreply.github.com"},"sourceCommit":{"committedDate":"2026-04-22T20:49:37Z","message":"[Security Solution] Turns on feature flag for deprecated prebuilt rules workflow (#263560)\n\n## Summary\n\nTurns on the `prebuiltRulesDeprecationUIEnabled` feature flag\nimplemented in this PR: https://github.com/elastic/kibana/pull/259673\n\nAlso makes some small UI fixes that were brought up during acceptance\ntesting:\n\n- Adds different language to the duplicate and delete modal informing\nthe user of the deletion\n - Adds padding to the tooltips in the duplication confirm modal\n - Adds link to docs in callout components\n\n### Screenshots\n\n\n\n<img width=\"1051\" height=\"468\" alt=\"Screenshot 2026-04-16 at 12 33\n29 PM\"\nsrc=\"https://github.com/user-attachments/assets/468d970d-9009-40ef-99d9-c693e46c3e1a\"\n/>\n\n\n<img width=\"1744\" height=\"321\" alt=\"Screenshot 2026-04-20 at 4 47 45 PM\"\nsrc=\"https://github.com/user-attachments/assets/db580301-edb8-44b5-851c-197aab366c81\"\n/>","sha":"65cf4f24f67642678bf71fe528fd17d11e2a1ee9","branchLabelMapping":{"^v9.5.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:Detections and Resp","Team: SecuritySolution","Team:Detection Rule Management","Feature:Prebuilt Detection Rules","backport:version","v9.4.0","v9.5.0"],"title":"[Security Solution] Turns on feature flag for deprecated prebuilt rules workflow","number":263560,"url":"https://github.com/elastic/kibana/pull/263560","mergeCommit":{"message":"[Security Solution] Turns on feature flag for deprecated prebuilt rules workflow (#263560)\n\n## Summary\n\nTurns on the `prebuiltRulesDeprecationUIEnabled` feature flag\nimplemented in this PR: https://github.com/elastic/kibana/pull/259673\n\nAlso makes some small UI fixes that were brought up during acceptance\ntesting:\n\n- Adds different language to the duplicate and delete modal informing\nthe user of the deletion\n - Adds padding to the tooltips in the duplication confirm modal\n - Adds link to docs in callout components\n\n### Screenshots\n\n\n\n<img width=\"1051\" height=\"468\" alt=\"Screenshot 2026-04-16 at 12 33\n29 PM\"\nsrc=\"https://github.com/user-attachments/assets/468d970d-9009-40ef-99d9-c693e46c3e1a\"\n/>\n\n\n<img width=\"1744\" height=\"321\" alt=\"Screenshot 2026-04-20 at 4 47 45 PM\"\nsrc=\"https://github.com/user-attachments/assets/db580301-edb8-44b5-851c-197aab366c81\"\n/>","sha":"65cf4f24f67642678bf71fe528fd17d11e2a1ee9"}},"sourceBranch":"main","suggestedTargetBranches":["9.4"],"targetPullRequestStates":[{"branch":"9.4","label":"v9.4.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.5.0","branchLabelMappingKey":"^v9.5.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/263560","number":263560,"mergeCommit":{"message":"[Security Solution] Turns on feature flag for deprecated prebuilt rules workflow (#263560)\n\n## Summary\n\nTurns on the `prebuiltRulesDeprecationUIEnabled` feature flag\nimplemented in this PR: https://github.com/elastic/kibana/pull/259673\n\nAlso makes some small UI fixes that were brought up during acceptance\ntesting:\n\n- Adds different language to the duplicate and delete modal informing\nthe user of the deletion\n - Adds padding to the tooltips in the duplication confirm modal\n - Adds link to docs in callout components\n\n### Screenshots\n\n\n\n<img width=\"1051\" height=\"468\" alt=\"Screenshot 2026-04-16 at 12 33\n29 PM\"\nsrc=\"https://github.com/user-attachments/assets/468d970d-9009-40ef-99d9-c693e46c3e1a\"\n/>\n\n\n<img width=\"1744\" height=\"321\" alt=\"Screenshot 2026-04-20 at 4 47 45 PM\"\nsrc=\"https://github.com/user-attachments/assets/db580301-edb8-44b5-851c-197aab366c81\"\n/>","sha":"65cf4f24f67642678bf71fe528fd17d11e2a1ee9"}}]}] BACKPORT--> Co-authored-by: Davis Plumlee <56367316+dplumlee@users.noreply.github.com>
…es workflow (elastic#263560) ## Summary Turns on the `prebuiltRulesDeprecationUIEnabled` feature flag implemented in this PR: elastic#259673 Also makes some small UI fixes that were brought up during acceptance testing: - Adds different language to the duplicate and delete modal informing the user of the deletion - Adds padding to the tooltips in the duplication confirm modal - Adds link to docs in callout components ### Screenshots <img width="1051" height="468" alt="Screenshot 2026-04-16 at 12 33 29 PM" src="https://github.com/user-attachments/assets/468d970d-9009-40ef-99d9-c693e46c3e1a" /> <img width="1744" height="321" alt="Screenshot 2026-04-20 at 4 47 45 PM" src="https://github.com/user-attachments/assets/db580301-edb8-44b5-851c-197aab366c81" />
Rebase conflict in df35c46 caused this flag to be reverted to false; restoring to true as set by elastic#263560. Made-with: Cursor
…es workflow (elastic#263560) ## Summary Turns on the `prebuiltRulesDeprecationUIEnabled` feature flag implemented in this PR: elastic#259673 Also makes some small UI fixes that were brought up during acceptance testing: - Adds different language to the duplicate and delete modal informing the user of the deletion - Adds padding to the tooltips in the duplication confirm modal - Adds link to docs in callout components ### Screenshots <img width="1051" height="468" alt="Screenshot 2026-04-16 at 12 33 29 PM" src="https://github.com/user-attachments/assets/468d970d-9009-40ef-99d9-c693e46c3e1a" /> <img width="1744" height="321" alt="Screenshot 2026-04-20 at 4 47 45 PM" src="https://github.com/user-attachments/assets/db580301-edb8-44b5-851c-197aab366c81" />
…es workflow (elastic#263560) ## Summary Turns on the `prebuiltRulesDeprecationUIEnabled` feature flag implemented in this PR: elastic#259673 Also makes some small UI fixes that were brought up during acceptance testing: - Adds different language to the duplicate and delete modal informing the user of the deletion - Adds padding to the tooltips in the duplication confirm modal - Adds link to docs in callout components ### Screenshots <img width="1051" height="468" alt="Screenshot 2026-04-16 at 12 33 29 PM" src="https://github.com/user-attachments/assets/468d970d-9009-40ef-99d9-c693e46c3e1a" /> <img width="1744" height="321" alt="Screenshot 2026-04-20 at 4 47 45 PM" src="https://github.com/user-attachments/assets/db580301-edb8-44b5-851c-197aab366c81" />

Summary
Turns on the
prebuiltRulesDeprecationUIEnabledfeature flag implemented in this PR: #259673Also makes some small UI fixes that were brought up during acceptance testing:
Screenshots