Skip to content
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

JENKINS-71798: TimeoutService threads are left after closing connection #155

Merged

Conversation

mpet
Copy link
Contributor

@mpet mpet commented Aug 17, 2023

See JENKINS-71798.

Fix has been verified in customers environment

@mpet
Copy link
Contributor Author

mpet commented Aug 31, 2023

@kuisathaverat this is a friendly reminder to check this review.

mpet and others added 2 commits September 5, 2023 09:31
Co-authored-by: Ivan Fernandez Calvo <kuisathaverat@users.noreply.github.com>
@mpet
Copy link
Contributor Author

mpet commented Sep 5, 2023

@kuisathaverat I updated according to your comment. Please review again.

@kuisathaverat kuisathaverat merged commit a969bd3 into jenkinsci:main Sep 5, 2023
4 checks passed
@mpet
Copy link
Contributor Author

mpet commented Sep 5, 2023

@kuisathaverat will there be a release of the library automatically? or do you manually release?

@mpet
Copy link
Contributor Author

mpet commented Sep 5, 2023

@kuisathaverat looks like release was already done. Thx a lot for your efforts!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants