Re: [pve-devel] [PATCH] volume_snapshot : add testfeature option param

2012-09-30 Thread Alexandre DERUMIER
De: "Dietmar Maurer" À: "Alexandre Derumier" , pve-devel@pve.proxmox.com Envoyé: Lundi 1 Octobre 2012 06:32:44 Objet: RE: [pve-devel] [PATCH] volume_snapshot : add testfeature option param This hide knowledge inside a function. Instead. I would prefer to have a declarative

Re: [pve-devel] [PATCH] volume_snapshot : add testfeature option param

2012-09-30 Thread Dietmar Maurer
r 2012 16:22 > To: pve-devel@pve.proxmox.com > Subject: [pve-devel] [PATCH] volume_snapshot : add testfeature option > param > > return 1 for storage which support snapshot before doing the snapshot > > Signed-off-by: Alexandre Derumier > --- > PVE/Storage.pm

Re: [pve-devel] [PATCH] volume_snapshot : add testfeature option param

2012-09-30 Thread Dietmar Maurer
> boun...@pve.proxmox.com] On Behalf Of Alexandre Derumier > Sent: Freitag, 28. September 2012 16:22 > To: pve-devel@pve.proxmox.com > Subject: [pve-devel] [PATCH] volume_snapshot : add testfeature option > param > > return 1 for storage which support snapsho

[pve-devel] [PATCH] volume_snapshot : add testfeature option param

2012-09-28 Thread Alexandre Derumier
return 1 for storage which support snapshot before doing the snapshot Signed-off-by: Alexandre Derumier --- PVE/Storage.pm|4 ++-- PVE/Storage/NexentaPlugin.pm |4 +++- PVE/Storage/Plugin.pm |6 +++--- PVE/Storage/RBDPlugin.pm |4 ++-- PVE/Storage/Sh