<div dir="ltr"><div><div>Hello, community.</div><div><br></div><div>I hope, that most of your know, that a bug with name "Log debugs should not have translations" (f.e. [1], [2], [3]) was recently raised in several projects. The reason for this work is related with the following concerns [4].</div>
<div>There is a special check that is used (or will be used in some projects, where the related patches have not merged yet) for verification process (f.e. [5] or [6]). As you can see, this ([5]) verification uses the LOG name of logger in regexp and "if" cases.</div>
<div>However, there are a lot of projects where both names "LOG" and "logger" are used [7].</div><div>So I have a question about the current situation:</div><div>- Should we use one uniform name for logger or add support for both names in checks?</div>
<div><br></div><div>In my opinion, declaration of one uniform name in hacking rules is preferable, because it cleans code from useless duplicate names of one variable and allows to create one uniform check for this rule.</div>
</div><div><br></div><div>[1] <a href="https://bugs.launchpad.net/neutron/+bug/1320867">https://bugs.launchpad.net/neutron/+bug/1320867</a></div><div>[2] <a href="https://bugs.launchpad.net/swift/+bug/1318333">https://bugs.launchpad.net/swift/+bug/1318333</a></div>
<div>[3] <a href="https://bugs.launchpad.net/oslo/+bug/1317950">https://bugs.launchpad.net/oslo/+bug/1317950</a></div><div>[4] <a href="https://wiki.openstack.org/wiki/LoggingStandards#Log_Translation">https://wiki.openstack.org/wiki/LoggingStandards#Log_Translation</a></div>
<div>[5] <a href="https://github.com/openstack/nova/blob/master/nova/hacking/checks.py#L201">https://github.com/openstack/nova/blob/master/nova/hacking/checks.py#L201</a></div><div>[6] <a href="https://review.openstack.org/#/c/94255/11/heat/hacking/checks.py">https://review.openstack.org/#/c/94255/11/heat/hacking/checks.py</a></div>
<div>[7] <a href="https://github.com/openstack/heat/search?q=getLogger&type=Code">https://github.com/openstack/heat/search?q=getLogger&type=Code</a></div><br clear="all"><div><div dir="ltr">Regards,<div>Sergey.</div>
</div></div>
</div>