]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Ignore pylint error 'hashlib' has no shaxxx member
authorXing Yang <xing.yang@emc.com>
Fri, 29 Aug 2014 04:09:28 +0000 (00:09 -0400)
committerXing Yang <xing.yang@emc.com>
Fri, 29 Aug 2014 16:18:06 +0000 (12:18 -0400)
The following pylint error was found:

cinder/backup/drivers/swift.py:486: [E1101, SwiftBackupDriver.backup]
Module 'hashlib' has no 'sha256' member

Affected patch:
https://review.openstack.org/#/c/110068/

A similar pylint error "Module 'hashlib' has no 'sha224' member"
was also generated.

However, both 'sha256' and 'sha224' are functions in 'hashlib'.  So
these errors should be ignored.

Change-Id: I84690813b6c6a15e661e27acdf97149996951978

tools/lintstack.py

index 22e267f65119033fd39b42e2ac9a3f0dfc45a04b..d8459ff0e301cc8430b6034dc3306e9014f66afb 100755 (executable)
@@ -32,11 +32,16 @@ ignore_codes = ["E1103"]
 # for cinder.tests modules
 # Note(fengqian): the second error message is the pattern of [E0611].
 # It should be ignored because use six module to keep py3.X compatibility.
-# Note(e0ne): the third error messege is for SQLAlchemy update() calls
+# Note(e0ne): the third error message is for SQLAlchemy update() calls
 # in DB schema migrations.
+# Note(xyang): the fourth and fifth error messages are for the code [E1101].
+# They should be ignored because 'sha256' and 'sha224' are functions in
+# 'hashlib'.
 ignore_messages = ["An attribute affected in cinder.tests",
                    "No name 'urllib' in module '_MovedItems'",
-                   "No value passed for parameter 'dml'"]
+                   "No value passed for parameter 'dml'",
+                   "Module 'hashlib' has no 'sha256' member",
+                   "Module 'hashlib' has no 'sha224' member"]
 # Note(maoy): we ignore all errors in openstack.common because it should be
 # checked elsewhere. We also ignore cinder.tests for now due to high false
 # positive rate.