On 09/30/2011 05:22 PM, Vivek Goyal wrote:
> On Fri, Sep 23, 2011 at 03:52:31PM +0200, Jan Safranek wrote:
>> Hello,
>>
>> here is first bunch of patches that are needed for better cooperation with
>> systemd. As agreed, service cgconfig stop/restart should only remove groups
>> that cgconfig start has created and only if they are empty. This patchset
>> focuses on that - it allows cgclear to parse a config file and remove only
>> groups defined there and (optionally) only if they are empty.
>>
> 
> Thanks for doing this work Jan.
> 
>> cgclear and cgconfigparser were updated to parse multiple files and even
>> directories to simplify the init script and future unit file(s). 
> 
> What's the use case of being able to specify mulitple files?
> 
>>
>> The best way to review the set is probably start with the last patch (=man
>> pages) so you see the functionality the patchset tries to implement. There 
>> are
>> also few tests you can examine.
> 
> I see that you introduced an option to cgclear to pass a file. So an init
> script is supposed to take snapshot of /etc/cgconfig.conf when cgroup
> service starts and then pass that snapshot file to cgclear when cgroup
> service is being stopped?

Yes, that's the idea for now. Comments are welcome, that's why the
series is RFC. I'll try to write an unit file and check how it works
with systemd and e.g. libvirt. And I am also working on the sticky bit,
but I probably need to break API for that :(.

> If a user is using cgconfigparser and cgclear directly then he/she is
> supposed to do this at their own?

Yes. If someone calls cgconfigparser several times, cgclear doesn't have
a clue what to clear - just the last cgconfigparser call? All of them?

Jan

------------------------------------------------------------------------------
All the data continuously generated in your IT infrastructure contains a
definitive record of customers, application performance, security
threats, fraudulent activity and more. Splunk takes this data and makes
sense of it. Business sense. IT sense. Common sense.
http://p.sf.net/sfu/splunk-d2dcopy1
_______________________________________________
Libcg-devel mailing list
Libcg-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/libcg-devel

Reply via email to