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

skip filtering flaky tests #22193

Merged

Conversation

stacey-gammon
Copy link
Contributor

@stacey-gammon stacey-gammon commented Aug 20, 2018

These are still flaky (see #20442)

I suspect that in the case of the filtering dashboard, there are so many elements on the page that it fails to find all of the elements within the 10 second timeout period. I think to fix this, we need to allow waitForRender to override the default timeout. But in the mean time lets get the build stabilized again.

@stacey-gammon stacey-gammon added Feature:Dashboard Dashboard related features :Sharing labels Aug 20, 2018
Copy link
Contributor

@nreese nreese left a comment

Choose a reason for hiding this comment

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

lgtm
code review

@elasticmachine
Copy link
Contributor

💔 Build Failed

@stacey-gammon
Copy link
Contributor Author

Random xpack failure, could not build es.

jenkins, test this

@nreese
Copy link
Contributor

nreese commented Aug 20, 2018

Random xpack failure, could not build es.

It is not very random. I saw one of those this morning

#21655 (comment)

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@stacey-gammon stacey-gammon merged commit 6b17339 into elastic:master Aug 20, 2018
stacey-gammon added a commit to stacey-gammon/kibana that referenced this pull request Aug 20, 2018
@liza-mae liza-mae added failed-test A test failure on a tracked branch, potentially flaky-test and removed flaky-failing-test labels Nov 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
failed-test A test failure on a tracked branch, potentially flaky-test Feature:Dashboard Dashboard related features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants