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 flaky x-pack libbeat testsuite #9689

Merged
merged 1 commit into from
Dec 20, 2018
Merged

Conversation

ruflin
Copy link
Contributor

@ruflin ruflin commented Dec 19, 2018

In most cases the x-pack libbeat testsuite fails on Travis. This PR skips it. Will create a follow up issue to make sure we fix this.

In most cases the x-pack libbeat testsuite fails on Travis. This PR skips it. Will create a follow up issue to make sure we fix this.
@ruflin ruflin added review :Testing flaky-test Unstable or unreliable test cases. labels Dec 19, 2018
@ruflin ruflin requested a review from a team as a code owner December 19, 2018 19:34
@ph
Copy link
Contributor

ph commented Dec 19, 2018

Can we just disable part of it? Because this completely disable the whole target.

@ruflin
Copy link
Contributor Author

ruflin commented Dec 20, 2018

The failure part is during building the images and not running the tests. Disabling parts would mean to change how we run things here. For now my priority is to get green builds for everyone.

@ruflin ruflin merged commit e79f64a into elastic:master Dec 20, 2018
@ruflin ruflin deleted the skip-travis branch December 20, 2018 09:52
@ruflin
Copy link
Contributor Author

ruflin commented Dec 20, 2018

@ph The team should follow up on #9690

ph added a commit that referenced this pull request Dec 28, 2018
* Revert "Skip flaky x-pack libbeat testsuite (#9689)"

This reverts commit e79f64a.

* restart on failure
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
flaky-test Unstable or unreliable test cases. review :Testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants