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

2015-09-14 Thread Christian Brauner
On Tue, Sep 15, 2015 at 12:57:26AM +, Serge Hallyn wrote: > Quoting Christian Brauner (christianvanbrau...@gmail.com): > > 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 +0

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 functio

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 > woul

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: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 (christianv

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 +0

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 (christianv

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 +0

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 (serge.hal...@ub

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 (christianvanbrau...@g

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 (christianvanbrau...@g

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 Hal

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 (christianvanbrau...@g

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 config,

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 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 (christianvanbrau...@gmail.com): > > > > When creating ephemeral containers that have the op

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 > > Acked-by: Serge E. Hallyn > > I wo

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 config,

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 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 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 lxc_snap

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 lxc_snap

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 > --- > .gitignore | 1 + > 1 file changed, 1 insertion(+) > > diff --g

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 35d89f2..9c60c92 100644 > --- a/.gitignore > +++ b/.gitignore > @@ -1

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

2015-09-14 Thread Serge Hallyn
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 Acked-by: Serge E. Hallyn I wonder if it's worth augmenting the script to know about missing, required, and a