From c0f1820b60c8349e658bd2e70868e734961e510a Mon Sep 17 00:00:00 2001
From: "ChangBo Guo(gcb)" <eric.guo@easystack.cn>
Date: Mon, 24 Nov 2014 14:13:01 +0800
Subject: [PATCH] Don't use module importutils from oslo-incubator

* Syncs latest module service
  Removes its dependency for oslo-incubator module importutils.
  Removes code which was never executed after switched to oslo.messaging.
Include changes:
5d40e14 Remove code that moved to oslo.i18n
6ede600 rpc, notifier: remove deprecated modules

* config.genrator has been removed from oslo-incubator, so update
config.generator to use importutils from oslo.utils directly.

* Removes module importutils

Change-Id: If5cf4100006b5ca4dba1ad92e0c8efc22f90083c
---
 cinder/openstack/common/config/generator.py |  2 +-
 cinder/openstack/common/importutils.py      | 73 ---------------------
 cinder/openstack/common/service.py          | 10 +--
 3 files changed, 2 insertions(+), 83 deletions(-)
 delete mode 100644 cinder/openstack/common/importutils.py

diff --git a/cinder/openstack/common/config/generator.py b/cinder/openstack/common/config/generator.py
index 4da102da39a..bb20dfb27c7 100644
--- a/cinder/openstack/common/config/generator.py
+++ b/cinder/openstack/common/config/generator.py
@@ -28,11 +28,11 @@ import sys
 import textwrap
 
 from oslo.config import cfg
+from oslo.utils import importutils
 import six
 import stevedore.named
 
 from cinder.openstack.common import gettextutils
-from cinder.openstack.common import importutils
 
 gettextutils.install('cinder')
 
diff --git a/cinder/openstack/common/importutils.py b/cinder/openstack/common/importutils.py
deleted file mode 100644
index 9183a946834..00000000000
--- a/cinder/openstack/common/importutils.py
+++ /dev/null
@@ -1,73 +0,0 @@
-# Copyright 2011 OpenStack Foundation.
-# All Rights Reserved.
-#
-#    Licensed under the Apache License, Version 2.0 (the "License"); you may
-#    not use this file except in compliance with the License. You may obtain
-#    a copy of the License at
-#
-#         http://www.apache.org/licenses/LICENSE-2.0
-#
-#    Unless required by applicable law or agreed to in writing, software
-#    distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
-#    WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
-#    License for the specific language governing permissions and limitations
-#    under the License.
-
-"""
-Import related utilities and helper functions.
-"""
-
-import sys
-import traceback
-
-
-def import_class(import_str):
-    """Returns a class from a string including module and class."""
-    mod_str, _sep, class_str = import_str.rpartition('.')
-    __import__(mod_str)
-    try:
-        return getattr(sys.modules[mod_str], class_str)
-    except AttributeError:
-        raise ImportError('Class %s cannot be found (%s)' %
-                          (class_str,
-                           traceback.format_exception(*sys.exc_info())))
-
-
-def import_object(import_str, *args, **kwargs):
-    """Import a class and return an instance of it."""
-    return import_class(import_str)(*args, **kwargs)
-
-
-def import_object_ns(name_space, import_str, *args, **kwargs):
-    """Tries to import object from default namespace.
-
-    Imports a class and return an instance of it, first by trying
-    to find the class in a default namespace, then failing back to
-    a full path if not found in the default namespace.
-    """
-    import_value = "%s.%s" % (name_space, import_str)
-    try:
-        return import_class(import_value)(*args, **kwargs)
-    except ImportError:
-        return import_class(import_str)(*args, **kwargs)
-
-
-def import_module(import_str):
-    """Import a module."""
-    __import__(import_str)
-    return sys.modules[import_str]
-
-
-def import_versioned_module(version, submodule=None):
-    module = 'cinder.v%s' % version
-    if submodule:
-        module = '.'.join((module, submodule))
-    return import_module(module)
-
-
-def try_import(import_str, default=None):
-    """Try to import a module and if it fails return default."""
-    try:
-        return import_module(import_str)
-    except ImportError:
-        return default
diff --git a/cinder/openstack/common/service.py b/cinder/openstack/common/service.py
index a63d38b3e23..e2c57d7d098 100644
--- a/cinder/openstack/common/service.py
+++ b/cinder/openstack/common/service.py
@@ -38,14 +38,12 @@ from eventlet import event
 from oslo.config import cfg
 
 from cinder.openstack.common import eventlet_backdoor
-from cinder.openstack.common.gettextutils import _LE, _LI, _LW
-from cinder.openstack.common import importutils
+from cinder.openstack.common._i18n import _LE, _LI, _LW
 from cinder.openstack.common import log as logging
 from cinder.openstack.common import systemd
 from cinder.openstack.common import threadgroup
 
 
-rpc = importutils.try_import('cinder.openstack.common.rpc')
 CONF = cfg.CONF
 LOG = logging.getLogger(__name__)
 
@@ -180,12 +178,6 @@ class ServiceLauncher(Launcher):
             status = exc.code
         finally:
             self.stop()
-            if rpc:
-                try:
-                    rpc.cleanup()
-                except Exception:
-                    # We're shutting down, so it doesn't matter at this point.
-                    LOG.exception(_LE('Exception during rpc cleanup.'))
 
         return status, signo