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

Revert skips added while triaging ES OOMs #97876

Merged
merged 5 commits into from
Apr 22, 2021

Conversation

spalger
Copy link
Contributor

@spalger spalger commented Apr 21, 2021

While triaging #97378 we started by skipping tests that seems to be causing the issue. After the first few skips we started to realize that the problem was more systemic and eventually fixed the issue for us with #97813 while we wait for the ES team to figure out the flaw in the GeoIP database downloader that is causing these issues. As such, we need to unskip the tests which were unnecessarily skipped.

Closes #97382
Closes #97387
Closes #97378

@spalger spalger added Team:Operations Team label for Operations Team v8.0.0 release_note:skip Skip the PR/issue when compiling release notes v7.14.0 v7.13.0 labels Apr 21, 2021
@spalger spalger requested a review from a team as a code owner April 21, 2021 18:47
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-operations (Team:Operations)

@spalger spalger enabled auto-merge (squash) April 21, 2021 18:48
@spalger spalger added the auto-backport Deprecated - use backport:version if exact versions are needed label Apr 21, 2021
@spalger spalger requested a review from a team as a code owner April 21, 2021 20:48
@spalger spalger force-pushed the unskip-oom-failures branch from bb656fd to 68348e8 Compare April 22, 2021 01:02
@spalger
Copy link
Contributor Author

spalger commented Apr 22, 2021

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

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

Copy link
Member

@jbudz jbudz left a comment

Choose a reason for hiding this comment

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

(re-approving to get the operations reviewer check)

@spalger spalger disabled auto-merge April 22, 2021 18:30
@spalger spalger merged commit 6bb2893 into elastic:master Apr 22, 2021
@spalger spalger deleted the unskip-oom-failures branch April 22, 2021 18:30
kibanamachine added a commit to kibanamachine/kibana that referenced this pull request Apr 22, 2021
* Revert "skip flaky suite (elastic#97382)"

This reverts commit e321f57.

* Revert "skip flaky suite (elastic#97387)"

This reverts commit a89b756.

* Revert "Skip test to try and stabilize master" (elastic#97378)

This reverts commit 194355f.

* upload heap dumps when they are created

Co-authored-by: spalger <[email protected]>
Co-authored-by: Kibana Machine <[email protected]>
kibanamachine added a commit to kibanamachine/kibana that referenced this pull request Apr 22, 2021
* Revert "skip flaky suite (elastic#97382)"

This reverts commit e321f57.

* Revert "skip flaky suite (elastic#97387)"

This reverts commit a89b756.

* Revert "Skip test to try and stabilize master" (elastic#97378)

This reverts commit 194355f.

* upload heap dumps when they are created

Co-authored-by: spalger <[email protected]>
Co-authored-by: Kibana Machine <[email protected]>
@kibanamachine
Copy link
Contributor

💚 Backport successful

Status Branch Result
7.13
7.x

The backport PRs will be merged automatically after passing CI.

kibanamachine added a commit that referenced this pull request Apr 22, 2021
* Revert "skip flaky suite (#97382)"

This reverts commit e321f57.

* Revert "skip flaky suite (#97387)"

This reverts commit a89b756.

* Revert "Skip test to try and stabilize master" (#97378)

This reverts commit 194355f.

* upload heap dumps when they are created

Co-authored-by: spalger <[email protected]>
Co-authored-by: Kibana Machine <[email protected]>

Co-authored-by: Spencer <[email protected]>
Co-authored-by: spalger <[email protected]>
kibanamachine added a commit that referenced this pull request Apr 22, 2021
* Revert "skip flaky suite (#97382)"

This reverts commit e321f57.

* Revert "skip flaky suite (#97387)"

This reverts commit a89b756.

* Revert "Skip test to try and stabilize master" (#97378)

This reverts commit 194355f.

* upload heap dumps when they are created

Co-authored-by: spalger <[email protected]>
Co-authored-by: Kibana Machine <[email protected]>

Co-authored-by: Spencer <[email protected]>
Co-authored-by: spalger <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment