Skip to content

[7.x] [Security Solution][Detections] Re-enable skipped integration test (#87254)#87396

Merged
rylnd merged 2 commits intoelastic:7.xfrom
rylnd:backport/7.x/pr-87254
Jan 6, 2021
Merged

[7.x] [Security Solution][Detections] Re-enable skipped integration test (#87254)#87396
rylnd merged 2 commits intoelastic:7.xfrom
rylnd:backport/7.x/pr-87254

Conversation

@rylnd
Copy link
Copy Markdown
Contributor

@rylnd rylnd commented Jan 5, 2021

Backports the following commits to 7.x:

…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
@rylnd
Copy link
Copy Markdown
Contributor Author

rylnd commented Jan 6, 2021

@elasticmachine merge upstream

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

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

@rylnd rylnd merged commit 3c01bf5 into elastic:7.x Jan 6, 2021
@rylnd rylnd deleted the backport/7.x/pr-87254 branch January 6, 2021 17:16
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