]> git.proxmox.com Git - pve-storage.git/commitdiff
rename ZFSDirPlugin to ZFSPoolPlugin
authorDietmar Maurer <dietmar@proxmox.com>
Sat, 24 Jan 2015 13:13:24 +0000 (14:13 +0100)
committerDietmar Maurer <dietmar@proxmox.com>
Sat, 24 Jan 2015 13:17:55 +0000 (14:17 +0100)
Turned out it makes no sense to duplicated DirPlugin features. So I
also changed the name to make it less confusing. So we can only
create zvols inside a zfs pool with this plugin.

PVE/Storage.pm
PVE/Storage/Makefile
PVE/Storage/ZFSDirPlugin.pm [deleted file]
PVE/Storage/ZFSPlugin.pm
PVE/Storage/ZFSPoolPlugin.pm [new file with mode: 0644]

index 380fbdf094e6eac8f0127d32ddf83a162a4743de..b491484977d485ad64da011796b03aea8ed977da 100755 (executable)
@@ -27,7 +27,7 @@ use PVE::Storage::RBDPlugin;
 use PVE::Storage::SheepdogPlugin;
 use PVE::Storage::ISCSIDirectPlugin;
 use PVE::Storage::GlusterfsPlugin;
-use PVE::Storage::ZFSDirPlugin;
+use PVE::Storage::ZFSPoolPlugin;
 use PVE::Storage::ZFSPlugin;
 
 # load and initialize all plugins
@@ -39,7 +39,7 @@ PVE::Storage::RBDPlugin->register();
 PVE::Storage::SheepdogPlugin->register();
 PVE::Storage::ISCSIDirectPlugin->register();
 PVE::Storage::GlusterfsPlugin->register();
-PVE::Storage::ZFSDirPlugin->register();
+PVE::Storage::ZFSPoolPlugin->register();
 PVE::Storage::ZFSPlugin->register();
 PVE::Storage::Plugin->init();
 
index c993ea4cde52c4fa0689f3efef9de50fed7bef91..f28d5d5b7642361a8afc500866c9a787da415fe1 100644 (file)
@@ -1,4 +1,4 @@
-SOURCES=Plugin.pm DirPlugin.pm LVMPlugin.pm NFSPlugin.pm ISCSIPlugin.pm RBDPlugin.pm SheepdogPlugin.pm ISCSIDirectPlugin.pm GlusterfsPlugin.pm ZFSDirPlugin.pm ZFSPlugin.pm
+SOURCES=Plugin.pm DirPlugin.pm LVMPlugin.pm NFSPlugin.pm ISCSIPlugin.pm RBDPlugin.pm SheepdogPlugin.pm ISCSIDirectPlugin.pm GlusterfsPlugin.pm ZFSPoolPlugin.pm ZFSPlugin.pm
 
 .PHONY: install
 install:
diff --git a/PVE/Storage/ZFSDirPlugin.pm b/PVE/Storage/ZFSDirPlugin.pm
deleted file mode 100644 (file)
index 5ab5e89..0000000
+++ /dev/null
@@ -1,499 +0,0 @@
-package PVE::Storage::ZFSDirPlugin;
-
-use strict;
-use warnings;
-use IO::File;
-use POSIX;
-use PVE::Tools qw(run_command);
-use PVE::Storage::Plugin;
-
-
-use base qw(PVE::Storage::Plugin);
-
-sub type {
-    return 'zfsdir';
-}
-
-sub plugindata {
-    return {
-       content => [ { images => 1, rootdir => 1, vztmpl => 1, iso => 1, backup => 1},
-                    { images => 1 }],
-    };
-}   
-
-sub properties {
-    return {
-       blocksize => {
-           description => "block size",
-           type => 'string',
-       },
-       sparse => {
-           description => "use sparse volumes",
-           type => 'boolean',
-       },
-    };
-}
-
-sub options {
-    return {
-       path => { fixed => 1 },
-       pool => { fixed => 1 },
-       blocksize => { optional => 1 },
-       sparse => { optional => 1 },
-       nodes => { optional => 1 },
-       disable => { optional => 1 },
-        maxfiles => { optional => 1 },
-       content => { optional => 1 },
-    };
-}
-
-# static zfs helper methods
-
-sub zfs_parse_size {
-    my ($text) = @_;
-
-    return 0 if !$text;
-    
-    if ($text =~ m/^(\d+(\.\d+)?)([TGMK])?$/) {
-
-       my ($size, $reminder, $unit) = ($1, $2, $3);
-       
-       if ($unit) {
-           if ($unit eq 'K') {
-               $size *= 1024;
-           } elsif ($unit eq 'M') {
-               $size *= 1024*1024;
-           } elsif ($unit eq 'G') {
-               $size *= 1024*1024*1024;
-           } elsif ($unit eq 'T') {
-               $size *= 1024*1024*1024*1024;
-           } else {
-               die "got unknown zfs size unit '$unit'\n";
-           }
-       }
-
-       if ($reminder) {
-           $size = ceil($size);
-       }
-       
-       return $size;
-    
-    }
-
-    warn "unable to parse zfs size '$text'\n";
-
-    return 0;
-}
-
-sub zfs_parse_zvol_list {
-    my ($text) = @_;
-
-    my $list = ();
-
-    return $list if !$text;
-
-    my @lines = split /\n/, $text;
-    foreach my $line (@lines) {
-       if ($line =~ /^(.+)\s+([a-zA-Z0-9\.]+|\-)\s+(.+)$/) {
-           my $zvol = {};
-           my @parts = split /\//, $1;
-           my $name = pop @parts;
-           my $pool = join('/', @parts);
-
-           if ($pool !~ /^rpool$/) {
-               next unless $name =~ m!^(\w+)-(\d+)-(\w+)-(\d+)$!;
-               $name = $pool . '/' . $name;
-           } else {
-               next;
-           }
-
-           $zvol->{pool} = $pool;
-           $zvol->{name} = $name;
-           $zvol->{size} = zfs_parse_size($2);
-           if ($3 !~ /^-$/) {
-               $zvol->{origin} = $3;
-           }
-           push @$list, $zvol;
-       }
-    }
-
-    return $list;
-}
-
-sub parse_volname {
-    my ($class, $volname) = @_;
-
-    if ($volname =~ m/^(((base|vm)-(\d+)-\S+)\/)?((base)?(vm)?-(\d+)-\S+)$/) {
-       return ('images', $5, $8, $2, $4, $6);
-    }
-
-    die "unable to parse zfs volume name '$volname'\n";
-}
-
-# virtual zfs methods (subclass can overwrite them)
-
-sub path {
-    my ($class, $scfg, $volname) = @_;
-
-    my ($vtype, $name, $vmid) = $class->parse_volname($volname);
-
-    my $path = '';
-
-    if($vtype eq "images"){
-       $path = "/dev/zvol/$scfg->{pool}/$volname";
-    } else {
-       die "$vtype is not allowed in ZFSDir!";
-    }
-
-    return ($path, $vmid, $vtype);
-}
-
-sub zfs_request {
-    my ($class, $scfg, $timeout, $method, @params) = @_;
-
-    $timeout = 5 if !$timeout;
-
-    my $cmd = [];
-
-    if ($method eq 'zpool_list') {
-       push @$cmd = 'zpool', 'list';
-    } else {
-       push @$cmd, 'zfs', $method;
-    }
-
-    push @$cmd, @params;
-    my $msg = '';
-
-    my $output = sub {
-        my $line = shift;
-        $msg .= "$line\n";
-    };
-
-    run_command($cmd, outfunc => $output, timeout => $timeout);
-
-    return $msg;
-}
-
-sub alloc_image {
-    my ($class, $storeid, $scfg, $vmid, $fmt, $name, $size) = @_;
-
-    die "unsupported format '$fmt'" if $fmt ne 'raw';
-
-    die "illegal name '$name' - sould be 'vm-$vmid-*'\n"
-    if $name && $name !~ m/^vm-$vmid-/;
-
-    $name = $class->zfs_find_free_diskname($storeid, $scfg, $vmid) if !$name;
-    
-    $class->zfs_create_zvol($scfg, $name, $size);
-
-    return $name;
-}
-
-sub free_image {
-    my ($class, $storeid, $scfg, $volname, $isBase) = @_;
-
-    my (undef, $name, undef) = $class->parse_volname($volname);
-
-    $class->zfs_delete_zvol($scfg, $name);
-
-    return undef;
-}
-
-sub list_images {
-    my ($class, $storeid, $scfg, $vmid, $vollist, $cache) = @_;
-
-    $cache->{zfs} = $class->zfs_list_zvol($scfg) if !$cache->{zfs};
-    my $zfspool = $scfg->{pool};
-    my $res = [];
-
-    if (my $dat = $cache->{zfs}->{$zfspool}) {
-
-       foreach my $image (keys %$dat) {
-
-           my $volname = $dat->{$image}->{name};
-           my $parent = $dat->{$image}->{parent};
-
-           my $volid = undef;
-            if ($parent && $parent =~ m/^(\S+)@(\S+)$/) {
-               my ($basename) = ($1);
-               $volid = "$storeid:$basename/$volname";
-           } else {
-               $volid = "$storeid:$volname";
-           }
-
-           my $owner = $dat->{$volname}->{vmid};
-           if ($vollist) {
-               my $found = grep { $_ eq $volid } @$vollist;
-               next if !$found;
-           } else {
-               next if defined ($vmid) && ($owner ne $vmid);
-           }
-
-           my $info = $dat->{$volname};
-           $info->{volid} = $volid;
-           push @$res, $info;
-       }
-    }
-
-    return $res;
-}
-
-sub zfs_get_pool_stats {
-    my ($class, $scfg) = @_;
-
-    my $available = 0;
-    my $used = 0;
-
-    my $text = $class->zfs_request($scfg, undef, 'get', '-o', 'value', '-Hp',
-               'available,used', $scfg->{pool});
-
-    my @lines = split /\n/, $text;
-
-    if($lines[0] =~ /^(\d+)$/) {
-       $available = $1;
-    }
-
-    if($lines[1] =~ /^(\d+)$/) {
-       $used = $1;
-    }
-
-    return ($available, $used);
-}
-
-sub zfs_get_zvol_size {
-    my ($class, $scfg, $zvol) = @_;
-
-    my $text = $class->zfs_request($scfg, undef, 'get', '-Hp', 'volsize', "$scfg->{pool}/$zvol");
-
-    if ($text =~ /volsize\s(\d+)/) {
-       return $1;
-    }
-
-    die "Could not get zvol size";
-}
-
-sub zfs_create_zvol {
-    my ($class, $scfg, $zvol, $size) = @_;
-    
-    my $cmd = ['create'];
-
-    push @$cmd, '-s' if $scfg->{sparse};
-
-    push @$cmd, '-b', $scfg->{blocksize} if $scfg->{blocksize};
-
-    push @$cmd, '-V', "${size}k", "$scfg->{pool}/$zvol";
-
-    $class->zfs_request($scfg, undef, @$cmd);
-}
-
-sub zfs_delete_zvol {
-    my ($class, $scfg, $zvol) = @_;
-
-    $class->zfs_request($scfg, undef, 'destroy', '-r', "$scfg->{pool}/$zvol");
-}
-
-sub zfs_list_zvol {
-    my ($class, $scfg) = @_;
-
-    my $text = $class->zfs_request($scfg, 10, 'list', '-o', 'name,volsize,origin', '-t', 'volume', '-Hr');
-    my $zvols = zfs_parse_zvol_list($text);
-    return undef if !$zvols;
-
-    my $list = ();
-    foreach my $zvol (@$zvols) {
-       my @values = split('/', $zvol->{name});
-
-       my $image = pop @values;
-       my $pool = join('/', @values);
-
-       next if $image !~ m/^((vm|base)-(\d+)-\S+)$/;
-       my $owner = $3;
-
-       my $parent = $zvol->{origin};
-       if($zvol->{origin} && $zvol->{origin} =~ m/^$scfg->{pool}\/(\S+)$/){
-           $parent = $1;
-       }
-
-       $list->{$pool}->{$image} = {
-           name => $image,
-           size => $zvol->{size},
-           parent => $parent,
-           format => 'raw',
-            vmid => $owner
-        };
-    }
-
-    return $list;
-}
-
-sub zfs_find_free_diskname {
-    my ($class, $storeid, $scfg, $vmid) = @_;
-
-    my $name = undef;
-    my $volumes = $class->zfs_list_zvol($scfg);
-
-    my $disk_ids = {};
-    my $dat = $volumes->{$scfg->{pool}};
-
-    foreach my $image (keys %$dat) {
-        my $volname = $dat->{$image}->{name};
-        if ($volname =~ m/(vm|base)-$vmid-disk-(\d+)/){
-            $disk_ids->{$2} = 1;
-        }
-    }
-
-    for (my $i = 1; $i < 100; $i++) {
-        if (!$disk_ids->{$i}) {
-            return "vm-$vmid-disk-$i";
-        }
-    }
-
-    die "unable to allocate an image name for VM $vmid in storage '$storeid'\n";
-}
-
-sub zfs_get_latest_snapshot {
-    my ($class, $scfg, $volname) = @_;
-
-    # abort rollback if snapshot is not the latest
-    my @params = ('-t', 'snapshot', '-o', 'name', '-s', 'creation');
-    my $text = zfs_request($class, $scfg, undef, 'list', @params);
-    my @snapshots = split(/\n/, $text);
-
-    my $recentsnap;
-    foreach (@snapshots) {
-        if (/$scfg->{pool}\/$volname/) {
-            s/^.*@//;
-            $recentsnap = $_;
-        }
-    }
-
-    return $recentsnap;
-}
-
-sub status {
-    my ($class, $storeid, $scfg, $cache) = @_;
-
-    my $total = 0;
-    my $free = 0;
-    my $used = 0;
-    my $active = 0;
-
-    eval {
-       ($free, $used) = $class->zfs_get_pool_stats($scfg);
-       $active = 1;
-       $total = $free + $used;
-    };
-    warn $@ if $@;
-
-    return ($total, $free, $used, $active);
-}
-
-sub volume_size_info {
-    my ($class, $scfg, $storeid, $volname, $timeout) = @_;
-
-    return $class->zfs_get_zvol_size($scfg, $volname);
-}
-
-sub volume_snapshot {
-    my ($class, $scfg, $storeid, $volname, $snap, $running) = @_;
-
-    $class->zfs_request($scfg, undef, 'snapshot', "$scfg->{pool}/$volname\@$snap");
-}
-
-sub volume_snapshot_delete {
-    my ($class, $scfg, $storeid, $volname, $snap, $running) = @_;
-
-    $class->zfs_request($scfg, undef, 'destroy', "$scfg->{pool}/$volname\@$snap");
-}
-
-sub volume_snapshot_rollback {
-    my ($class, $scfg, $storeid, $volname, $snap) = @_;
-
-    # abort rollback if snapshot is not the latest
-    my $recentsnap = $class->zfs_get_latest_snapshot($scfg, $volname);
-    if ($snap ne $recentsnap) {
-        die "cannot rollback, more recent snapshots exist\n";
-    }
-
-    zfs_request($class, $scfg, undef, 'rollback', "$scfg->{pool}/$volname\@$snap");
-}
-
-sub activate_volume {
-    my ($class, $storeid, $scfg, $volname, $exclusive, $cache) = @_;
-    return 1;
-}
-
-sub deactivate_volume {
-    my ($class, $storeid, $scfg, $volname, $exclusive, $cache) = @_;
-    return 1;
-}
-
-sub clone_image {
-    my ($class, $scfg, $storeid, $volname, $vmid, $snap) = @_;
-
-    $snap ||= '__base__';
-
-    my ($vtype, $basename, $basevmid, undef, undef, $isBase) =
-        $class->parse_volname($volname);
-
-    die "clone_image only works on base images\n" if !$isBase;
-
-    my $name = $class->zfs_find_free_diskname($storeid, $scfg, $vmid);
-
-    $class->zfs_request($scfg, undef, 'clone', "$scfg->{pool}/$basename\@$snap", "$scfg->{pool}/$name");
-
-    return $name;
-}
-
-sub create_base {
-    my ($class, $storeid, $scfg, $volname) = @_;
-
-    my $snap = '__base__';
-
-    my ($vtype, $name, $vmid, $basename, $basevmid, $isBase) =
-        $class->parse_volname($volname);
-
-    die "create_base not possible with base image\n" if $isBase;
-
-    my $newname = $name;
-    $newname =~ s/^vm-/base-/;
-
-    my $newvolname = $basename ? "$basename/$newname" : "$newname";
-
-    $class->zfs_request($scfg, undef, 'rename', "$scfg->{pool}/$name", "$scfg->{pool}/$newname");
-
-    my $running  = undef; #fixme : is create_base always offline ?
-
-    $class->volume_snapshot($scfg, $storeid, $newname, $snap, $running);
-
-    return $newvolname;
-}
-
-sub volume_has_feature {
-    my ($class, $scfg, $feature, $storeid, $volname, $snapname, $running) = @_;
-
-    my $features = {
-       snapshot => { current => 1, snap => 1},
-       clone => { base => 1},
-       template => { current => 1},
-       copy => { base => 1, current => 1},
-    };
-
-    my ($vtype, $name, $vmid, $basename, $basevmid, $isBase) =
-       $class->parse_volname($volname);
-
-    my $key = undef;
-
-    if ($snapname) {
-       $key = 'snap';
-    } else {
-       $key = $isBase ? 'base' : 'current';
-    }
-
-    return 1 if $features->{$feature}->{$key};
-
-    return undef;
-}
-
-1;
index fab68cb79c9ef2dbb44d3ebac3586b2118523372..30215d4f3a4de48c38a82eff06ab02efef547126 100644 (file)
@@ -5,9 +5,9 @@ use warnings;
 use IO::File;
 use POSIX;
 use PVE::Tools qw(run_command);
-use PVE::Storage::ZFSDirPlugin;
+use PVE::Storage::ZFSPoolPlugin;
 
-use base qw(PVE::Storage::ZFSDirPlugin);
+use base qw(PVE::Storage::ZFSPoolPlugin);
 use PVE::Storage::LunCmd::Comstar;
 use PVE::Storage::LunCmd::Istgt;
 use PVE::Storage::LunCmd::Iet;
diff --git a/PVE/Storage/ZFSPoolPlugin.pm b/PVE/Storage/ZFSPoolPlugin.pm
new file mode 100644 (file)
index 0000000..40cd753
--- /dev/null
@@ -0,0 +1,497 @@
+package PVE::Storage::ZFSPoolPlugin;
+
+use strict;
+use warnings;
+use IO::File;
+use POSIX;
+use PVE::Tools qw(run_command);
+use PVE::Storage::Plugin;
+
+
+use base qw(PVE::Storage::Plugin);
+
+sub type {
+    return 'zfspool';
+}
+
+sub plugindata {
+    return {
+       content => [ {images => 1}, { images => 1 }],
+    };
+}
+
+sub properties {
+    return {
+       blocksize => {
+           description => "block size",
+           type => 'string',
+       },
+       sparse => {
+           description => "use sparse volumes",
+           type => 'boolean',
+       },
+    };
+}
+
+sub options {
+    return {
+       pool => { fixed => 1 },
+       blocksize => { optional => 1 },
+       sparse => { optional => 1 },
+       nodes => { optional => 1 },
+       disable => { optional => 1 },
+        maxfiles => { optional => 1 },
+       content => { optional => 1 },
+    };
+}
+
+# static zfs helper methods
+
+sub zfs_parse_size {
+    my ($text) = @_;
+
+    return 0 if !$text;
+    
+    if ($text =~ m/^(\d+(\.\d+)?)([TGMK])?$/) {
+
+       my ($size, $reminder, $unit) = ($1, $2, $3);
+       
+       if ($unit) {
+           if ($unit eq 'K') {
+               $size *= 1024;
+           } elsif ($unit eq 'M') {
+               $size *= 1024*1024;
+           } elsif ($unit eq 'G') {
+               $size *= 1024*1024*1024;
+           } elsif ($unit eq 'T') {
+               $size *= 1024*1024*1024*1024;
+           } else {
+               die "got unknown zfs size unit '$unit'\n";
+           }
+       }
+
+       if ($reminder) {
+           $size = ceil($size);
+       }
+       
+       return $size;
+    
+    }
+
+    warn "unable to parse zfs size '$text'\n";
+
+    return 0;
+}
+
+sub zfs_parse_zvol_list {
+    my ($text) = @_;
+
+    my $list = ();
+
+    return $list if !$text;
+
+    my @lines = split /\n/, $text;
+    foreach my $line (@lines) {
+       if ($line =~ /^(.+)\s+([a-zA-Z0-9\.]+|\-)\s+(.+)$/) {
+           my $zvol = {};
+           my @parts = split /\//, $1;
+           my $name = pop @parts;
+           my $pool = join('/', @parts);
+
+           if ($pool !~ /^rpool$/) {
+               next unless $name =~ m!^(\w+)-(\d+)-(\w+)-(\d+)$!;
+               $name = $pool . '/' . $name;
+           } else {
+               next;
+           }
+
+           $zvol->{pool} = $pool;
+           $zvol->{name} = $name;
+           $zvol->{size} = zfs_parse_size($2);
+           if ($3 !~ /^-$/) {
+               $zvol->{origin} = $3;
+           }
+           push @$list, $zvol;
+       }
+    }
+
+    return $list;
+}
+
+sub parse_volname {
+    my ($class, $volname) = @_;
+
+    if ($volname =~ m/^(((base|vm)-(\d+)-\S+)\/)?((base)?(vm)?-(\d+)-\S+)$/) {
+       return ('images', $5, $8, $2, $4, $6);
+    }
+
+    die "unable to parse zfs volume name '$volname'\n";
+}
+
+# virtual zfs methods (subclass can overwrite them)
+
+sub path {
+    my ($class, $scfg, $volname) = @_;
+
+    my ($vtype, $name, $vmid) = $class->parse_volname($volname);
+
+    my $path = '';
+
+    if($vtype eq "images"){
+       $path = "/dev/zvol/$scfg->{pool}/$volname";
+    } else {
+       die "$vtype is not allowed in ZFSPool!";
+    }
+
+    return ($path, $vmid, $vtype);
+}
+
+sub zfs_request {
+    my ($class, $scfg, $timeout, $method, @params) = @_;
+
+    $timeout = 5 if !$timeout;
+
+    my $cmd = [];
+
+    if ($method eq 'zpool_list') {
+       push @$cmd = 'zpool', 'list';
+    } else {
+       push @$cmd, 'zfs', $method;
+    }
+
+    push @$cmd, @params;
+    my $msg = '';
+
+    my $output = sub {
+        my $line = shift;
+        $msg .= "$line\n";
+    };
+
+    run_command($cmd, outfunc => $output, timeout => $timeout);
+
+    return $msg;
+}
+
+sub alloc_image {
+    my ($class, $storeid, $scfg, $vmid, $fmt, $name, $size) = @_;
+
+    die "unsupported format '$fmt'" if $fmt ne 'raw';
+
+    die "illegal name '$name' - sould be 'vm-$vmid-*'\n"
+    if $name && $name !~ m/^vm-$vmid-/;
+
+    $name = $class->zfs_find_free_diskname($storeid, $scfg, $vmid) if !$name;
+    
+    $class->zfs_create_zvol($scfg, $name, $size);
+
+    return $name;
+}
+
+sub free_image {
+    my ($class, $storeid, $scfg, $volname, $isBase) = @_;
+
+    my (undef, $name, undef) = $class->parse_volname($volname);
+
+    $class->zfs_delete_zvol($scfg, $name);
+
+    return undef;
+}
+
+sub list_images {
+    my ($class, $storeid, $scfg, $vmid, $vollist, $cache) = @_;
+
+    $cache->{zfs} = $class->zfs_list_zvol($scfg) if !$cache->{zfs};
+    my $zfspool = $scfg->{pool};
+    my $res = [];
+
+    if (my $dat = $cache->{zfs}->{$zfspool}) {
+
+       foreach my $image (keys %$dat) {
+
+           my $volname = $dat->{$image}->{name};
+           my $parent = $dat->{$image}->{parent};
+
+           my $volid = undef;
+            if ($parent && $parent =~ m/^(\S+)@(\S+)$/) {
+               my ($basename) = ($1);
+               $volid = "$storeid:$basename/$volname";
+           } else {
+               $volid = "$storeid:$volname";
+           }
+
+           my $owner = $dat->{$volname}->{vmid};
+           if ($vollist) {
+               my $found = grep { $_ eq $volid } @$vollist;
+               next if !$found;
+           } else {
+               next if defined ($vmid) && ($owner ne $vmid);
+           }
+
+           my $info = $dat->{$volname};
+           $info->{volid} = $volid;
+           push @$res, $info;
+       }
+    }
+
+    return $res;
+}
+
+sub zfs_get_pool_stats {
+    my ($class, $scfg) = @_;
+
+    my $available = 0;
+    my $used = 0;
+
+    my $text = $class->zfs_request($scfg, undef, 'get', '-o', 'value', '-Hp',
+               'available,used', $scfg->{pool});
+
+    my @lines = split /\n/, $text;
+
+    if($lines[0] =~ /^(\d+)$/) {
+       $available = $1;
+    }
+
+    if($lines[1] =~ /^(\d+)$/) {
+       $used = $1;
+    }
+
+    return ($available, $used);
+}
+
+sub zfs_get_zvol_size {
+    my ($class, $scfg, $zvol) = @_;
+
+    my $text = $class->zfs_request($scfg, undef, 'get', '-Hp', 'volsize', "$scfg->{pool}/$zvol");
+
+    if ($text =~ /volsize\s(\d+)/) {
+       return $1;
+    }
+
+    die "Could not get zvol size";
+}
+
+sub zfs_create_zvol {
+    my ($class, $scfg, $zvol, $size) = @_;
+    
+    my $cmd = ['create'];
+
+    push @$cmd, '-s' if $scfg->{sparse};
+
+    push @$cmd, '-b', $scfg->{blocksize} if $scfg->{blocksize};
+
+    push @$cmd, '-V', "${size}k", "$scfg->{pool}/$zvol";
+
+    $class->zfs_request($scfg, undef, @$cmd);
+}
+
+sub zfs_delete_zvol {
+    my ($class, $scfg, $zvol) = @_;
+
+    $class->zfs_request($scfg, undef, 'destroy', '-r', "$scfg->{pool}/$zvol");
+}
+
+sub zfs_list_zvol {
+    my ($class, $scfg) = @_;
+
+    my $text = $class->zfs_request($scfg, 10, 'list', '-o', 'name,volsize,origin', '-t', 'volume', '-Hr');
+    my $zvols = zfs_parse_zvol_list($text);
+    return undef if !$zvols;
+
+    my $list = ();
+    foreach my $zvol (@$zvols) {
+       my @values = split('/', $zvol->{name});
+
+       my $image = pop @values;
+       my $pool = join('/', @values);
+
+       next if $image !~ m/^((vm|base)-(\d+)-\S+)$/;
+       my $owner = $3;
+
+       my $parent = $zvol->{origin};
+       if($zvol->{origin} && $zvol->{origin} =~ m/^$scfg->{pool}\/(\S+)$/){
+           $parent = $1;
+       }
+
+       $list->{$pool}->{$image} = {
+           name => $image,
+           size => $zvol->{size},
+           parent => $parent,
+           format => 'raw',
+            vmid => $owner
+        };
+    }
+
+    return $list;
+}
+
+sub zfs_find_free_diskname {
+    my ($class, $storeid, $scfg, $vmid) = @_;
+
+    my $name = undef;
+    my $volumes = $class->zfs_list_zvol($scfg);
+
+    my $disk_ids = {};
+    my $dat = $volumes->{$scfg->{pool}};
+
+    foreach my $image (keys %$dat) {
+        my $volname = $dat->{$image}->{name};
+        if ($volname =~ m/(vm|base)-$vmid-disk-(\d+)/){
+            $disk_ids->{$2} = 1;
+        }
+    }
+
+    for (my $i = 1; $i < 100; $i++) {
+        if (!$disk_ids->{$i}) {
+            return "vm-$vmid-disk-$i";
+        }
+    }
+
+    die "unable to allocate an image name for VM $vmid in storage '$storeid'\n";
+}
+
+sub zfs_get_latest_snapshot {
+    my ($class, $scfg, $volname) = @_;
+
+    # abort rollback if snapshot is not the latest
+    my @params = ('-t', 'snapshot', '-o', 'name', '-s', 'creation');
+    my $text = zfs_request($class, $scfg, undef, 'list', @params);
+    my @snapshots = split(/\n/, $text);
+
+    my $recentsnap;
+    foreach (@snapshots) {
+        if (/$scfg->{pool}\/$volname/) {
+            s/^.*@//;
+            $recentsnap = $_;
+        }
+    }
+
+    return $recentsnap;
+}
+
+sub status {
+    my ($class, $storeid, $scfg, $cache) = @_;
+
+    my $total = 0;
+    my $free = 0;
+    my $used = 0;
+    my $active = 0;
+
+    eval {
+       ($free, $used) = $class->zfs_get_pool_stats($scfg);
+       $active = 1;
+       $total = $free + $used;
+    };
+    warn $@ if $@;
+
+    return ($total, $free, $used, $active);
+}
+
+sub volume_size_info {
+    my ($class, $scfg, $storeid, $volname, $timeout) = @_;
+
+    return $class->zfs_get_zvol_size($scfg, $volname);
+}
+
+sub volume_snapshot {
+    my ($class, $scfg, $storeid, $volname, $snap, $running) = @_;
+
+    $class->zfs_request($scfg, undef, 'snapshot', "$scfg->{pool}/$volname\@$snap");
+}
+
+sub volume_snapshot_delete {
+    my ($class, $scfg, $storeid, $volname, $snap, $running) = @_;
+
+    $class->zfs_request($scfg, undef, 'destroy', "$scfg->{pool}/$volname\@$snap");
+}
+
+sub volume_snapshot_rollback {
+    my ($class, $scfg, $storeid, $volname, $snap) = @_;
+
+    # abort rollback if snapshot is not the latest
+    my $recentsnap = $class->zfs_get_latest_snapshot($scfg, $volname);
+    if ($snap ne $recentsnap) {
+        die "cannot rollback, more recent snapshots exist\n";
+    }
+
+    zfs_request($class, $scfg, undef, 'rollback', "$scfg->{pool}/$volname\@$snap");
+}
+
+sub activate_volume {
+    my ($class, $storeid, $scfg, $volname, $exclusive, $cache) = @_;
+    return 1;
+}
+
+sub deactivate_volume {
+    my ($class, $storeid, $scfg, $volname, $exclusive, $cache) = @_;
+    return 1;
+}
+
+sub clone_image {
+    my ($class, $scfg, $storeid, $volname, $vmid, $snap) = @_;
+
+    $snap ||= '__base__';
+
+    my ($vtype, $basename, $basevmid, undef, undef, $isBase) =
+        $class->parse_volname($volname);
+
+    die "clone_image only works on base images\n" if !$isBase;
+
+    my $name = $class->zfs_find_free_diskname($storeid, $scfg, $vmid);
+
+    $class->zfs_request($scfg, undef, 'clone', "$scfg->{pool}/$basename\@$snap", "$scfg->{pool}/$name");
+
+    return $name;
+}
+
+sub create_base {
+    my ($class, $storeid, $scfg, $volname) = @_;
+
+    my $snap = '__base__';
+
+    my ($vtype, $name, $vmid, $basename, $basevmid, $isBase) =
+        $class->parse_volname($volname);
+
+    die "create_base not possible with base image\n" if $isBase;
+
+    my $newname = $name;
+    $newname =~ s/^vm-/base-/;
+
+    my $newvolname = $basename ? "$basename/$newname" : "$newname";
+
+    $class->zfs_request($scfg, undef, 'rename', "$scfg->{pool}/$name", "$scfg->{pool}/$newname");
+
+    my $running  = undef; #fixme : is create_base always offline ?
+
+    $class->volume_snapshot($scfg, $storeid, $newname, $snap, $running);
+
+    return $newvolname;
+}
+
+sub volume_has_feature {
+    my ($class, $scfg, $feature, $storeid, $volname, $snapname, $running) = @_;
+
+    my $features = {
+       snapshot => { current => 1, snap => 1},
+       clone => { base => 1},
+       template => { current => 1},
+       copy => { base => 1, current => 1},
+    };
+
+    my ($vtype, $name, $vmid, $basename, $basevmid, $isBase) =
+       $class->parse_volname($volname);
+
+    my $key = undef;
+
+    if ($snapname) {
+       $key = 'snap';
+    } else {
+       $key = $isBase ? 'base' : 'current';
+    }
+
+    return 1 if $features->{$feature}->{$key};
+
+    return undef;
+}
+
+1;