Skip to content

[7.11] [Security Solution][Detections] Re-enable skipped integration test (#87254)#87397

Merged
rylnd merged 1 commit intoelastic:7.11from
rylnd:backport/7.11/pr-87254
Jan 6, 2021
Merged

[7.11] [Security Solution][Detections] Re-enable skipped integration test (#87254)#87397
rylnd merged 1 commit intoelastic:7.11from
rylnd:backport/7.11/pr-87254

Conversation

@rylnd
Copy link
Copy Markdown
Contributor

@rylnd rylnd commented Jan 5, 2021

Backports the following commits to 7.11:

…lastic#87254)

* Re-enable test skipped due to temporary failure

This failure may still be present/intermittent, but it passes reliably
locally; trying again on CI to check status.

* Triggering build

This should fail on the latest snapshot

* Update EQL integration tests to reflect new default pipe

We were previously using what is effectively `results | head` to
retrieve the desired amount of results. The default behavior was changed
in elastic/elasticsearch#66387, which caused these tests to fail as
different results were returned over such a large dataset.

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
@rylnd rylnd added the backport This PR is a backport of another PR label Jan 5, 2021
@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
triggersActionsUi 1.5MB 1.5MB -26.9KB

Distributable file count

id before after diff
default 47634 48397 +763

Page load bundle

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

id before after diff
triggersActionsUi 162.0KB 162.1KB +102.0B
Unknown metric groups

async chunk count

id before after diff
triggersActionsUi 31 32 +1

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

@rylnd rylnd merged commit eac0359 into elastic:7.11 Jan 6, 2021
@rylnd rylnd deleted the backport/7.11/pr-87254 branch January 6, 2021 00:23
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