Bug 102686 - Please reduce log level of service activation messages to lower than NOTICE
Summary: Please reduce log level of service activation messages to lower than NOTICE
Status: RESOLVED FIXED
Alias: None
Product: dbus
Classification: Unclassified
Component: core (show other bugs)
Version: unspecified
Hardware: Other Linux (All)
: medium normal
Assignee: Simon McVittie
QA Contact: D-Bus Maintainers
URL:
Whiteboard: review+
Keywords: patch
Depends on:
Blocks:
 
Reported: 2017-09-12 17:55 UTC by Alan Jenkins
Modified: 2017-09-25 12:47 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
unix: Reduce log level for DBUS_SYSTEM_LOG_INFO to LOG_INFO (988 bytes, patch)
2017-09-22 20:10 UTC, Simon McVittie
Details | Splinter Review

Description Alan Jenkins 2017-09-12 17:55:45 UTC
dbus-daemon[599]: [system] Activating via systemd: service name='net.reactivated.Fprint'
dbus-daemon[599]: [system] Successfully activated service 'net.reactivated.Fprint'

Note services like Fprint de-activate themselves after a few seconds being idle, as well as activating on-demand.

There are too many of these messages to warrant being "highlighted" in high-intensity white in the system log (journalctl).  Please reduce the loglevel to lower than NOTICE.

E.g. systemd does not log a NOTICE for every single system service it starts, it uses the INFO level for these messages.

The other highlighted messages I see journalctl are mainly warnings, and big events like the start of a system clock adjustment, initial kernel boot message, switching from initramfs to the root filesystem.  Starting FPrintd does not count as a big event, when it will be stopped again 30 seconds later :).


Version: dbus-1.11.16-1.fc26.x86_64
Comment 1 Simon McVittie 2017-09-22 20:04:30 UTC
Commit 69ed32cbccbec9d613447cb64e9d7b1ffa11ce3c "Add syslog of security denials and configuration file reloads" added _dbus_log_info(), which inexplicably used LOG_NOTICE rather than LOG_INFO. NOTICE perhaps made sense for the original use-case (configuration file reloads) but I'm not sure why the mismatched name.

Using LOG_INFO for DBUS_SYSTEM_LOG_INFO makes a lot of sense to me.
Comment 2 Simon McVittie 2017-09-22 20:10:59 UTC
Created attachment 134442 [details] [review]
unix: Reduce log level for DBUS_SYSTEM_LOG_INFO to LOG_INFO

This is a better match for the way we use it in practice.
Comment 3 Philip Withnall 2017-09-25 10:18:47 UTC
Comment on attachment 134442 [details] [review]
unix: Reduce log level for DBUS_SYSTEM_LOG_INFO to LOG_INFO

Review of attachment 134442 [details] [review]:
-----------------------------------------------------------------

++
Comment 4 Simon McVittie 2017-09-25 12:47:14 UTC
Thanks, fixed in git for 1.11.18.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.