Re: [pve-devel] ZFSPlugin snapshot question

2015-08-29 Thread Dietmar Maurer
Earlier you seemed to imply that the ZFS/iet plugin works with targetcli - is that the case? I don't think so. ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Re: [pve-devel] ZFSPlugin snapshot question

2015-08-28 Thread Dietmar Maurer
Yes, I will look in to this this weekend. Forgot to mention. Since snapshots by nature is read-only you have to set option wp=true to have iscsi properly advertise read-only to the mount util. I think Linux will make some internal optimization given it knows writes is not supported. Just

Re: [pve-devel] ZFSPlugin snapshot question

2015-08-28 Thread Dietmar Maurer
On August 28, 2015 at 12:19 PM datanom.net m...@datanom.net wrote: On 2015-08-28 10:25, Dietmar Maurer wrote: Another approach would be: 1.) create a clone 2.) backup cloned VM with 'stop' mode That way we do not even need to implement 'snapshot' backups ;-) Any comments

Re: [pve-devel] ZFSPlugin snapshot question

2015-08-28 Thread datanom.net
On 2015-08-28 10:25, Dietmar Maurer wrote: Another approach would be: 1.) create a clone 2.) backup cloned VM with 'stop' mode That way we do not even need to implement 'snapshot' backups ;-) Any comments about that idea? Will creating a clone instead of a snapshot not take considerably

Re: [pve-devel] ZFSPlugin snapshot question

2015-08-28 Thread Dietmar Maurer
Yes, i guess so. It also has other problems, because not all storage type supports cloning from snapshots ... So it was an interesting idea, but does not really solve all problems :-/ To solve the problem for the ZFSPlugin, I think it may help to introduce another storage API call to

Re: [pve-devel] ZFSPlugin snapshot question

2015-08-28 Thread Dietmar Maurer
While thinking about this, I came to a totally different topic. Would it make sense to add an Linux iSCSI (LIO) backend (LunCMD driver)? It is really easy to setup a ZFS box with our installer, so one could easily use such box as external iSCSI storage then? Comments?

Re: [pve-devel] ZFSPlugin snapshot question

2015-08-28 Thread Michael Rasmussen
On Fri, 28 Aug 2015 19:35:51 +0200 (CEST) Dietmar Maurer diet...@proxmox.com wrote: I just thought we can write a PVE::Storage::LunCmd::LIO, so that we can use a normal Linux/Proxmox box (with zfsonlinux) with the PVE::Storage::ZFSPlugin. This makes sense. I guess it is already

Re: [pve-devel] ZFSPlugin snapshot question

2015-08-28 Thread Lindsay Mathieson
-devel] ZFSPlugin snapshot question I guess it is already possible with PVE::Storage::LunCmd::Iet, but LIO should provide better performance? I have been thinking about this as well because Iet is deprecated in newer distributions (younger Debian Wheezy) But I just noticed

Re: [pve-devel] ZFSPlugin snapshot question

2015-08-28 Thread Dietmar Maurer
I guess it is already possible with PVE::Storage::LunCmd::Iet, but LIO should provide better performance? I have been thinking about this as well because Iet is deprecated in newer distributions (younger Debian Wheezy) But I just noticed that there is no targetcli package for jessie! I

Re: [pve-devel] ZFSPlugin snapshot question

2015-08-27 Thread Michael Rasmussen
On Thu, 27 Aug 2015 19:31:49 +0200 (CEST) Dietmar Maurer diet...@proxmox.com wrote: Do you have time to implement that? We would need it to implement snapshot backup for LXC containers. Yes, I will look in to this this weekend. Forgot to mention. Since snapshots by nature is read-only you

Re: [pve-devel] ZFSPlugin snapshot question

2015-08-27 Thread Michael Rasmussen
On Thu, 27 Aug 2015 17:46:45 +0200 (CEST) Dietmar Maurer diet...@proxmox.com wrote: But how can we implement this with the ZFSPlugin?`Any idea if/how it is possible to access a snapshotted device? There are two preconditions which must be fulfilled: 1) LUN must be pointing to the snapshot

[pve-devel] ZFSPlugin snapshot question

2015-08-27 Thread Dietmar Maurer
Hi all, the PVE::Storage::path method has the following signature: PVE::Storage::path($cfg, $volid, $snapname); As you can see, It has that $snapname parameter, and I tried to correctly implement this here: