-
-
Notifications
You must be signed in to change notification settings - Fork 8.8k
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
Backporting for LTS 2.303.1 #5659
Conversation
(cherry picked from commit 0b0085d)
…insci#5634) (cherry picked from commit 9bf8b33)
(cherry picked from commit 0c89a7f)
Based on closed tickets with @basil I cannot execute the script so the format of the description is not the expected one. I cannot include |
Will try to review tomorrow. I am back to the Jenkins core maintenance, more or less |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Looks good to me pending #5599.
…jenkinsci#5621) * [JENKINS-66139] Avoid polluting the log if usage stats cannot be sent * Update core/src/main/java/hudson/model/UsageStatistics.java Co-authored-by: James Nord <jtnord@users.noreply.github.com> * Update core/src/main/java/hudson/model/UsageStatistics.java Co-authored-by: James Nord <jtnord@users.noreply.github.com> * Update core/src/main/java/hudson/model/UsageStatistics.java Co-authored-by: James Nord <jtnord@users.noreply.github.com> (cherry picked from commit 1f1eb71)
@timja @basil @jtnord I included JENKINS-66094 - PR 5621 |
@timja can we merge this PR? I saw the release candidate is not including this changes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Backports look correctly applied.
History looks a bit messy but seems with the support from Tim, so it should be fine.
it should be, it's an incremental build passed off of this. We can merge it anyway |
Candidates