Re: [lxc-devel] [RFC 1/1] snapshots in api: define lxcapi_snapshot

2013-09-05 Thread Serge Hallyn
Quoting Stéphane Graber (stgra...@ubuntu.com): > On Wed, Sep 04, 2013 at 05:14:49PM -0500, Serge Hallyn wrote: > > Quoting Stéphane Graber (stgra...@ubuntu.com): > > > On Wed, Sep 04, 2013 at 11:05:02AM -0500, Serge Hallyn wrote: > > > > Hi, > > > > > > > > before I go on with the snapshots-in-api

Re: [lxc-devel] [RFC 1/1] snapshots in api: define lxcapi_snapshot

2013-09-04 Thread Stéphane Graber
On Wed, Sep 04, 2013 at 05:14:49PM -0500, Serge Hallyn wrote: > Quoting Stéphane Graber (stgra...@ubuntu.com): > > On Wed, Sep 04, 2013 at 11:05:02AM -0500, Serge Hallyn wrote: > > > Hi, > > > > > > before I go on with the snapshots-in-api patchset, I wanted to floar > > > this and ask a few quest

Re: [lxc-devel] [RFC 1/1] snapshots in api: define lxcapi_snapshot

2013-09-04 Thread Serge Hallyn
Quoting Stéphane Graber (stgra...@ubuntu.com): > On Wed, Sep 04, 2013 at 11:05:02AM -0500, Serge Hallyn wrote: > > Hi, > > > > before I go on with the snapshots-in-api patchset, I wanted to floar > > this and ask a few questions, as I'm having a hard time deciding how > > we want to talk over the

Re: [lxc-devel] [RFC 1/1] snapshots in api: define lxcapi_snapshot

2013-09-04 Thread Serge Hallyn
> In addition, I'll add lxcapi_snapshot_destroy(), which will look like: > > c = lxc_container_new("c0", "/var/lib/lxc"); > c->snapshot_destroy(c, "snap1"); > lxc_container_put(c); > > As for snapshot_list, I'm thinking it will just look like: > > c = lxc_container_new("c

Re: [lxc-devel] [RFC 1/1] snapshots in api: define lxcapi_snapshot

2013-09-04 Thread Stéphane Graber
On Wed, Sep 04, 2013 at 11:05:02AM -0500, Serge Hallyn wrote: > Hi, > > before I go on with the snapshots-in-api patchset, I wanted to floar > this and ask a few questions, as I'm having a hard time deciding how > we want to talk over the API. > > (Though as I've been typing this out, I think I n