-
Notifications
You must be signed in to change notification settings - Fork 31
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
Print out configuration issues on stderr #26
Comments
Actually, pause this bug. I think maybe |
If this is happening, I can't tell. But editing the |
techsy730
added a commit
to techsy730/system76-scheduler
that referenced
this issue
Mar 2, 2022
See pop-os#26: this fixes the "failed to parse" situation
techsy730
added a commit
to techsy730/system76-scheduler
that referenced
this issue
Mar 2, 2022
See pop-os#26: this fixes the "failed to parse" situation
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Print out a warning about Configuration issues to stderr (probably add
--silent
option too for the CLI for those that don't even want that printed)Configuration issues such as:
.ron
file that doesn't parse as a valid Rust syntax datatype (syntax errors, etc).ron
file that had unexpected types (like an assignments/foo.ron whose contents is something like["a", "b"]
, aka an Array literal instead of a Map literalThe daemon should probably keep track of which
.ron
files it has printed such a warning about in the past, and not warn about those again unless the modification time of the file has been updated since the last printed warning.That way it doesn't repeat this warning every time system76-scheduler "runs a renice cycle".
Reproduction:
Create a file
/etc/system76-scheduler/assignments/invalid.ron
And populate it with something with a blatantly invalid syntax like
(no quotes around
f
and no symbolf
is currently defined, no value listed)Restart the
com.system76.Scheduler.service
service.Note how nothing shows up in
journalctl
(journalctl -eu com.system76.Scheduler.service
)Also, running
RUST_LOG=system76-scheduler=warn system76-scheduler daemon
manually, no warnings print out.The text was updated successfully, but these errors were encountered: