diff options
| author | Monty Taylor <mordred@inaugust.com> | 2012-11-21 10:53:18 -0800 |
|---|---|---|
| committer | Mark McLoughlin <markmc@redhat.com> | 2012-11-28 17:05:36 +0000 |
| commit | cf705c59615fb68cdbfe664e7827e4a275a4a282 (patch) | |
| tree | f7d8a2f674acb0d4df9339f77f3511a123f7a525 /openstack/common/rpc/common.py | |
| parent | f10b956dd5b88b048c582cadc3c22ec5360521c9 (diff) | |
| download | oslo-cf705c59615fb68cdbfe664e7827e4a275a4a282.tar.gz oslo-cf705c59615fb68cdbfe664e7827e4a275a4a282.tar.xz oslo-cf705c59615fb68cdbfe664e7827e4a275a4a282.zip | |
Make project pyflakes clean.
Added both a tox test-env for pyflakes and fixed the current pyflakes errors.
This did actually fix a couple of bugs.
The CI team has started using pyflakes on its projects and also has started
using oslo for things, so ignoring pyflakes warnings on the oslo code was
starting to get old. However, additionally, pyflakes is pretty solid, so we
should maybe consider gating on it across the board. (% locals() is the
biggest thing that we do that it doesn't like)
Change-Id: Iac1ca62db301892b7863711162fcbc74807eb24f
Diffstat (limited to 'openstack/common/rpc/common.py')
| -rw-r--r-- | openstack/common/rpc/common.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/openstack/common/rpc/common.py b/openstack/common/rpc/common.py index a95fc66..c56c9a6 100644 --- a/openstack/common/rpc/common.py +++ b/openstack/common/rpc/common.py @@ -40,7 +40,7 @@ class RPCException(Exception): try: message = self.message % kwargs - except Exception as e: + except Exception: # kwargs doesn't match a variable in the message # log the issue and the kwargs LOG.exception(_('Exception in string format operation')) @@ -258,7 +258,7 @@ def deserialize_remote_exception(conf, data): # we cannot necessarily change an exception message so we must override # the __str__ method. failure.__class__ = new_ex_type - except TypeError as e: + except TypeError: # NOTE(ameade): If a core exception then just add the traceback to the # first exception argument. failure.args = (message,) + failure.args[1:] |