]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Use get_my_ipv4 from oslo.utils
authorChangBo Guo(gcb) <eric.guo@easystack.cn>
Tue, 23 Dec 2014 08:17:49 +0000 (16:17 +0800)
committerChangBo Guo(gcb) <eric.guo@easystack.cn>
Wed, 28 Jan 2015 03:17:24 +0000 (11:17 +0800)
Remove _get_my_ip from Cinder and replace it's usages
by usages of get_my_ipv4 from netutils, because this methods
implements the same functionality.

Change-Id: I293128364dcd1a83e990d9c75555b04c1e4226fb

cinder/common/config.py

index e3761bb285b0ced1cacf26151962f83f92a49f98..6c54156136cac381641f052af7b7afc92a58c9bd 100644 (file)
@@ -28,32 +28,13 @@ stepping stone.
 import socket
 
 from oslo_config import cfg
+from oslo_utils import netutils
 
 from cinder.i18n import _
 
 
 CONF = cfg.CONF
 
-
-def _get_my_ip():
-    """
-    Returns the actual ip of the local machine.
-
-    This code figures out what source address would be used if some traffic
-    were to be sent out to some well known address on the Internet. In this
-    case, a Google DNS server is used, but the specific address does not
-    matter much.  No traffic is actually sent.
-    """
-    try:
-        csock = socket.socket(socket.AF_INET, socket.SOCK_DGRAM)
-        csock.connect(('8.8.8.8', 80))
-        (addr, _port) = csock.getsockname()
-        csock.close()
-        return addr
-    except socket.error:
-        return "127.0.0.1"
-
-
 core_opts = [
     cfg.StrOpt('api_paste_config',
                default="api-paste.ini",
@@ -71,7 +52,7 @@ CONF.register_cli_opts(debug_opts)
 
 global_opts = [
     cfg.StrOpt('my_ip',
-               default=_get_my_ip(),
+               default=netutils.get_my_ipv4(),
                help='IP address of this host'),
     cfg.StrOpt('glance_host',
                default='$my_ip',