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

(cherry picked from commit e346293)
  • Loading branch information
ruflin committed Aug 28, 2018
1 parent cd40b48 commit 3726ac5
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 3726ac5

Please sign in to comment.