[CRIU] [PATCH 1/4] cg: Add ability to dump custom cgroup properties

[Tycho Andersen]

On Fri, May 06, 2016 at 05:02:29PM +0300, Cyrill Gorcunov wrote:
> On Fri, May 06, 2016 at 04:51:48PM +0300, Pavel Emelyanov wrote:
> > >>>
> > >>> For vz7 we will need command line options (ie cgroup-props
> > >>> and cgroup-props-file) because it's convenient to test such
> > >>> things via cli patching libvzctl scripts.
> > >>>
> > >>> Btw, what is the better place for this "cgroup.yaml" file
> > >>> we gonna use as default one?
> > >>
> > >> I don't know. Maybe Tycho can suggest something? Does LX[CD] have
> > >> some list of default parameters for cgroups and, if it does, where
> > >> does it keep it?
> > > 
> > > We alrady setting up /usr/libexec/criu/scripts/ so maybe keep
> > > it in /usr/libexec/criu?
> > 
> > I'm not an expert in placing files over linux fs tree :) But the substring
> > "exec" in the path makes me think that this is a place for executables or
> > scripts.
> 
> Then /etc/criu I think a good start. Anyway, gonna wait for Tycho opinion.

So for debian/ubuntu I would drop it in /usr/share/criu/cgroups.yaml.
I don't know enough about redhat/fedora packaging policy to say where
it would go there, but presumably it is similar.

I will say that I'm not super convinced this is really necessary. At
least from an ubuntu perspective, to update the file, we'll need to
release a new package which will have a new binary anyway. Plus it
will complicate development :). I don't mind doing it if you guys
think it is needed, though.

I'll also update the list once we decide where it lives and land
Cyrill's set here.

Tycho