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-68116 Increase log size limit #304

Merged
merged 3 commits into from
Mar 25, 2022

Conversation

timja
Copy link
Member

@timja timja commented Mar 25, 2022

Previously each log file was limited to 33KB and 5 files.
On our busy Jenkins instance we only got 15 minutes worth of logs before
they were rotated out.

It would be good for this to be increased to make analysis of issues
easier

see https://issues.jenkins.io/browse/JENKINS-68116

Previously each log file was limited to 33KB and 5 files.
On our busy Jenkins instance we only got 15 minutes worth of logs before
they were rotated out.

It would be good for this to be increased to make analysis of issues
easier
@timja timja marked this pull request as ready for review March 25, 2022 17:01
@timja timja requested a review from a team March 25, 2022 17:01
@timja
Copy link
Member Author

timja commented Mar 25, 2022

This was very useful for debugging https://issues.jenkins.io/browse/JENKINS-68116

Copy link
Member

@jtnord jtnord left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved with reservations, if others disagree with me feel free to take this as an approval

src/main/java/jenkins/branch/MultiBranchProject.java Outdated Show resolved Hide resolved
@jtnord jtnord requested a review from a team March 25, 2022 17:20
Co-authored-by: James Nord <jtnord@users.noreply.github.com>
@timja timja requested a review from jtnord March 25, 2022 20:07
@jtnord jtnord merged commit 0393857 into jenkinsci:master Mar 25, 2022
@timja timja deleted the increase-logging branch March 25, 2022 21:23
@timja
Copy link
Member Author

timja commented Mar 28, 2022

Possible to release this please @jtnord ?

@jtnord
Copy link
Member

jtnord commented Mar 28, 2022

Possible to release this please @jtnord ?

another repo without CD :-(

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

Successfully merging this pull request may close these issues.

2 participants