Content-Length: 302003 | pFad | https://github.com/All-Hands-AI/OpenHands/pull/10089

C9 fix file_ handler to TimedRotatingFileHandler type to prevent log fil… by 787627858 · Pull Request #10089 · All-Hands-AI/OpenHands · GitHub
Skip to content

Conversation

787627858
Copy link
Contributor

…es from being too large in production environments

  • This change is worth documenting at https://docs.all-hands.dev/
  • Include this change in the Release Notes. If checked, you must provide an end-user friendly description for your change below

End-user friendly description of the problem this fixes or functionality this introduces.
When users use OpenHands for production environments, the logs cannot be scrolled, which can cause the log files to become larger. By using TimedRotatingFileHandler, logs can be stored on a daily basis, making it easier to troubleshoot logs based on conversation time. By default, logs are saved for 7 days and will not cause the log files to become larger


Summarize what the PR does, explaining any non-trivial design decisions.
fix file_ handler to TimedRotatingFileHandler type to prevent log files from being too large in production environments


Link of any specific issues this addresses:

@xingyaoww
Copy link
Collaborator

@787627858 can you fix the linter?

@787627858
Copy link
Contributor Author

787627858 commented Aug 14, 2025

@787627858 can you fix the linter?
@xingyaoww Thank you. I have fixed the linter issues in the latest commit [4f9d008] and will ensure code compliance in future development.
apply pre-commit formatting

@787627858 787627858 requested a review from raymyers August 14, 2025 02:59
@xingyaoww xingyaoww enabled auto-merge (squash) August 14, 2025 03:00
@xingyaoww xingyaoww merged commit 2f32064 into All-Hands-AI:main Aug 14, 2025
16 checks passed
enyst pushed a commit that referenced this pull request Aug 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants









ApplySandwichStrip

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier!      Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

Fetched URL: https://github.com/All-Hands-AI/OpenHands/pull/10089

Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy