Skip to content

Log rotation#308

Merged
brickbots merged 8 commits intobrickbots:mainfrom
mrosseel:log-rotation
Apr 6, 2025
Merged

Log rotation#308
brickbots merged 8 commits intobrickbots:mainfrom
mrosseel:log-rotation

Conversation

@mrosseel
Copy link
Copy Markdown
Collaborator

@mrosseel mrosseel commented Apr 5, 2025 •
edited
Loading

  • removes the manual log rotation from main
  • changes the watchedfilehandler from multiproclogging with a rotatingfilehandler
  • configures 5 log files of maximum 50Mb each (I've seen 300Mb+ log files before this fix)

Numbers can be discussed of course.

@mrosseel mrosseel changed the title [WIP] Log rotation Log rotation Apr 6, 2025
mrosseel added 2 commits April 6, 2025 20:02
@mrosseel
Ecef fix (brickbots#307)
0d79769 
* Merged main with the location changes

* no ubx yet

* Revert "no ubx yet"

This reverts commit c872394.

* fix ecef errors

no ubx yet

Revert "no ubx yet"

This reverts commit c872394.

Let the logger do the rollover
@mrosseel
Merge branch 'log-rotation' of github.com:mrosseel/PiFinder into log-…
0f40d19 
…rotation
@brickbots brickbots merged commit 2e88d22 into brickbots:main Apr 6, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mrosseel @brickbots