Bug 1348874 - libcgroup - Order of mem values after cgsnapshot results in "Failure to remove non-empty group" on start.
Summary: libcgroup - Order of mem values after cgsnapshot results in "Failure to remov...
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: libcgroup
Version: rawhide
Hardware: Unspecified
OS: Unspecified
urgent
unspecified
Target Milestone: ---
Assignee: Nikola Forró
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 1315543
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-06-22 09:00 UTC by Nikola Forró
Modified: 2016-07-01 12:50 UTC (History)
10 users (show)

Fixed In Version: libcgroup-0.41-9.fc25
Doc Type: If docs needed, set a value
Doc Text:
Clone Of: 1315543
Environment:
Last Closed: 2016-07-01 12:50:43 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
api.c: fix order of memory subsystem parameters generated by cgsnapshot (2.06 KB, patch)
2016-06-22 09:03 UTC, Nikola Forró
no flags Details | Diff

Description Nikola Forró 2016-06-22 09:00:11 UTC
+++ This bug was initially created as a clone of Bug #1315543 +++

Description of problem:

 Setting memory.limit_in_bytes and memory.memsw.limit_in_bytes, and
 then using cgsnapshot -f, results in a file that causes the
 cgconfigparser to fail, due to the their ordering in the file:

 > Failed to remove a non-empty group Failed to parse /etc/cgconfig.conf or /etc/cgconfig.d

 It is either that cgsnapshot should not output entries in the
 incorrect order, or that this should not fail when checked or loaded.


Version-Release number of selected component (if applicable):

 libcgroup-0.41-8.fc24


Additional info:

 Swapping each set of lines to be the following will allow it to load:

                memory.limit_in_bytes="...";
                memory.memsw.limit_in_bytes="..."

 ------------------------------------------------------------
 # service cgconfig restart
 Stopping cgconfig service:                                 [  OK  ]
 Starting cgconfig service:                                 [  OK  ]
 ------------------------------------------------------------

Comment 1 Nikola Forró 2016-06-22 09:03:22 UTC
Created attachment 1170640 [details]
api.c: fix order of memory subsystem parameters generated by cgsnapshot


Note You need to log in before you can comment on or make changes to this bug.