author | david.comay@oracle.com |
Fri, 24 Apr 2015 10:52:04 -0700 | |
changeset 4188 | 23e54414e57c |
permissions | -rw-r--r-- |
4188
23e54414e57c
20884839 san.py (used by ZFSSA driver) needs to account for lack of paramiko
david.comay@oracle.com
parents:
diff
changeset
|
1 |
This internal-only patch is to prevent Cinder's SanDriver from |
23e54414e57c
20884839 san.py (used by ZFSSA driver) needs to account for lack of paramiko
david.comay@oracle.com
parents:
diff
changeset
|
2 |
attempting to use the Paramiko-based routines until the latter is |
23e54414e57c
20884839 san.py (used by ZFSSA driver) needs to account for lack of paramiko
david.comay@oracle.com
parents:
diff
changeset
|
3 |
integrated into Solaris. It also provides a specific error in the case |
23e54414e57c
20884839 san.py (used by ZFSSA driver) needs to account for lack of paramiko
david.comay@oracle.com
parents:
diff
changeset
|
4 |
where san_is_local=false is defined in the configuration and a driver |
23e54414e57c
20884839 san.py (used by ZFSSA driver) needs to account for lack of paramiko
david.comay@oracle.com
parents:
diff
changeset
|
5 |
is specified that attempts to use that configuration (which implies, |
23e54414e57c
20884839 san.py (used by ZFSSA driver) needs to account for lack of paramiko
david.comay@oracle.com
parents:
diff
changeset
|
6 |
again, an attempt to use Paramiko). |
23e54414e57c
20884839 san.py (used by ZFSSA driver) needs to account for lack of paramiko
david.comay@oracle.com
parents:
diff
changeset
|
7 |
|
23e54414e57c
20884839 san.py (used by ZFSSA driver) needs to account for lack of paramiko
david.comay@oracle.com
parents:
diff
changeset
|
8 |
--- cinder-2014.2.2/cinder/volume/drivers/san/san.py.~1~ 2015-02-05 08:03:26.000000000 -0800 |
23e54414e57c
20884839 san.py (used by ZFSSA driver) needs to account for lack of paramiko
david.comay@oracle.com
parents:
diff
changeset
|
9 |
+++ cinder-2014.2.2/cinder/volume/drivers/san/san.py 2015-04-22 00:54:38.481125902 -0700 |
23e54414e57c
20884839 san.py (used by ZFSSA driver) needs to account for lack of paramiko
david.comay@oracle.com
parents:
diff
changeset
|
10 |
@@ -29,7 +29,6 @@ from cinder.i18n import _ |
23e54414e57c
20884839 san.py (used by ZFSSA driver) needs to account for lack of paramiko
david.comay@oracle.com
parents:
diff
changeset
|
11 |
from cinder.openstack.common import excutils |
23e54414e57c
20884839 san.py (used by ZFSSA driver) needs to account for lack of paramiko
david.comay@oracle.com
parents:
diff
changeset
|
12 |
from cinder.openstack.common import log as logging |
23e54414e57c
20884839 san.py (used by ZFSSA driver) needs to account for lack of paramiko
david.comay@oracle.com
parents:
diff
changeset
|
13 |
from cinder.openstack.common import processutils |
23e54414e57c
20884839 san.py (used by ZFSSA driver) needs to account for lack of paramiko
david.comay@oracle.com
parents:
diff
changeset
|
14 |
-from cinder import ssh_utils |
23e54414e57c
20884839 san.py (used by ZFSSA driver) needs to account for lack of paramiko
david.comay@oracle.com
parents:
diff
changeset
|
15 |
from cinder import utils |
23e54414e57c
20884839 san.py (used by ZFSSA driver) needs to account for lack of paramiko
david.comay@oracle.com
parents:
diff
changeset
|
16 |
from cinder.volume import driver |
23e54414e57c
20884839 san.py (used by ZFSSA driver) needs to account for lack of paramiko
david.comay@oracle.com
parents:
diff
changeset
|
17 |
|
23e54414e57c
20884839 san.py (used by ZFSSA driver) needs to account for lack of paramiko
david.comay@oracle.com
parents:
diff
changeset
|
18 |
@@ -165,10 +164,8 @@ class SanDriver(driver.VolumeDriver): |
23e54414e57c
20884839 san.py (used by ZFSSA driver) needs to account for lack of paramiko
david.comay@oracle.com
parents:
diff
changeset
|
19 |
def check_for_setup_error(self): |
23e54414e57c
20884839 san.py (used by ZFSSA driver) needs to account for lack of paramiko
david.comay@oracle.com
parents:
diff
changeset
|
20 |
"""Returns an error if prerequisites aren't met.""" |
23e54414e57c
20884839 san.py (used by ZFSSA driver) needs to account for lack of paramiko
david.comay@oracle.com
parents:
diff
changeset
|
21 |
if not self.run_local: |
23e54414e57c
20884839 san.py (used by ZFSSA driver) needs to account for lack of paramiko
david.comay@oracle.com
parents:
diff
changeset
|
22 |
- if not (self.configuration.san_password or |
23e54414e57c
20884839 san.py (used by ZFSSA driver) needs to account for lack of paramiko
david.comay@oracle.com
parents:
diff
changeset
|
23 |
- self.configuration.san_private_key): |
23e54414e57c
20884839 san.py (used by ZFSSA driver) needs to account for lack of paramiko
david.comay@oracle.com
parents:
diff
changeset
|
24 |
- raise exception.InvalidInput( |
23e54414e57c
20884839 san.py (used by ZFSSA driver) needs to account for lack of paramiko
david.comay@oracle.com
parents:
diff
changeset
|
25 |
- reason=_('Specify san_password or san_private_key')) |
23e54414e57c
20884839 san.py (used by ZFSSA driver) needs to account for lack of paramiko
david.comay@oracle.com
parents:
diff
changeset
|
26 |
+ raise exception.InvalidInput( |
23e54414e57c
20884839 san.py (used by ZFSSA driver) needs to account for lack of paramiko
david.comay@oracle.com
parents:
diff
changeset
|
27 |
+ reason=_("san_is_local=false is not currently supported.")) |
23e54414e57c
20884839 san.py (used by ZFSSA driver) needs to account for lack of paramiko
david.comay@oracle.com
parents:
diff
changeset
|
28 |
|
23e54414e57c
20884839 san.py (used by ZFSSA driver) needs to account for lack of paramiko
david.comay@oracle.com
parents:
diff
changeset
|
29 |
# The san_ip must always be set, because we use it for the target |
23e54414e57c
20884839 san.py (used by ZFSSA driver) needs to account for lack of paramiko
david.comay@oracle.com
parents:
diff
changeset
|
30 |
if not self.configuration.san_ip: |