Skip to content

[7.x] [Security Solution][Detections][Threshold Rules] Threshold multiple aggregations with cardinality (#90826)#91792

Merged
madirey merged 1 commit intoelastic:7.xfrom
madirey:backport/7.x/pr-90826
Feb 18, 2021
Merged

[7.x] [Security Solution][Detections][Threshold Rules] Threshold multiple aggregations with cardinality (#90826)#91792
madirey merged 1 commit intoelastic:7.xfrom
madirey:backport/7.x/pr-90826

Conversation

@madirey
Copy link
Contributor

@madirey madirey commented Feb 18, 2021

Backports the following commits to 7.x:

…ggregations with cardinality (elastic#90826)

* Remove unnecessary spreads

* Layout, round 1

* Revert "Layout, round 1"

This reverts commit b73b34a.

* Make threshold field an array

* Add cardinality fields

* Fix validation schema

* Query for multi-aggs

* Finish multi-agg aggregation

* Translate to multi-agg buckets

* Fix existing tests and add new test skeletons

* clean up

* Fix types

* Fix threshold_result data structure

* previous signals filter

* Fix previous signal detection

* Finish previous signal parsing

* tying up loose ends

* Fix timeline view for multi-agg threshold signals

* Fix build_bulk_body tests

* test fixes

* Add test for threshold bucket filters

* Address comments

* Fixing schema errors

* Remove unnecessary comment

* Fix tests

* Fix types

* linting

* linting

* Fixes

* Handle pre-7.12 threshold format in timeline view

* missing null check

* adding in follow-up pr

* Handle pre-7.12 filters

* unnecessary change

* Revert "unnecessary change"

This reverts commit 3edc7f2.

* linting

* Fix rule schemas

* Fix tests

Co-authored-by: Marshall Main <marshall.main@elastic.co>
@madirey madirey added the backport This PR is a backport of another PR label Feb 18, 2021
@madirey madirey enabled auto-merge (squash) February 18, 2021 04:47
@kibanamachine
Copy link
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.7MB 7.7MB +5.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 238.5KB 239.3KB +790.0B

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

@madirey madirey merged commit bfff061 into elastic:7.x Feb 18, 2021
@madirey madirey deleted the backport/7.x/pr-90826 branch February 18, 2021 14:06
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