Skip to content

[7.x] [Security Solution][Detections][7.12] Critical Threshold Rule Fixes (#92667)#93140

Merged
madirey merged 1 commit intoelastic:7.xfrom
madirey:backport/7.x/pr-92667
Mar 2, 2021
Merged

[7.x] [Security Solution][Detections][7.12] Critical Threshold Rule Fixes (#92667)#93140
madirey merged 1 commit intoelastic:7.xfrom
madirey:backport/7.x/pr-92667

Conversation

@madirey
Copy link
Copy Markdown
Contributor

@madirey madirey commented Mar 1, 2021

Backports the following commits to 7.x:

…lastic#92667)

* Threshold cardinality validation

* Remove comments

* Fix legacy threshold signal dupe mitigation

* Add find_threshold_signals tests

* remove comment

* bug fixes

* Fix edit form value initialization for cardinality_value

* Fix test

* Type and test fixes

* Tests/types

* Reenable threshold cypress test

* Schema fixes

* Types and tests, normalize threshold field util

* Continue cleaning up types

* Some more pre-7.12 tests

* Limit cardinality_field to length 1 for now

* Cardinality to array

* Cardinality to array

* Tests/types

* cardinality can be null

* Handle empty threshold field in bulk_create_threshold_signals

* Remove cardinality_field, cardinality_value
@madirey madirey added the backport This PR is a backport of another PR label Mar 1, 2021
@madirey madirey enabled auto-merge (squash) March 1, 2021 22:11
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
securitySolution 7.8MB 7.8MB +1.2KB
triggersActionsUi 1.6MB 1.5MB -23.9KB
total -22.7KB

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
securitySolution 235.8KB 236.5KB +766.0B
triggersActionsUi 104.0KB 104.1KB +82.0B
total +848.0B
Unknown metric groups

async chunk count

id before after diff
triggersActionsUi 41 42 +1

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

@madirey madirey merged commit 402f332 into elastic:7.x Mar 2, 2021
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.

2 participants