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

Stabilize dashboard filtering test #58225

Merged
merged 2 commits into from
Feb 26, 2020

Conversation

flash1293
Copy link
Contributor

Fixes #55770

I could reproduce the problem of the original issue when testing against a real cloud instance. It seems like we are hitting some kind of circuit breaker which causes the request to fail with 504, probably because we are issuing lots of requests at once.

│ERROR browser[SEVERE] https://488dc2b34ba04eac84a982640b5dc769.eastus2.staging.azure.foundit.no:9243/elasticsearch/logstash-*/_search?rest_total_hits_as_int=true&ignore_unavailable=true&ignore_throttled=true&preference=1579821531484 - Failed to load resource: the server responded with a status of 504 ()
[

This is the error of the proxy:

{   "error_message": "backend closed connection",   "type": "proxy" }

On a second request against the same index it seems like the timing is different, because I never experienced a failure after refreshing - probably because some requests are already cached. Maybe this happens because the Kibana instance is underprovisioned in terms of RAM.

@flash1293 flash1293 marked this pull request as ready for review February 21, 2020 17:34
@flash1293 flash1293 added Feature:Dashboard Dashboard related features release_note:skip Skip the PR/issue when compiling release notes Team:Visualizations Visualization editors, elastic-charts and infrastructure test-cloud v7.7.0 v8.0.0 labels Feb 21, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app (Team:KibanaApp)

@flash1293
Copy link
Contributor Author

@elasticmachine merge upstream

@flash1293
Copy link
Contributor Author

@liza-mae could you review this one so we can check whether it stabilizes the test?

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

History

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

@liza-mae
Copy link
Contributor

@flash1293 I ran this 25 times and did not see any failures, so it looks good to me.

@liza-mae liza-mae self-requested a review February 26, 2020 01:33
Copy link
Contributor

@liza-mae liza-mae left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Dashboard Dashboard related features release_note:skip Skip the PR/issue when compiling release notes Team:Visualizations Visualization editors, elastic-charts and infrastructure test-cloud v7.6.1 v7.7.0 v8.0.0
Projects
None yet
4 participants