Skip to content
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

[Security Solution] Enable the feature flag for JSON diffing in prebuilt rules #173368

Merged
merged 5 commits into from
Dec 21, 2023

Conversation

nikitaindik
Copy link
Contributor

@nikitaindik nikitaindik commented Dec 14, 2023

Ticket: #169160

Summary

Sets the jsonPrebuiltRulesDiffingEnabled to true to enable the new "Updates" tab in prebuilt rules flyout.

IMPORTANT: Merge only once acceptance and exploratory testing are passed!

Scherm­afbeelding 2023-12-13 om 17 37 07

@nikitaindik nikitaindik added release_note:skip Skip the PR/issue when compiling release notes Team:Detections and Resp Security Detection Response Team Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. Team:Detection Rule Management Security Detection Rule Management Team Feature:Prebuilt Detection Rules Security Solution Prebuilt Detection Rules area v8.12.0 v8.13.0 labels Dec 14, 2023
@nikitaindik nikitaindik self-assigned this Dec 14, 2023
@nikitaindik nikitaindik marked this pull request as ready for review December 14, 2023 11:47
@nikitaindik nikitaindik requested a review from a team as a code owner December 14, 2023 11:47
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-detections-response (Team:Detections and Resp)

@elasticmachine
Copy link
Contributor

Pinging @elastic/security-solution (Team: SecuritySolution)

Copy link
Contributor

@banderror banderror left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please merge whenever you're ready @nikitaindik

@banderror
Copy link
Contributor

I'm planning to merge this PR today or tomorrow after the ongoing deployment to Serverless finishes.

@banderror banderror enabled auto-merge (squash) December 21, 2023 10:30
@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @nikitaindik

@banderror banderror merged commit d7b99a5 into elastic:main Dec 21, 2023
37 checks passed
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Dec 21, 2023
…ilt rules (elastic#173368)

**Ticket:** elastic#169160

## Summary

Sets the `jsonPrebuiltRulesDiffingEnabled` to `true` to enable the new
"Updates" tab in prebuilt rules flyout.

### IMPORTANT: Merge only once acceptance and exploratory testing are
passed!

<img width="1178" alt="Scherm­afbeelding 2023-12-13 om 17 37 07"
src="https://github.com/elastic/kibana/assets/15949146/9e07e069-595f-49a6-98c2-d2ed4aa3f977">

Co-authored-by: Georgii Gorbachev <[email protected]>
(cherry picked from commit d7b99a5)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.12

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

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Dec 21, 2023
…n prebuilt rules (#173368) (#173822)

# Backport

This will backport the following commits from `main` to `8.12`:
- [[Security Solution] Enable the feature flag for JSON diffing in
prebuilt rules (#173368)](#173368)

<!--- Backport version: 8.9.7 -->

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

<!--BACKPORT [{"author":{"name":"Nikita
Indik","email":"[email protected]"},"sourceCommit":{"committedDate":"2023-12-21T11:39:16Z","message":"[Security
Solution] Enable the feature flag for JSON diffing in prebuilt rules
(#173368)\n\n**Ticket:**
https://github.com/elastic/kibana/issues/169160\r\n\r\n##
Summary\r\n\r\nSets the `jsonPrebuiltRulesDiffingEnabled` to `true` to
enable the new\r\n\"Updates\" tab in prebuilt rules flyout.\r\n\r\n###
IMPORTANT: Merge only once acceptance and exploratory testing
are\r\npassed!\r\n\r\n<img width=\"1178\" alt=\"Scherm­afbeelding
2023-12-13 om 17 37
07\"\r\nsrc=\"https://github.com/elastic/kibana/assets/15949146/9e07e069-595f-49a6-98c2-d2ed4aa3f977\">\r\n\r\nCo-authored-by:
Georgii Gorbachev
<[email protected]>","sha":"d7b99a587799e34c9534e4ba793aad65b4f78d55","branchLabelMapping":{"^v8.13.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","v8.12.0","v8.13.0"],"number":173368,"url":"https://github.com/elastic/kibana/pull/173368","mergeCommit":{"message":"[Security
Solution] Enable the feature flag for JSON diffing in prebuilt rules
(#173368)\n\n**Ticket:**
https://github.com/elastic/kibana/issues/169160\r\n\r\n##
Summary\r\n\r\nSets the `jsonPrebuiltRulesDiffingEnabled` to `true` to
enable the new\r\n\"Updates\" tab in prebuilt rules flyout.\r\n\r\n###
IMPORTANT: Merge only once acceptance and exploratory testing
are\r\npassed!\r\n\r\n<img width=\"1178\" alt=\"Scherm­afbeelding
2023-12-13 om 17 37
07\"\r\nsrc=\"https://github.com/elastic/kibana/assets/15949146/9e07e069-595f-49a6-98c2-d2ed4aa3f977\">\r\n\r\nCo-authored-by:
Georgii Gorbachev
<[email protected]>","sha":"d7b99a587799e34c9534e4ba793aad65b4f78d55"}},"sourceBranch":"main","suggestedTargetBranches":["8.12"],"targetPullRequestStates":[{"branch":"8.12","label":"v8.12.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.13.0","labelRegex":"^v8.13.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/173368","number":173368,"mergeCommit":{"message":"[Security
Solution] Enable the feature flag for JSON diffing in prebuilt rules
(#173368)\n\n**Ticket:**
https://github.com/elastic/kibana/issues/169160\r\n\r\n##
Summary\r\n\r\nSets the `jsonPrebuiltRulesDiffingEnabled` to `true` to
enable the new\r\n\"Updates\" tab in prebuilt rules flyout.\r\n\r\n###
IMPORTANT: Merge only once acceptance and exploratory testing
are\r\npassed!\r\n\r\n<img width=\"1178\" alt=\"Scherm­afbeelding
2023-12-13 om 17 37
07\"\r\nsrc=\"https://github.com/elastic/kibana/assets/15949146/9e07e069-595f-49a6-98c2-d2ed4aa3f977\">\r\n\r\nCo-authored-by:
Georgii Gorbachev
<[email protected]>","sha":"d7b99a587799e34c9534e4ba793aad65b4f78d55"}}]}]
BACKPORT-->

Co-authored-by: Nikita Indik <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Prebuilt Detection Rules Security Solution Prebuilt Detection Rules area release_note:skip Skip the PR/issue when compiling release notes Team:Detection Rule Management Security Detection Rule Management Team Team:Detections and Resp Security Detection Response Team Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. v8.12.0 v8.13.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants