Bug 816738

Summary: cgsnapshot saves tabs but cgconfigparser doesn't accept them.
Product: Red Hat Enterprise Linux 6 Reporter: Edward <redhat>
Component: libcgroupAssignee: Peter Schiffer <pschiffe>
Status: CLOSED DUPLICATE QA Contact: Red Hat Kernel QE team <kernel-qe>
Severity: low Docs Contact:
Priority: unspecified    
Version: 6.2CC: jsafrane
Target Milestone: rc   
Target Release: ---   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-04-27 09:29:37 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Edward 2012-04-26 20:25:31 UTC
Description of problem: Using cgsnapshot to save a configuration that includes blkio.throttle.write_bps_device will result in a line in the cgconfig.conf like: 

blkio.throttle.write_bps_device="8:0    10485769";

Between the 8:0 and the 10485769 is a tab.  cgconfig will complain that it can't parse this file.  Replacing that tab with a space corrects the issue.

Version-Release number of selected component (if applicable):
x86_64 0.37 3.el6 (This is actually CentOS version)

How reproducible: Very


Steps to Reproduce:
1. Start with a cgconfig that uses blkio
2. echo "8:0 10485769" > blkio.throttle.write_bps_device 
3. cgsnapshot -s  > /etc/cgconfig.conf 
4. service cgconfig restart

Actual results:
[root@air /]# service cgconfig restart                                                                                                                                                                               
Stopping cgconfig service:                                 [  OK  ]
Starting cgconfig service: Loading configuration file /etc/cgconfig.conf failed
Value setting does not succeed
Failed to parse /etc/cgconfig.conf                         [FAILED]

Expected results:
This should result in a successful restart.

Additional info:

Comment 2 Jan Safranek 2012-04-27 09:29:37 UTC

*** This bug has been marked as a duplicate of bug 788580 ***

Comment 3 Edward 2012-04-27 13:22:59 UTC
Strange.... before entering this bug, I searched for cgsnapshot and only bug 676700 and bug 647107 came up.  Now, of course, I search and can see bug 788580.  Sorry for the trouble!

Comment 4 Jan Safranek 2012-04-27 13:37:27 UTC
No problem, bug 788580 has limited visibility (dunno why), now you are on cc: there you can actually see it.