-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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] Test test_registrar.test_clean_removed fails #7690
Comments
It seems in this test the What I would expect in the above log is that after the event is sent that some states are cleaned up. The test needs to be modified to wait for the |
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
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 #7690
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)
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)
Reopening this issue as it seems test still keeps failing and it seems to be on linux and mac. Error message:
Log file:
Registry:
|
See elastic#7690 (cherry picked from commit be187b9)
The test failed again on Windows:
Logs
|
`test_clean_inactive` (elastic#8102) and `test_clean_removed` (elastic#7690) have been flaky recently again on Windows. Skipping these test for Windows.
We made some improvements to detect file removal. Time to have a look at test_registrar.py and unskip tests. |
Unfortunately the test still fails on windows. |
Backlog grooming: Closing it for now until further activity. |
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 6625835)
Job: https://beats-ci.elastic.co/job/elastic+beats+master+multijob-linux/1249/beat=filebeat,label=linux/
Platform: linux
Message:
Logs (I have more if needed)
The text was updated successfully, but these errors were encountered: