Activity log for bug #825598

Date Who What changed Old value New value Message
2011-08-13 00:37:47 Ryan Lovett bug added bug
2011-08-15 18:39:40 Serge Hallyn libcgroup (Ubuntu): status New In Progress
2011-08-15 18:39:44 Serge Hallyn libcgroup (Ubuntu): importance Undecided High
2011-08-15 18:39:49 Serge Hallyn libcgroup (Ubuntu): assignee Serge Hallyn (serge-hallyn)
2011-08-15 18:41:30 Serge Hallyn libcgroup (Ubuntu): status In Progress Fix Released
2011-08-15 18:41:42 Serge Hallyn nominated for series Ubuntu Lucid
2011-08-15 18:41:42 Serge Hallyn nominated for series Ubuntu Maverick
2011-08-15 19:19:17 Serge Hallyn attachment added Proposed fix for lucid https://bugs.launchpad.net/ubuntu/+source/libcgroup/+bug/825598/+attachment/2280800/+files/libcgroup.debdiff
2011-08-15 19:25:59 Serge Hallyn description The /etc/init.d/cgred startup script looks for the file /etc/default/cgred.conf rather than /etc/default/cgred. Additionally, the script makes no use of the DAEMON_OPTS variable found in the default file. Lastly, the log file parameter as provided won't work. (the daemon exits with a usage error) One has to use the -f option instead of --log-file in order for the daemon to start. It might be good to document LOG_FILE via a comment in /etc/default/cgred. -if [ -f "/etc/default/cgred.conf" ] ; then - . /etc/default/cgred.conf - OPTIONS="$NODAEMON $LOG" +if [ -f "/etc/default/cgred" ] ; then + . /etc/default/cgred + OPTIONS="$NODAEMON $LOG $DAEMON_OPTS" if [ -n "$LOG_FILE" ]; then - OPTIONS="$OPTIONS --log-file=$LOG_FILE" + OPTIONS="$OPTIONS -f $LOG_FILE" fi ======================================= SRU Justification: 1. Impact: startup modifications specified in /etc/default/cgred.conf are ignored. 2. How bug was addressed: the cgred init file is fixed. 3. Patch: see attached debdiff. 4. TEST CASE: add 'LOG_FILE=/var/log/snood' to /etc/default/cgred.conf. It will be ignored. 5. Regression potential: if people have updated their /etc/default/cgred with errors, which had up to now been ignored, then those errors would now be seen. ======================================= The /etc/init.d/cgred startup script looks for the file /etc/default/cgred.conf rather than /etc/default/cgred. Additionally, the script makes no use of the DAEMON_OPTS variable found in the default file. Lastly, the log file parameter as provided won't work. (the daemon exits with a usage error) One has to use the -f option instead of --log-file in order for the daemon to start. It might be good to document LOG_FILE via a comment in /etc/default/cgred. -if [ -f "/etc/default/cgred.conf" ] ; then - . /etc/default/cgred.conf - OPTIONS="$NODAEMON $LOG" +if [ -f "/etc/default/cgred" ] ; then + . /etc/default/cgred + OPTIONS="$NODAEMON $LOG $DAEMON_OPTS"   if [ -n "$LOG_FILE" ]; then - OPTIONS="$OPTIONS --log-file=$LOG_FILE" + OPTIONS="$OPTIONS -f $LOG_FILE"   fi
2011-08-15 19:26:08 Serge Hallyn bug added subscriber Ubuntu Stable Release Updates Team
2011-08-25 13:15:12 Martin Pitt bug task added libcgroup (Ubuntu Lucid)
2011-08-25 13:15:22 Martin Pitt bug task added libcgroup (Ubuntu Maverick)
2011-08-25 13:15:47 Martin Pitt libcgroup (Ubuntu Lucid): status New Fix Committed
2011-08-25 13:15:50 Martin Pitt bug added subscriber SRU Verification
2011-08-25 13:15:55 Martin Pitt tags verification-needed
2011-08-25 13:21:56 Martin Pitt libcgroup (Ubuntu Maverick): status New Fix Committed
2011-08-25 14:16:24 Launchpad Janitor branch linked lp:ubuntu/lucid-proposed/libcgroup
2011-08-25 14:16:31 Launchpad Janitor branch linked lp:ubuntu/maverick-proposed/libcgroup
2011-09-19 19:24:27 Ubuntu Foundations Team Bug Bot tags verification-needed testcase verification-needed
2012-11-15 15:49:31 Serge Hallyn tags testcase verification-needed testcase verification-done
2012-11-16 11:46:18 Colin Watson removed subscriber Ubuntu Stable Release Updates Team
2012-11-16 11:46:19 Colin Watson libcgroup (Ubuntu Maverick): status Fix Committed Won't Fix
2012-11-16 11:47:13 Launchpad Janitor libcgroup (Ubuntu Lucid): status Fix Committed Fix Released