-
Notifications
You must be signed in to change notification settings - Fork 97
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
Stop the hanging threads in runner client forcefully after 30 seconds #416
Conversation
* add logging * add logging * add logging
🎉 All Contributor License Agreements have been signed. Ready to merge. |
can you update the CODEONWERS file and add cp-test-frameworks team. |
@PrasanthV454 For now I've added them in review, in separate PR will make the change today. |
@sprsi : Can you plz add sample semaphore job runs - 1 with upgrade tests and 1-2 with system tests (across branches say 7.3.x and 7.6.x) using this commit to ensure there is no regression. |
@prayadav-cp These are the runs on the new branch, looks good to me! |
Multiple threads are spawned while running tests and ultimately all of them need to join() to finish and publish the results (be it a success or a failure). However, if one or more threads don’t join(), they will eventually complete but the timing is non-deterministic.
This PR solves this join() issue (or rather harden the runner client threading mechanism)