From: Michael Rasmussen <[email protected]>

Signed-off-by: Michael Rasmussen <[email protected]>
---
 PVE/Storage/ZFSPlugin.pm | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/PVE/Storage/ZFSPlugin.pm b/PVE/Storage/ZFSPlugin.pm
index 0ec6632..a562635 100644
--- a/PVE/Storage/ZFSPlugin.pm
+++ b/PVE/Storage/ZFSPlugin.pm
@@ -19,6 +19,7 @@ sub zfs_request {
        my $cmd;
        my $msg = '';
        my $zfs;
+       my $target;
        
        my $output = sub {
                my $line = shift;
@@ -39,9 +40,14 @@ sub zfs_request {
                die 'unknown iscsi provider. Available [comstar]';
        }
        
-       $luncmd->{cmd} = 'sudo ' . $luncmd->{cmd} if ($scfg->{sudo});
+       if ($scfg->{sudo}) {
+                       $luncmd->{cmd} = 'sudo ' . $luncmd->{cmd};
+                       $target = $scfg->{portal};
+       } else {
+                       $target = 'root@' . $scfg->{portal};
+       }
        if ($luncmd->{$method}) {
-               $cmd = [@ssh_cmd, $scfg->{portal}, $luncmd->{cmd}, 
$luncmd->{$method}, @params];
+               $cmd = [@ssh_cmd, $target, $luncmd->{cmd}, $luncmd->{$method}, 
@params];
        } else {
                if ($method =~ /^zpool-list$/) {
                        $zfs = 'zpool';
@@ -49,7 +55,7 @@ sub zfs_request {
                } else {
                        $zfs = 'zfs';
                }
-               $cmd = [@ssh_cmd, $scfg->{portal}, $zfs, $method, @params];
+               $cmd = [@ssh_cmd, $target, $zfs, $method, @params];
        }
        
        eval {
-- 
1.8.3.1

_______________________________________________
pve-devel mailing list
[email protected]
http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to