Skip to content

Remove usages of deprecated APIs #111

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

Merged
merged 4 commits into from
Jul 29, 2019
Merged

Conversation

basil
Copy link
Member

@basil basil commented Jul 22, 2019

While perusing the source tree, I noticed a number of deprecated methods:

Accordingly, I replaced any calls to User.get(java.lang.String with calls to User.getById(java.lang.String, boolean, I replaced any usages of ACL.impersonate with usages of try-with-resources and ACL.as, I replaced any calls to Jenkins.getActiveInstance() with calls to Jenkins.get(), and I replaced any calls to Jenkins.getInstance() with calls to Jenkins.getInstanceOrNull().

@basil basil changed the title Deprecated Remove usages of deprecated APIs Jul 22, 2019
@basil
Copy link
Member Author

basil commented Jul 22, 2019

mvn clean package passed when I ran the tests locally. The JVM in the CI build appears to have exited with status 143 (128 + 15), which indicates that the JVM received SIGTERM. I suspect the Jenkins agent being used by ci.jenkins.io is out of memory and that the test was killed by the Linux OOM killer, though without direct access to the Jenkins agent I have no way of verifying this.

@dwnusbaum
Copy link
Member

dwnusbaum commented Jul 22, 2019

I think the failure you got was because the Windows agent is disconnected (INFRA-2075):

win2012-007de0 was marked offline: Connection was broken: java.util.concurrent.TimeoutException: Ping started at 1563817994628 hasn't completed by 1563818234629

java.nio.channels.ClosedChannelException
Also: hudson.model.Computer$TerminationRequest: Termination requested at Mon Jul 22 17:57:14 UTC 2019 by Thread[Ping thread for channel hudson.remoting.Channel@1737b1c1:JNLP4-connect connection from 52.179.154.120/52.179.154.120:49190,5,main]

For infra issues like this, you can close the PR, wait a few (~ten) seconds, and reopen it to get a fresh build.

@dwnusbaum dwnusbaum closed this Jul 22, 2019
@dwnusbaum dwnusbaum reopened this Jul 22, 2019
@dwnusbaum dwnusbaum requested review from dwnusbaum and car-roll July 22, 2019 18:22
@dwnusbaum dwnusbaum merged commit 5b7c4e4 into jenkinsci:master Jul 29, 2019
@basil basil deleted the deprecated branch July 29, 2019 20:17
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.

2 participants