Re: [lxc-devel] [PATCH 3/3] gitignore: add strange lxc@.service file

2015-09-14 Thread Serge Hallyn
Quoting Tycho Andersen (tycho.ander...@canonical.com): > I have no idea what this file is, but the build system seems to be > generating it, so let's ignore it. > > Signed-off-by: Tycho Andersen Acked-by: Serge E. Hallyn > --- >

Re: [lxc-devel] [PATCH] Add remove_snapshots_entry() (rebased - v2)

2015-09-14 Thread Serge Hallyn
Quoting Christian Brauner (christianvanbrau...@gmail.com): > When creating ephemeral containers that have the option lxc.ephemeral = 1 set > in their config, they will be destroyed on shutdown. As they are simple > overlay > clones of an existing container they should be registered in the

Re: [lxc-devel] [PATCH 2/3] gitignore: add Korean man page output

2015-09-14 Thread Serge Hallyn
Quoting Tycho Andersen (tycho.ander...@canonical.com): > Signed-off-by: Tycho Andersen Acked-by: Serge E. Hallyn > --- > .gitignore | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/.gitignore b/.gitignore > index

Re: [lxc-devel] [PATCH 1/3] lxc-checkconfig: add some more config options

2015-09-14 Thread Tycho Andersen
On Mon, Sep 14, 2015 at 02:17:03PM +, Serge Hallyn wrote: > Quoting Tycho Andersen (tycho.ander...@canonical.com): > > Here's some more config options that we do actually require to be able to > > boot containers. > > > > Signed-off-by: Tycho Andersen > >

Re: [lxc-devel] [PATCH] Add remove_snapshots_entry() (rebased - v2)

2015-09-14 Thread Serge Hallyn
Quoting Christian Brauner (christianvanbrau...@gmail.com): > When creating ephemeral containers that have the option lxc.ephemeral = 1 set > in their config, they will be destroyed on shutdown. As they are simple > overlay > clones of an existing container they should be registered in the

Re: [lxc-devel] [PATCH] Add remove_snapshots_entry() (rebased - v2)

2015-09-14 Thread Christian Brauner
On Mon, Sep 14, 2015 at 03:03:58PM +, Serge Hallyn wrote: > Quoting Christian Brauner (christianvanbrau...@gmail.com): > > When creating ephemeral containers that have the option lxc.ephemeral = 1 > > set > > in their config, they will be destroyed on shutdown. As they are simple > > overlay

Re: [lxc-devel] [PATCH] Add remove_snapshots_entry() (rebased - v2)

2015-09-14 Thread Christian Brauner
On Mon, Sep 14, 2015 at 02:50:39PM +, Serge Hallyn wrote: > Quoting Christian Brauner (christianvanbrau...@gmail.com): > > When creating ephemeral containers that have the option lxc.ephemeral = 1 > > set > > in their config, they will be destroyed on shutdown. As they are simple > > overlay

Re: [lxc-devel] [PATCH] Add remove_snapshots_entry() (rebased - v2)

2015-09-14 Thread Serge Hallyn
Quoting Christian Brauner (christianvanbrau...@gmail.com): > On Mon, Sep 14, 2015 at 02:50:39PM +, Serge Hallyn wrote: > > Quoting Christian Brauner (christianvanbrau...@gmail.com): > > > When creating ephemeral containers that have the option lxc.ephemeral = 1 > > > set > > > in their

Re: [lxc-devel] [PATCH] Add remove_snapshots_entry() (rebased - v2)

2015-09-14 Thread Christian Brauner
On Mon, Sep 14, 2015 at 02:50:39PM +, Serge Hallyn wrote: > Quoting Christian Brauner (christianvanbrau...@gmail.com): > > When creating ephemeral containers that have the option lxc.ephemeral = 1 > > set > > in their config, they will be destroyed on shutdown. As they are simple > > overlay

Re: [lxc-devel] [PATCH] Add remove_snapshots_entry() (rebased - v2)

2015-09-14 Thread Christian Brauner
On Mon, Sep 14, 2015 at 08:51:08PM +, Serge Hallyn wrote: > Quoting Christian Brauner (christianvanbrau...@gmail.com): > > On Mon, Sep 14, 2015 at 08:31:59PM +, Serge Hallyn wrote: > > > Quoting Christian Brauner (christianvanbrau...@gmail.com): > > > > On Mon, Sep 14, 2015 at 07:27:06PM

Re: [lxc-devel] [PATCH] Add remove_snapshots_entry() (rebased - v2)

2015-09-14 Thread Serge Hallyn
Quoting Christian Brauner (christianvanbrau...@gmail.com): > On Mon, Sep 14, 2015 at 08:31:59PM +, Serge Hallyn wrote: > > Quoting Christian Brauner (christianvanbrau...@gmail.com): > > > On Mon, Sep 14, 2015 at 07:27:06PM +, Serge Hallyn wrote: > > > > Quoting Christian Brauner

Re: [lxc-devel] [PATCH] Add remove_snapshots_entry() (rebased - v2)

2015-09-14 Thread Christian Brauner
On Mon, Sep 14, 2015 at 07:27:06PM +, Serge Hallyn wrote: > Quoting Christian Brauner (christianvanbrau...@gmail.com): > > On Mon, Sep 14, 2015 at 04:33:05PM +, Serge Hallyn wrote: > > > Quoting Serge Hallyn (serge.hal...@ubuntu.com): > > > > Quoting Christian Brauner

Re: [lxc-devel] [PATCH] Add remove_snapshots_entry() (rebased - v2)

2015-09-14 Thread Serge Hallyn
Quoting Christian Brauner (christianvanbrau...@gmail.com): > On Mon, Sep 14, 2015 at 07:27:06PM +, Serge Hallyn wrote: > > Quoting Christian Brauner (christianvanbrau...@gmail.com): > > > On Mon, Sep 14, 2015 at 04:33:05PM +, Serge Hallyn wrote: > > > > Quoting Serge Hallyn

Re: [lxc-devel] [PATCH] Add remove_snapshots_entry() (rebased - v2)

2015-09-14 Thread Christian Brauner
On Mon, Sep 14, 2015 at 08:31:59PM +, Serge Hallyn wrote: > Quoting Christian Brauner (christianvanbrau...@gmail.com): > > On Mon, Sep 14, 2015 at 07:27:06PM +, Serge Hallyn wrote: > > > Quoting Christian Brauner (christianvanbrau...@gmail.com): > > > > On Mon, Sep 14, 2015 at 04:33:05PM

Re: [lxc-devel] [PATCH] Add remove_snapshots_entry() (rebased - v2)

2015-09-14 Thread Christian Brauner
On Mon, Sep 14, 2015 at 10:54:34PM +, Serge Hallyn wrote: > Does it help if we simply define c->delete_with_snapshot_clones(), and have > src/lxc/destroy.c use that? Then we can contain mod_all_rdeps to being a > static function in src/lxc/lxccontainer.c If not, remind me where else we >

Re: [lxc-devel] [PATCH] Add remove_snapshots_entry() (rebased - v2)

2015-09-14 Thread Serge Hallyn
Quoting Serge Hallyn (serge.hal...@ubuntu.com): > Quoting Christian Brauner (christianvanbrau...@gmail.com): > > On Mon, Sep 14, 2015 at 10:54:34PM +, Serge Hallyn wrote: > > > Does it help if we simply define c->delete_with_snapshot_clones(), and > > > have > > > src/lxc/destroy.c use that?

Re: [lxc-devel] [PATCH] Add remove_snapshots_entry() (rebased - v2)

2015-09-14 Thread Serge Hallyn
Quoting Christian Brauner (christianvanbrau...@gmail.com): > On Mon, Sep 14, 2015 at 10:54:34PM +, Serge Hallyn wrote: > > Does it help if we simply define c->delete_with_snapshot_clones(), and have > > src/lxc/destroy.c use that? Then we can contain mod_all_rdeps to being a > > static

Re: [lxc-devel] [PATCH] Add remove_snapshots_entry() (rebased - v2)

2015-09-14 Thread Christian Brauner
On Mon, Sep 14, 2015 at 07:27:06PM +, Serge Hallyn wrote: > Quoting Christian Brauner (christianvanbrau...@gmail.com): > > On Mon, Sep 14, 2015 at 04:33:05PM +, Serge Hallyn wrote: > > > Quoting Serge Hallyn (serge.hal...@ubuntu.com): > > > > Quoting Christian Brauner

Re: [lxc-devel] [PATCH] Add remove_snapshots_entry() (rebased - v2)

2015-09-14 Thread Christian Brauner
On Mon, Sep 14, 2015 at 04:33:05PM +, Serge Hallyn wrote: > Quoting Serge Hallyn (serge.hal...@ubuntu.com): > > Quoting Christian Brauner (christianvanbrau...@gmail.com): > > > On Mon, Sep 14, 2015 at 02:50:39PM +, Serge Hallyn wrote: > > > > Quoting Christian Brauner

Re: [lxc-devel] [PATCH] Add remove_snapshots_entry() (rebased - v2)

2015-09-14 Thread Serge Hallyn
Quoting Christian Brauner (christianvanbrau...@gmail.com): > On Mon, Sep 14, 2015 at 04:33:05PM +, Serge Hallyn wrote: > > Quoting Serge Hallyn (serge.hal...@ubuntu.com): > > > Quoting Christian Brauner (christianvanbrau...@gmail.com): > > > > On Mon, Sep 14, 2015 at 02:50:39PM +, Serge