diff options
| author | Javeme <zhangmei.li@easystack.cn> | 2016-01-14 20:35:52 +0800 |
|---|---|---|
| committer | Javeme <zhangmei.li@easystack.cn> | 2016-01-14 20:35:52 +0800 |
| commit | 564e5ca578ca4c6dfe538b1c22472772b85bb6e6 (patch) | |
| tree | d02a28609b65358e2cf6db5e60ffe7813b598e4f /oslo_vmware/api.py | |
| parent | a89ac06029d306fc23095b1dbb317b2751b07075 (diff) | |
| download | oslo-vmware-564e5ca578ca4c6dfe538b1c22472772b85bb6e6.tar.gz | |
Use oslo_utils reflection to get function name
In order to get a better name, let's use oslo_utils reflection to
get function name in RetryDecorator call.
for example:
from oslo_utils import reflection
class Foo(object):
def test_fun(self):
pass
foo = Foo()
print(foo.test_fun.__name__) # "test_fun"
print(reflection.get_callable_name(foo.test_fun)) # "__main__.Foo.test_fun"
Change-Id: Ia7970b564e33ef7eafcd31fd907f4d53db1672f0
Diffstat (limited to 'oslo_vmware/api.py')
| -rw-r--r-- | oslo_vmware/api.py | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/oslo_vmware/api.py b/oslo_vmware/api.py index 4dd2752..aaeb77b 100644 --- a/oslo_vmware/api.py +++ b/oslo_vmware/api.py @@ -25,6 +25,7 @@ import logging from oslo_concurrency import lockutils from oslo_utils import excutils +from oslo_utils import reflection import six from oslo_vmware._i18n import _, _LE, _LI, _LW @@ -82,9 +83,9 @@ class RetryDecorator(object): self._sleep_time = 0 def __call__(self, f): + func_name = reflection.get_callable_name(f) def _func(*args, **kwargs): - func_name = f.__name__ result = None try: if self._retry_count: @@ -118,7 +119,7 @@ class RetryDecorator(object): def func(*args, **kwargs): loop = loopingcall.DynamicLoopingCall(_func, *args, **kwargs) evt = loop.start(periodic_interval_max=self._max_sleep_time) - LOG.debug("Waiting for function %s to return.", f.__name__) + LOG.debug("Waiting for function %s to return.", func_name) return evt.wait() return func |