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 per field diffs feature #177495

Merged
merged 2 commits into from
Feb 23, 2024

Conversation

banderror
Copy link
Contributor

@banderror banderror commented Feb 21, 2024

Addresses: #166489

Summary

Turns on the perFieldPrebuiltRulesDiffingEnabled feature flag by default.

This will enable the Updates tab containing per-field rule diffs in the rule upgrade flyout. The feature will be enabled in 8.13.0 and Serverless. See more info in the related ticket.

Checklist

@banderror banderror 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 ci:cloud-deploy Create or update a Cloud deployment ci:project-deploy-security Create a Security Serverless Project v8.13.0 v8.14.0 labels Feb 21, 2024
@banderror banderror self-assigned this Feb 21, 2024
@banderror banderror requested a review from a team as a code owner February 21, 2024 17:23
@elasticmachine
Copy link
Contributor

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

@elasticmachine
Copy link
Contributor

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

@elasticmachine
Copy link
Contributor

Pinging @elastic/security-detection-rule-management (Team:Detection Rule Management)

@banderror banderror requested review from a team, jpdjere and dplumlee and removed request for jpdjere February 21, 2024 17:54
@banderror banderror force-pushed the per-field-diffs-enabled-by-default branch 2 times, most recently from 5662dea to a75c77c Compare February 22, 2024 18:43
Copy link
Contributor

@dplumlee dplumlee left a comment

Choose a reason for hiding this comment

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

Just merged in the tests for this, we should be good to go!

@banderror banderror force-pushed the per-field-diffs-enabled-by-default branch from a75c77c to 634caac Compare February 23, 2024 11:04
@banderror banderror enabled auto-merge (squash) February 23, 2024 11:14
@kibana-ci
Copy link
Collaborator

kibana-ci commented Feb 23, 2024

💚 Build Succeeded

Metrics [docs]

Canvas Sharable Runtime

The Canvas "shareable runtime" is an bundle produced to enable running Canvas workpads outside of Kibana. This bundle is included in third-party webpages that embed canvas and therefor should be as slim as possible.

id before after diff
module count - 5940 +5940
total size - 6.5MB +6.5MB

History

  • 💚 Build #195075 succeeded a75c77cc8688f12e5afa5a4a265d9ee6f371b08c
  • 💛 Build #194942 was flaky 5662dea1cc866f0917fe7e8c7cb9ddfa0c4c8e41
  • 💔 Build #194767 failed ada14f27faf94cdb8d6cba88654c44848268dff2

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

cc @banderror

@banderror banderror merged commit 61b7f67 into elastic:main Feb 23, 2024
42 checks passed
@kibanamachine
Copy link
Contributor

💔 All backports failed

Status Branch Result
8.13 Backport failed because of merge conflicts

Manual backport

To create the backport manually run:

node scripts/backport --pr 177495

Questions ?

Please refer to the Backport tool documentation

banderror added a commit to banderror/kibana that referenced this pull request Feb 23, 2024
**Addresses:** elastic#166489

## Summary

Turns on the `perFieldPrebuiltRulesDiffingEnabled` feature flag by
default.

This will enable the `Updates` tab containing per-field rule diffs in
the rule upgrade flyout. The feature will be enabled in `8.13.0` and
Serverless. See more info in the related ticket.

### Checklist

- [ ] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

(cherry picked from commit 61b7f67)

# Conflicts:
#	x-pack/plugins/security_solution/common/experimental_features.ts
@banderror
Copy link
Contributor Author

💚 All backports created successfully

Status Branch Result
8.13

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

Questions ?

Please refer to the Backport tool documentation

@banderror banderror deleted the per-field-diffs-enabled-by-default branch February 23, 2024 12:49
@banderror
Copy link
Contributor Author

@dplumlee Could you please approve the manually created backport and update the ticket?

banderror added a commit that referenced this pull request Feb 23, 2024
…177708)

# Backport

This will backport the following commits from `main` to `8.13`:
- [[Security Solution] Enable per field diffs feature
(#177495)](#177495)

<!--- Backport version: 8.9.8 -->

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

<!--BACKPORT [{"author":{"name":"Georgii
Gorbachev","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-02-23T12:28:22Z","message":"[Security
Solution] Enable per field diffs feature (#177495)\n\n**Addresses:**
https://github.com/elastic/kibana/issues/166489\r\n\r\n##
Summary\r\n\r\nTurns on the `perFieldPrebuiltRulesDiffingEnabled`
feature flag by\r\ndefault.\r\n\r\nThis will enable the `Updates` tab
containing per-field rule diffs in\r\nthe rule upgrade flyout. The
feature will be enabled in `8.13.0` and\r\nServerless. See more info in
the related ticket.\r\n\r\n\r\n### Checklist\r\n\r\n- [ ] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common
scenarios","sha":"61b7f67bda066d57455f5ca2ed69fd4e61657ffb","branchLabelMapping":{"^v8.14.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","ci:cloud-deploy","ci:project-deploy-security","v8.13.0","v8.14.0"],"number":177495,"url":"https://github.com/elastic/kibana/pull/177495","mergeCommit":{"message":"[Security
Solution] Enable per field diffs feature (#177495)\n\n**Addresses:**
https://github.com/elastic/kibana/issues/166489\r\n\r\n##
Summary\r\n\r\nTurns on the `perFieldPrebuiltRulesDiffingEnabled`
feature flag by\r\ndefault.\r\n\r\nThis will enable the `Updates` tab
containing per-field rule diffs in\r\nthe rule upgrade flyout. The
feature will be enabled in `8.13.0` and\r\nServerless. See more info in
the related ticket.\r\n\r\n\r\n### Checklist\r\n\r\n- [ ] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common
scenarios","sha":"61b7f67bda066d57455f5ca2ed69fd4e61657ffb"}},"sourceBranch":"main","suggestedTargetBranches":["8.13"],"targetPullRequestStates":[{"branch":"8.13","label":"v8.13.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.14.0","labelRegex":"^v8.14.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/177495","number":177495,"mergeCommit":{"message":"[Security
Solution] Enable per field diffs feature (#177495)\n\n**Addresses:**
https://github.com/elastic/kibana/issues/166489\r\n\r\n##
Summary\r\n\r\nTurns on the `perFieldPrebuiltRulesDiffingEnabled`
feature flag by\r\ndefault.\r\n\r\nThis will enable the `Updates` tab
containing per-field rule diffs in\r\nthe rule upgrade flyout. The
feature will be enabled in `8.13.0` and\r\nServerless. See more info in
the related ticket.\r\n\r\n\r\n### Checklist\r\n\r\n- [ ] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common
scenarios","sha":"61b7f67bda066d57455f5ca2ed69fd4e61657ffb"}}]}]
BACKPORT-->
fkanout pushed a commit to fkanout/kibana that referenced this pull request Mar 4, 2024
**Addresses:** elastic#166489

## Summary

Turns on the `perFieldPrebuiltRulesDiffingEnabled` feature flag by
default.

This will enable the `Updates` tab containing per-field rule diffs in
the rule upgrade flyout. The feature will be enabled in `8.13.0` and
Serverless. See more info in the related ticket.


### Checklist

- [ ] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci:cloud-deploy Create or update a Cloud deployment ci:project-deploy-security Create a Security Serverless Project 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.13.0 v8.14.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants