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

Added logging of update center URL #346

Merged
merged 1 commit into from Jul 9, 2021
Merged

Conversation

@EugeneLesnov
Copy link
Contributor

@EugeneLesnov EugeneLesnov commented Jul 8, 2021

Intended to close #333

try {
URIBuilder uriBuilder = new URIBuilder(cfg.getJenkinsUc().toURI());
if (jenkinsVersion != null) {
uriBuilder.addParameter("version", jenkinsVersion.toString()).build();
}
latestUcJson = getJson(uriBuilder.build().toURL(), "update-center" + cacheSuffix);
URL url = uriBuilder.build().toURL();
logVerbose("\nUpdate center URL: " + url);

This comment has been minimized.

@timja

timja Jul 8, 2021
Member

shouldn't need a \n?

This comment has been minimized.

@EugeneLesnov

EugeneLesnov Jul 9, 2021
Author Contributor

Yep, it looks better that way without \n

@EugeneLesnov EugeneLesnov force-pushed the EugeneLesnov:log_uc_url branch from f50ae4f to 5a890bc Jul 9, 2021
@timja timja merged commit a8ecd25 into jenkinsci:master Jul 9, 2021
13 checks passed
13 checks passed
@ci-jenkins-io
CPD No issues.
Details
@ci-jenkins-io
CheckStyle No issues.
Details
@ci-jenkins-io
Java Compiler No issues.
Details
@ci-jenkins-io
JavaDoc 7 new issues, 14 total.
Details
@ci-jenkins-io
Jenkins Success
Details
@ci-jenkins-io
Maven 6 new issues, 20 total.
Details
@ci-jenkins-io
Open Tasks Scanner No new issues, 8 total.
Details
@ci-jenkins-io
PMD No issues.
Details
@ci-jenkins-io
SpotBugs No issues.
Details
@ci-jenkins-io
Tests / linux-11 / Build (linux-11) skipped: 2, passed: 156
Details
@ci-jenkins-io
Tests / linux-8 / Build (linux-8) skipped: 2, passed: 156
Details
@ci-jenkins-io
Tests / windows-11 / Build (windows-11) skipped: 4, passed: 154
Details
@jenkinsadmin
continuous-integration/jenkins/incrementals Deployed to Incrementals.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

2 participants