Repository navigation
Log priorities for journald and Docker, and new default jobs - #60
Merged
Merged
Conversation
…olours and filters by level Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JVEkZofdPjQ4jtXFKipeUX
… 9:00, earnings monthly, performance 1w/1m/3m; no podcast, bi-annual or annual defaults Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JVEkZofdPjQ4jtXFKipeUX
…r's journald driver files them as err and info The unit sends stdout to the journal too (the <N> priority prefix still sets each line's priority there). Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JVEkZofdPjQ4jtXFKipeUX
…og lines carry no <N> prefix Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JVEkZofdPjQ4jtXFKipeUX
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.
Summary
<3>error,<4>warning,<6>info,<7>debug), sojournalctlcolours and filters (-p) by level.errandinfo, sojournalctl CONTAINER_NAME=finbot -p warningshows the problems.bot.pywrites both streams a whole line at a time;finbot.servicesends stdout to the journal too (it was discarded).earningsmonthly;performanceshown as1w,1m,3m. The podcast jobs and the bi-annual and annualmoversandperformancejobs are no longer defaults (/jobs add podcasts FEED_URLstill works).JOURNAL_STREAM(GitHub's runner sets it, which put<N>on log lines and hung the subprocess tests) and send every level to stderr (log.SPLIT); new tests cover the real split and the priority prefix.doc/configuration.md,doc/docker.md,doc/scheduling.md(default-jobs table renumbered) and the logging note inCLAUDE.md.Test plan
.venv/bin/python -m unittest discover -s tests -b(655 tests) passes, with and withoutJOURNAL_STREAMset🤖 Generated with Claude Code
https://claude.ai/code/session_01JVEkZofdPjQ4jtXFKipeUX