Use is_valid_ipv6 from oslo.utils

Remove _is_valid_ipv6 from Cinder and replace it's usages
by usages of is_valid_ipv6 from netutils, because this methods
implements the same functionality.

Change-Id: Ica16f0a34abc2f0c6f7240fe553a494d77dedc16

diff --git a/cinder/tests/integrated/api/client.py b/cinder/tests/integrated/api/client.py
index d22f78aa142d9bb905080cbca21f6e1c27e62acf..1c16306c9ab031a2f3ad21ab720fdb1dd69ea961 100644 (file)
--- a/cinder/tests/integrated/api/client.py
+++ b/cinder/tests/integrated/api/client.py
@@ -12,8 +12,8 @@
 #    License for the specific language governing permissions and limitations
 #    under the License.
 
-import netaddr
 from oslo.serialization import jsonutils
+from oslo.utils import netutils
 import requests
 import six.moves.urllib.parse as urlparse
 
@@ -88,7 +88,7 @@ class TestOpenStackClient(object):
         hostname = parsed_url.hostname
         scheme = parsed_url.scheme
 
-        if netaddr.valid_ipv6(hostname):
+        if netutils.is_valid_ipv6(hostname):
             hostname = "[%s]" % hostname
 
         relative_url = parsed_url.path

diff --git a/cinder/volume/drivers/vmware/read_write_util.py b/cinder/volume/drivers/vmware/read_write_util.py
index 38957fc35bf1962af22dc88444adc68450839096..86351713d29c1332bcb9309dfc673cce0ff74755 100644 (file)
--- a/cinder/volume/drivers/vmware/read_write_util.py
+++ b/cinder/volume/drivers/vmware/read_write_util.py
@@ -23,7 +23,7 @@ import httplib
 import urllib
 import urllib2
 
-import netaddr
+from oslo.utils import netutils
 import six.moves.urllib.parse as urlparse
 
 from cinder.i18n import _, _LI
@@ -102,16 +102,9 @@ class VMwareHTTPFile(object):
         """Get size of the file to be read."""
         raise NotImplementedError()
 
-    def _is_valid_ipv6(self, address):
-        """Whether given host address is a valid IPv6 address."""
-        try:
-            return netaddr.valid_ipv6(address)
-        except Exception:
-            return False
-
     def get_soap_url(self, scheme, host):
         """return IPv4/v6 compatible url constructed for host."""
-        if self._is_valid_ipv6(host):
+        if netutils.is_valid_ipv6(host):
             return '%s://[%s]' % (scheme, host)
         return '%s://%s' % (scheme, host)
 

diff --git a/cinder/volume/drivers/vmware/vim_util.py b/cinder/volume/drivers/vmware/vim_util.py
index 9487732430aba415705f243a8f67ba6bedd534e7..97861d8b215d34c7e8f57df3799d7357d9225c83 100644 (file)
--- a/cinder/volume/drivers/vmware/vim_util.py
+++ b/cinder/volume/drivers/vmware/vim_util.py
@@ -17,7 +17,7 @@
 The VMware API utility module.
 """
 
-import netaddr
+from oslo.utils import netutils
 
 
 def get_soap_url(protocol, host, path='sdk'):
@@ -28,7 +28,7 @@ def get_soap_url(protocol, host, path='sdk'):
     :param path: path part of the SOAP URL
     :return: URL to SOAP services for ESX/VC server
     """
-    if netaddr.valid_ipv6(host):
+    if netutils.is_valid_ipv6(host):
         return '%s://[%s]/%s' % (protocol, host, path)
     return '%s://%s/%s' % (protocol, host, path)