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-19445, JENKINS-34213, JENKINS-34808, JENKINS-34121] Bump remoting to 2.59. #2344

Merged
merged 2 commits into from
May 14, 2016

Conversation

oleg-nenashev
Copy link
Member

@oleg-nenashev oleg-nenashev commented May 13, 2016

Changes:

2.58:

2.59:

@reviewbybees @jenkinsci/code-reviewers

Changes:
* [JENKINS-34213](https://issues.jenkins-ci.org/browse/JENKINS-34213) - Ensure that the unexporter cleans up whatever it can each sweep (jenkinsci/remoting#81)
* [JENKINS-19445](https://issues.jenkins-ci.org/browse/JENKINS-19445) Force class load on UserRequest in order to prevent deadlock on windows nodes when using JNA and Subversion (jenkinsci/remoting#81)
* [JENKINS-34808](https://issues.jenkins-ci.org/browse/JENKINS-34808) - Allow user to adjust socket timeout (jenkinsci/remoting#68)
@oleg-nenashev oleg-nenashev added the needs-more-reviews Complex change, which would benefit from more eyes label May 13, 2016
@ghost
Copy link

ghost commented May 13, 2016

This pull request originates from a CloudBees employee. At CloudBees, we require that all pull requests be reviewed by other CloudBees employees before we seek to have the change accepted. If you want to learn more about our process please see this explanation.

@andresrc
Copy link
Contributor

🐝

1 similar comment
@rsandell
Copy link
Member

🐝

@oleg-nenashev
Copy link
Member Author

@reviewbybees done

@stephenc
Copy link
Member

🐝 👍

@ghost
Copy link

ghost commented May 13, 2016

This pull request has completed our internal processes and we now respectfully request the maintainers of this repository to consider our proposal contained within this pull request for merging.

@christ66
Copy link
Member

Should the remoting version be bumped up to 2.29 as @kohsuke just released a new version: jenkinsci/remoting@ab984a0?

@oleg-nenashev
Copy link
Member Author

@christ66 sure

@oleg-nenashev oleg-nenashev changed the title [JENKINS-19445, JENKINS-34213, JENKINS-34808] Bump remoting to 2.58. [JENKINS-19445, JENKINS-34213, JENKINS-34808, JENKINS-34121] Bump remoting to 2.59. May 13, 2016
@oleg-nenashev
Copy link
Member Author

Seems to be an INFRA issue, restarting the build

@oleg-nenashev
Copy link
Member Author

Actually it takes some time to get remoting in Maven central. Just stay tuned

@jglick
Copy link
Member

jglick commented May 13, 2016

🐝 but needs to wait for Central sync.

@oleg-nenashev oleg-nenashev reopened this May 13, 2016
@oleg-nenashev oleg-nenashev added ready-for-merge The PR is ready to go, and it will be merged soon if there is no negative feedback and removed needs-more-reviews Complex change, which would benefit from more eyes labels May 14, 2016
@oleg-nenashev oleg-nenashev merged commit 409438f into jenkinsci:master May 14, 2016
@oleg-nenashev oleg-nenashev removed the ready-for-merge The PR is ready to go, and it will be merged soon if there is no negative feedback label May 14, 2016
oleg-nenashev added a commit that referenced this pull request May 14, 2016
olivergondza pushed a commit that referenced this pull request May 25, 2016
…oting to 2.59. (#2344)

* [JENKINS-19445, JENKINS-34213, JENKINS-34808] Bump remoting to 2.58.

Changes:
* [JENKINS-34213](https://issues.jenkins-ci.org/browse/JENKINS-34213) - Ensure that the unexporter cleans up whatever it can each sweep (jenkinsci/remoting#81)
* [JENKINS-19445](https://issues.jenkins-ci.org/browse/JENKINS-19445) Force class load on UserRequest in order to prevent deadlock on windows nodes when using JNA and Subversion (jenkinsci/remoting#81)
* [JENKINS-34808](https://issues.jenkins-ci.org/browse/JENKINS-34808) - Allow user to adjust socket timeout (jenkinsci/remoting#68)

* [JENKINS-34121] - Upgrade remoting to 2.59

(cherry picked from commit 409438f)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants