]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Remove unused 3PAR driver method
authorKurt Martin <kurt.f.martin@hp.com>
Mon, 21 Apr 2014 21:59:02 +0000 (14:59 -0700)
committerKurt Martin <kurt.f.martin@hp.com>
Mon, 21 Apr 2014 21:59:02 +0000 (14:59 -0700)
Remove the "parse_create_host_error" method as it is no longer
used in the 3PAR drivers. It can safely be removed from the
3PAR common code and the unit test were no longer using the method.

Change-Id: I81898e075107b55002c2d07f5e06db76a16c3e28
Closes-bug: 1310807

cinder/volume/drivers/san/hp/hp_3par_common.py

index 884a5c92114a9bb3a0906773f3a187f1bc7e6a49..0b55b15658a1e353e503627e5b159c8b982e89a5 100644 (file)
@@ -121,11 +121,12 @@ class HP3PARCommon(object):
         2.0.5 - Fix extend volume units bug #1284368
         2.0.6 - use loopingcall.wait instead of time.sleep
         2.0.7 - Allow extend volume based on snapshot bug #1285906
-        2.0.8 - Fix detach issue for multiple hosts Bug #1288927
+        2.0.8 - Fix detach issue for multiple hosts bug #1288927
+        2.0.9 - Remove unused 3PAR driver method bug #1310807
 
     """
 
-    VERSION = "2.0.8"
+    VERSION = "2.0.9"
 
     stats = {}
 
@@ -1269,14 +1270,6 @@ class HP3PARCommon(object):
         # try again with name retrieved from 3par
         self.delete_vlun(volume, hostname)
 
-    def parse_create_host_error(self, hostname, out):
-        search_str = "already used by host "
-        if search_str in out[1]:
-            # host exists, return name used by 3par
-            hostname_3par = self.get_next_word(out[1], search_str)
-            self.hosts_naming_dict[hostname] = hostname_3par
-            return hostname_3par
-
     def build_nsp(self, portPos):
         return '%s:%s:%s' % (portPos['node'],
                              portPos['slot'],