SanISCSIDriver SSH execution fixes.

In ssh_execute methon in nova/utils there was a problem with cmd passed to it.
From SanISCSIDriver and all other places it is called from cmd is a string.
But in ssh_execute method cmd was processed as a list. This can make problems
with SanISCSIdriver using.

Also it may be useful to pass execute parameter to this SanISCSIDriver class
to have the opportunity of overriding _execute method (as it is in
VolumeDriver class, that is the root of this class hierarchy). That's why
special checking was added.

Fixes bug 1070489

Change-Id: I73f74f9d095a2c4316cab88148afe6a0bde44c5b

1 files changed, 5 insertions, 3 deletions

diff --git a/nova/volume/san.py b/nova/volume/san.py
index 22147141a..cf4507f31 100644
--- a/nova/volume/san.py
+++ b/nova/volume/san.py
@@ -86,8 +86,10 @@ class SanISCSIDriver(nova.volume.driver.ISCSIDriver):
     remote protocol.
     """
 
-    def __init__(self, *args, **kwargs):
-        super(SanISCSIDriver, self).__init__(*args, **kwargs)
+    def __init__(self, execute=None, *args, **kwargs):
+        if execute is None:
+            execute = self._execute
+        super(SanISCSIDriver, self).__init__(execute, *args, **kwargs)
         self.run_local = FLAGS.san_is_local
 
     def _build_iscsi_target_name(self, volume):
@@ -119,7 +121,7 @@ class SanISCSIDriver(nova.volume.driver.ISCSIDriver):
         if self.run_local:
             return utils.execute(*cmd, **kwargs)
         else:
-            check_exit_code = kwargs.pop('check_exit_code', None)
+            check_exit_code = kwargs.pop('check_exit_code', True)
             command = ' '.join(cmd)
             return self._run_ssh(command, check_exit_code)