]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Ignore No value passed for parameter 'dml' message
authorIvan Kolodyazhny <e0ne@e0ne.info>
Mon, 18 Aug 2014 19:24:15 +0000 (22:24 +0300)
committerIvan Kolodyazhny <e0ne@e0ne.info>
Mon, 18 Aug 2014 19:37:15 +0000 (22:37 +0300)
There are a lot false positive PyLint errors on
DB migrations for table update method

Change-Id: Id033a1f5af04cde4d8bd888e01bfcc48e656a252

tools/lintstack.py

index 1a79ef3b4e2a6aee40a853305fcb2356bae606c3..22e267f65119033fd39b42e2ac9a3f0dfc45a04b 100755 (executable)
@@ -30,9 +30,13 @@ from six.moves import cStringIO as StringIO
 ignore_codes = ["E1103"]
 # Note(maoy): the error message is the pattern of E0202. It should be ignored
 # 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.
-ignore_messages = ["An attribute affected in cinder.tests", "No name 'urllib' in module '_MovedItems'"]
+# 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
+# in DB schema migrations.
+ignore_messages = ["An attribute affected in cinder.tests",
+                   "No name 'urllib' in module '_MovedItems'",
+                   "No value passed for parameter 'dml'"]
 # 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.