Re: [libvirt] [PATCH 0/3] Add delete-snapshots option to virsh commands

2015-12-18 Thread Wido den Hollander
On 12/17/2015 09:54 PM, John Ferlan wrote: > > ping? > Looks good to me. Didn't think you needed a reply from me. But yes, I forgot this part. Only went for the API since that was what I needed in CloudStack. > Thanks - > > John > > On 12/02/2015 06:18 PM, John Ferlan wrote: >> Commit id

Re: [libvirt] [PATCH 0/3] Add delete-snapshots option to virsh commands

2015-12-18 Thread Michal Privoznik
On 03.12.2015 00:18, John Ferlan wrote: > Commit id '3c7590e0' added the ability to delete snapshots for rbd backends, > but did not provide the virsh commands in order to handle that option. > > The first two patches fix a couple of minor nits - not using virCheckFlags > in

Re: [libvirt] [PATCH 0/3] Add delete-snapshots option to virsh commands

2015-12-17 Thread John Ferlan
ping? Thanks - John On 12/02/2015 06:18 PM, John Ferlan wrote: > Commit id '3c7590e0' added the ability to delete snapshots for rbd backends, > but did not provide the virsh commands in order to handle that option. > > The first two patches fix a couple of minor nits - not using virCheckFlags

[libvirt] [PATCH 0/3] Add delete-snapshots option to virsh commands

2015-12-02 Thread John Ferlan
Commit id '3c7590e0' added the ability to delete snapshots for rbd backends, but did not provide the virsh commands in order to handle that option. The first two patches fix a couple of minor nits - not using virCheckFlags in virStorageBackendRBDDeleteVol even though the flags are used. Also, the