powerd debug messages flood powerd upstart log

Bug #1189538 reported by Tony Espy
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
powerd
Fix Released
Medium
Seth Forshee
powerd (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

There have been reports from various developers about the fact that powerd's approach to logging results in the powerd upstart log file being very large compared to the other upstart job log files. Looking at one of my phones running a recent image ( build 152 ), the powerd upstart log is about 3 times the size of the next largest log file ( one of the system network jobs ).

This is a result of powerd using GLib logging, but not actually setting up a default log handler, so the default log handler is used, which writes all messages to stdout/stderr, which results in the log messages going to the upstart log vs. the normal system syslog. The fix is simply to configure a log handler which writes to syslog:

https://developer.gnome.org/glib/2.35/glib-Message-Logging.html

Before a log handler is added, it also would be wise to ensure that the logging verbosity can be controlled via a command-line switch ( eg. -d ) or environment variable ( eg. POWERD_DEBUG=TRUE ).

Related branches

Seth Forshee (sforshee)
Changed in powerd:
assignee: nobody → Seth Forshee (sforshee)
importance: Undecided → Medium
status: New → In Progress
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:powerd at revision 42, scheduled for release in powerd, milestone ubuntu-13.05

Changed in powerd:
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package powerd - 0.13daily13.06.12-0ubuntu1

---------------
powerd (0.13daily13.06.12-0ubuntu1) saucy; urgency=low

  [ Seth Forshee ]
  * * Add support for display requests * Add display request dbus
    interface * Convert activity timeout and proximity sensor support to
    use display requests.
  * Log powerd messages to syslog instead of stdout/stderr (LP:
    #1189538). (LP: #1189538)

  [ Matthew Fischer ]
  * * Add support for display requests * Add display request dbus
    interface * Convert activity timeout and proximity sensor support to
    use display requests.

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 42
 -- Ubuntu daily release <email address hidden> Wed, 12 Jun 2013 02:02:03 +0000

Changed in powerd (Ubuntu):
status: New → Fix Released
Seth Forshee (sforshee)
Changed in powerd:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.