openstack/kolla
Code Issues Proposed changes

Ensure doesn't use LOG.warn

Browse Source 
This patchs ensure doesn't use LOG.warn because LOG.warn deprecated
in Python 3 [1]. Using LOG.warning should be used instead.

[1] https://docs.python.org/3/library/logging.html#logging.warning

TrivialFix

Change-Id: I464314b33b8d4f31b6be0a765019e1f5668cc6d9
This commit is contained in:
Nguyen Phuong An 2016-09-01 15:52:58 +07:00
parent fdb33d04ec
commit 23921f11eb
1 changed files with 14 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
kolla/hacking/checks.py
View File

@ -16,6 +16,19 @@ import re
mutable_default_args = re.compile(r"^\s*def .+\((.+=\{\}|.+=\[\])")
def no_log_warn(logical_line):
"""Disallow 'LOG.warn('
Deprecated LOG.warn(), instead use LOG.warning
https://bugs.launchpad.net/senlin/+bug/1508442
N352
"""
msg = ("N352: LOG.warn is deprecated, please use LOG.warning!")
if "LOG.warn(" in logical_line:
yield (0, msg)
def no_mutable_default_args(logical_line):
msg = "N301: Method's default argument shouldn't be mutable!"
if mutable_default_args.match(logical_line):
@ -24,3 +37,4 @@ def no_mutable_default_args(logical_line):
def factory(register):
register(no_mutable_default_args)
register(no_log_warn)