Merge "Use oslo.utils.reflection to extract class name"

heatclient/exc.py

@@ -13,6 +13,7 @@
 import sys
 
 from oslo_serialization import jsonutils
+from oslo_utils import reflection
 
 from heatclient.openstack.common._i18n import _
 
@@ -78,7 +79,7 @@ class HTTPMultipleChoices(HTTPException):
                          "available.")
         return (_("%(name)s (HTTP %(code)s) %(details)s") %
                 {
-                    'name': self.__class__.__name__,
+                'name': reflection.get_class_name(self, fully_qualified=False),
                 'code': self.code,
                 'details': self.details})
 

heatclient/openstack/common/apiclient/base.py

@@ -41,6 +41,7 @@ import abc
 import copy
 import logging
 
+from oslo_utils import reflection
 from oslo_utils import strutils
 import six
 from six.moves.urllib import parse
@@ -466,7 +467,8 @@ class Resource(object):
                           for k in self.__dict__.keys()
                           if k[0] != '_' and k != 'manager')
         info = ", ".join("%s=%s" % (k, getattr(self, k)) for k in reprkeys)
-        return "<%s %s>" % (self.__class__.__name__, info)
+        class_name = reflection.get_class_name(self, fully_qualified=False)
+        return "<%s %s>" % (class_name, info)
 
     @property
     def human_id(self):

heatclient/tests/unit/osc/fakes.py

@@ -16,6 +16,7 @@
 import json
 import sys
 
+from oslo_utils import reflection
 import requests
 import six
 
@@ -222,7 +223,8 @@ class FakeResource(object):
         reprkeys = sorted(k for k in self.__dict__.keys() if k[0] != '_' and
                           k != 'manager')
         info = ", ".join("%s=%s" % (k, getattr(self, k)) for k in reprkeys)
-        return "<%s %s>" % (self.__class__.__name__, info)
+        class_name = reflection.get_class_name(self, fully_qualified=False)
+        return "<%s %s>" % (class_name, info)
 
 
 class FakeResponse(requests.Response):