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

[Filebeat] Skip flaky windows tests #11540

Merged
merged 1 commit into from
Apr 1, 2019
Merged

Conversation

ruflin
Copy link
Member

@ruflin ruflin commented Mar 29, 2019

test_clean_inactive (#8102) and test_clean_removed (#7690) have been flaky recently again on Windows. Skipping these test for Windows.

`test_clean_inactive` (elastic#8102) and `test_clean_removed` (elastic#7690) have been flaky recently again on Windows. Skipping these test for Windows.
@ruflin ruflin added review Filebeat Filebeat :Testing flaky-test Unstable or unreliable test cases. labels Mar 29, 2019
@ruflin ruflin requested a review from a team as a code owner March 29, 2019 15:14
@ruflin ruflin merged commit 4d1b9ce into elastic:master Apr 1, 2019
@ruflin ruflin deleted the skip-flaky-tests branch April 1, 2019 10:57
@urso
Copy link

urso commented Apr 1, 2019

We should collect recent artifacts + logs. It might be a timing issue. Maybe just increasing timeouts on Windows helps. I actually reduced the timeouts when fixing some bugs and it seemed to be green for a while.

@ruflin
Copy link
Member Author

ruflin commented Apr 1, 2019

@urso I saw it several times last week. Were your fix before or after? I did not collect the artifacts as I saw there is already an open issue for it, but probably should have :-(

@urso
Copy link

urso commented Apr 1, 2019

Fix was merged mid February: #10747

@ruflin
Copy link
Member Author

ruflin commented Apr 2, 2019

@urso test_clean_inactive was failing again on a PR not rebased on master yet. I added the artifacts to the related isseu.

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

Successfully merging this pull request may close these issues.

3 participants