-
Notifications
You must be signed in to change notification settings - Fork 523
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
WatchServiceTest is failing on Windows #2968
Comments
I'm a beginner but i would like to try resolve it. |
Hello, as i was trying to see what's wrong with the test i noticed that sometimes it passed normally. I think maybe it has to do with if the docker desktop is open or not since the tests are for docker. If that's the case i could enable them only on linux or add this as a requirement? |
@Sintivrousai : Could it be a timeout issue? Does test pass if you increase the timeout from 5 seconds to more? |
That should be the case. I tried with 10sec and it passed. |
@Sintivrousai : Hmm, I see. Does it pass everytime or it fails some time? |
Not everytime apparently. I am testing with 20secs now and its okey so far. I am going to experiment it a little more. |
I keep facing some problems and i have increased the time limits more than I expected (even to 3minutes). However, one good thing is that the fails has been reduced quite a bit. What i mean is that when i try to build it 2 times out of 3 it does not fail. I have changed the time empirically. It does not seem right to me to keep increasing the time. |
@Sintivrousai : Maybe it's better to check what's causing wait in test execution and see if it's something related to windows. On Linux, the test passes normally |
Hi @Sintivrousai , @rohanKanojia : I'm unable to replicate this issue. Are we sure that the test is still failing? |
It's not just on windows, this test is failing intermittently on MacOS as well:
|
Component
JKube Kit
Task description
Description
Originally posted by @Sintivrousai in Gitter Thread
It looks like WatchServiceTest is failing on Windows again. We had merged a fix #2678 to fix this test but not sure why test has started failing again:
Expected Behavior
WatchServiceTest should pass on windows
Acceptance Criteria
The text was updated successfully, but these errors were encountered: