chore: Reduce number of log files#646
Merged
mergify[bot] merged 2 commits intosigp:release-v1.0.0from Oct 8, 2025
Merged
Conversation
AgeManning
changed the title
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR reduces the default maximum number of log files from 100 to 10 to address excessive disk usage from compressed log files on servers.
- Reduces default log file retention from 100 to 10 files
- Maintains the same 100MB default size per file, reducing total default storage from ~10GB to ~1GB
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
dknopik
previously approved these changes
Oct 6, 2025
Member
dknopik
left a comment
dknopik
left a comment
There was a problem hiding this comment.
LGTM, the old default was back from when we logged even more
Member
|
I think we even want this in |
dknopik
force-pushed
the
reduce-log-files
branch
from
f7992f8 to
e2b3737
Compare
17 tasks
petarjuki7
pushed a commit
to petarjuki7/anchor
that referenced
this pull request
Oct 16, 2025
I was looking through some logs on our servers and noticed a large number of compressed log files. I see that we have a default of 100 compressed log files. Each are default to 100MB. I think this is a bit excessive. I've reduced the default to 10 log files. Co-Authored-By: Age Manning <Age@AgeManning.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Issue Addressed
I was looking through some logs on our servers and noticed a large number of compressed log files.
I see that we have a default of 100 compressed log files. Each are default to 100MB. I think this is a bit excessive. I've reduced the default to 10 log files.