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

Allow configuring the max log files to keep #4777

Open
evenyag opened this issue Sep 27, 2024 · 0 comments
Open

Allow configuring the max log files to keep #4777

evenyag opened this issue Sep 27, 2024 · 0 comments
Labels
C-enhancement Category Enhancements good first issue Good for newcomers

Comments

@evenyag
Copy link
Contributor

evenyag commented Sep 27, 2024

What type of enhancement is this?

User experience

What does the enhancement do?

Currently, we don't limit the total number of log files. A long-running database may generate too many log files. We need to limit the max log files to keep by default.

Implementation challenges

The tracing-appender already supports setting the max log files. We can expose this in our LoggingOptions.
https://docs.rs/tracing-appender/latest/tracing_appender/rolling/struct.Builder.html#method.max_log_files

@evenyag evenyag added the C-enhancement Category Enhancements label Sep 27, 2024
@killme2008 killme2008 added the good first issue Good for newcomers label Sep 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-enhancement Category Enhancements good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants