Correct failures for check H703

The H703 "Multiple positional placeholders" rule was failing and
ignored. Now it's enforced.

Change-Id: Id6d3bc1767e264d3f07c4e14758d67e10223f02f
This commit is contained in:
Brant Knudson 2015-01-11 18:34:37 -06:00
parent c252a8095d
commit a31ed5ca4f
2 changed files with 3 additions and 4 deletions

View File

@ -1220,8 +1220,8 @@ class AuthProtocol(object):
current_mode = stat.S_IMODE(os.stat(self._signing_dirname).st_mode)
if current_mode != stat.S_IRWXU:
self._LOG.warning(
_LW('signing_dir mode is %s instead of %s'),
oct(current_mode), oct(stat.S_IRWXU))
_LW('signing_dir mode is %(mode)s instead of %(need)s'),
{'mode': oct(current_mode), 'need': oct(stat.S_IRWXU)})
else:
os.makedirs(self._signing_dirname, stat.S_IRWXU)

View File

@ -36,8 +36,7 @@ commands = oslo_debug_helper {posargs}
# F821: undefined name
# H304: no relative imports
# H405: multi line docstring summary not separated with an empty line
# H703: Multiple positional placeholders
ignore = E122,F821,H304,H405,H703
ignore = E122,F821,H304,H405
show-source = True
exclude = .venv,.tox,dist,doc,*egg,build,*openstack/common*