Re: [pve-devel] zfs plugin improvements

2014-05-17 Thread Dietmar Maurer
After considerations I have come to the conclusion that this is intentional since the lock is controlled higher up in the chain of commands (QemuServer.pm). This is therefore no bug. I consider the waiting patches ready for inclusion. Please can you send me the exactly list of patches

Re: [pve-devel] zfs plugin improvements

2014-05-17 Thread Michael Rasmussen
On Sat, 17 May 2014 07:31:16 + Dietmar Maurer diet...@proxmox.com wrote: Please can you send me the exactly list of patches (what exactly is ready for inclusion)? This are the patches ready for inclusion: commit b3a716fa613391dc54244f7a894088de36b6a303 commit

Re: [pve-devel] zfs plugin improvements

2014-05-16 Thread Michael Rasmussen
On Mon, 12 May 2014 01:09:38 +0200 Michael Rasmussen m...@datanom.net wrote: commit b3a716fa613391dc54244f7a894088de36b6a303: snapshot recent-both I have tested this patch and have discovered a bug. If the user makes a rollback which trigger the patch's test to not allow to make a rollback

Re: [pve-devel] zfs plugin improvements

2014-05-16 Thread Michael Rasmussen
On Fri, 16 May 2014 20:25:41 +0200 Michael Rasmussen m...@datanom.net wrote: On Mon, 12 May 2014 01:09:38 +0200 Michael Rasmussen m...@datanom.net wrote: commit b3a716fa613391dc54244f7a894088de36b6a303: snapshot recent-both I have tested this patch and have discovered a bug. If the

Re: [pve-devel] zfs plugin improvements

2014-05-11 Thread Michael Rasmussen
On Fri, 9 May 2014 02:16:18 +0200 Michael Rasmussen m...@datanom.net wrote: The following remains to be tested: commit 5da23bad9844adfb61d3c093d08bf89eef86aadc: nowritecache-comstar commit 082e79f35b2f7b75862dc3014fb7de8e65fa76c6: sparse volumes-comstar,iet Above tested and no errors found.

Re: [pve-devel] zfs plugin improvements

2014-05-11 Thread Dietmar Maurer
One question: Can an option be enabled/disabled in the Panel depending on the value of a text box? If yes, how to implement? provider = comstar = sparse, nowritecache, target group, and host group enabled provider = istgt = sparse and nowritecache enabled. target group, and host group

Re: [pve-devel] zfs plugin improvements

2014-05-11 Thread Alexandre DERUMIER
Hi, it could be great to add option for compression enable|disable by volume :) (can be done live) - Mail original - De: Dietmar Maurer diet...@proxmox.com À: Michael Rasmussen m...@datanom.net, pve-devel@pve.proxmox.com Envoyé: Lundi 12 Mai 2014 05:52:03 Objet: Re: [pve-devel] zfs

Re: [pve-devel] zfs plugin improvements

2014-05-08 Thread Michael Rasmussen
On Tue, 6 May 2014 05:54:43 + Dietmar Maurer diet...@proxmox.com wrote: Thanks, that would be great. I don't really want to commit things to the ZFS plugin without prior acknowledge from you or Alexandre. I have tested the following commits on FreeBSD (Istgt): commit

[pve-devel] zfs plugin improvements

2014-05-05 Thread Adrian Costin
Hi, I would like to make the following improvements to the zfs plugin. I would appreciate any comments: 1. add a parameter to storage.cfg called sparse which would create sparse zvols if set to true. This can default to false for compatibility with the current version 2. add SRP support. I was

Re: [pve-devel] zfs plugin improvements

2014-05-05 Thread Michael Rasmussen
On Mon, 5 May 2014 20:56:32 +0300 Adrian Costin adrian.cos...@gmail.com wrote: 1. add a parameter to storage.cfg called sparse which would create sparse zvols if set to true. This can default to false for compatibility with the current version Already in git: commit

Re: [pve-devel] zfs plugin improvements

2014-05-05 Thread Adrian Costin
Already in git: commit 082e79f35b2f7b75862dc3014fb7de8e65fa76c6 Sorry, I didn't see if. It's not visible here: https://git.proxmox.com/?p=pve-storage.git;a=summary Best regards, Adrian Costin ___ pve-devel mailing list pve-devel@pve.proxmox.com

Re: [pve-devel] zfs plugin improvements

2014-05-05 Thread Michael Rasmussen
On Mon, 5 May 2014 21:26:41 +0300 Adrian Costin adrian.cos...@gmail.com wrote: Sorry, I didn't see if. It's not visible here: https://git.proxmox.com/?p=pve-storage.git;a=summary Strange. Seems al these are missing? git log commit b3a716fa613391dc54244f7a894088de36b6a303 Author: Chris

Re: [pve-devel] zfs plugin improvements

2014-05-05 Thread Michael Rasmussen
On Tue, 6 May 2014 04:25:11 + Dietmar Maurer diet...@proxmox.com wrote: Did you already test those patches? No (I might have forgotten about there existents;-). I intend to do it the coming days. -- Hilsen/Regards Michael Rasmussen Get my public GnuPG keys: michael at rasmussen dot cc

Re: [pve-devel] zfs plugin improvements

2014-05-05 Thread Dietmar Maurer
Did you already test those patches? No (I might have forgotten about there existents;-). I intend to do it the coming days. Thanks, that would be great. I don't really want to commit things to the ZFS plugin without prior acknowledge from you or Alexandre.