Openpilot's disappearing logs and oversized-record crash fixed
A contributor used Claude Code to repair log rotation that deleted recent files and logging records that could crash openpilot's logging process.
Credits
- Source byline PeterPhuTran Source ↗
Earliest dated source:
The full story
One change corrects inconsistent file ordering that caused log rotation after a restart to remove the newest existing logs. Another rejects records that exceed the message queue's actual capacity instead of allowing the logging daemon to abort.
The contributor supplies reproductions and regression checks, including a device test of the oversized-record failure. Claude helped repair the existing logging bugs; the sources do not say it created them.
How it was done
Tools
Themes
Loading themes…
Sources
From the maker's own write-up or code. We summarise what they report; we haven't tested it ourselves.
- comma.ai openpilot: swaglog: delete oldest logs on rollover, not newestSource byline: PeterPhuTranPublished
The contributor’s pull request, including its AI disclosure and test notes.
- comma.ai openpilot: logmessaged: drop records too big for the msgq queue instead of dyingSource byline: PeterPhuTranPublished
A related contribution with implementation and verification details.
- comma.ai openpilot: logging_extra: dispatch events on the flag value, not its presenceSource byline: PeterPhuTranPublished
A related contribution with implementation and verification details.