Skip to content

Commit

Permalink
Fix flaky clean_removed test (elastic#8092)
Browse files Browse the repository at this point in the history
This is a potential fix for the flaky clean_removed test. The problem in the test seems to be that sometimes not all states are cleaned up yet. This is changing it by waiting for all pending cleanups to happen.

Closes elastic#7690
  • Loading branch information
ruflin authored and jsoriano committed Aug 28, 2018
1 parent 5577137 commit 6625835
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions filebeat/tests/system/test_registrar.py
Original file line number Diff line number Diff line change
Expand Up @@ -895,6 +895,9 @@ def test_clean_removed(self):
lambda: self.output_has(lines=3),
max_timeout=10)

# Make sure all states are cleaned up
self.wait_until(lambda: self.log_contains("Before: 1, After: 1, Pending: 0"))

filebeat.check_kill_and_wait()

# Check that the first to files were removed from the registry
Expand Down

0 comments on commit 6625835

Please sign in to comment.