-
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] Flaky test_clean_removed_with_clean_inactive test #7842
Labels
Comments
The issue here seems to be very similar to #7690 There needs to be a better way to wait for the clean_removed to happen. |
ruflin
added a commit
to ruflin/beats
that referenced
this issue
Aug 27, 2018
This removes the time.sleep part from the test in the hope it will make it more stable. I am not 100% confident that this will fully remove the flakyness of test_clean_removed_with_clean_inactive. If it fails again the future, the full log line + the registry should be added to Github issue to see what is still inside the registry. Closes elastic#7842
jsoriano
pushed a commit
that referenced
this issue
Aug 27, 2018
This removes the time.sleep part from the test in the hope it will make it more stable. I am not 100% confident that this will fully remove the flakyness of test_clean_removed_with_clean_inactive. If it fails again the future, the full log line + the registry should be added to Github issue to see what is still inside the registry. Closes #7842
ruflin
added a commit
to ruflin/beats
that referenced
this issue
Aug 28, 2018
This removes the time.sleep part from the test in the hope it will make it more stable. I am not 100% confident that this will fully remove the flakyness of test_clean_removed_with_clean_inactive. If it fails again the future, the full log line + the registry should be added to Github issue to see what is still inside the registry. Closes elastic#7842 (cherry picked from commit 3c1c6c1)
ruflin
added a commit
to ruflin/beats
that referenced
this issue
Aug 28, 2018
This removes the time.sleep part from the test in the hope it will make it more stable. I am not 100% confident that this will fully remove the flakyness of test_clean_removed_with_clean_inactive. If it fails again the future, the full log line + the registry should be added to Github issue to see what is still inside the registry. Closes elastic#7842 (cherry picked from commit 3c1c6c1)
exekias
pushed a commit
that referenced
this issue
Aug 28, 2018
This removes the time.sleep part from the test in the hope it will make it more stable. I am not 100% confident that this will fully remove the flakyness of test_clean_removed_with_clean_inactive. If it fails again the future, the full log line + the registry should be added to Github issue to see what is still inside the registry. Closes #7842 (cherry picked from commit 3c1c6c1)
ruflin
added a commit
that referenced
this issue
Aug 29, 2018
This removes the time.sleep part from the test in the hope it will make it more stable. I am not 100% confident that this will fully remove the flakyness of test_clean_removed_with_clean_inactive. If it fails again the future, the full log line + the registry should be added to Github issue to see what is still inside the registry. Closes #7842 (cherry picked from commit 3c1c6c1)
leweafan
pushed a commit
to leweafan/beats
that referenced
this issue
Apr 28, 2023
…ic#8120) This removes the time.sleep part from the test in the hope it will make it more stable. I am not 100% confident that this will fully remove the flakyness of test_clean_removed_with_clean_inactive. If it fails again the future, the full log line + the registry should be added to Github issue to see what is still inside the registry. Closes elastic#7842 (cherry picked from commit 602f0c3)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Error:
Log
The text was updated successfully, but these errors were encountered: