Skip to content

Skip flaky x-pack libbeat testsuite#9689

Merged
ruflin merged 1 commit intoelastic:masterfrom
ruflin:skip-travis
Dec 20, 2018
Merged

Skip flaky x-pack libbeat testsuite#9689
ruflin merged 1 commit intoelastic:masterfrom
ruflin:skip-travis

Conversation

@ruflin
Copy link
Copy Markdown
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
Copy Markdown
Contributor

ph commented Dec 19, 2018

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

@ruflin
Copy link
Copy Markdown
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
Copy Markdown
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