From: Ivan Kolodyazhny <e0ne@e0ne.info>
Date: Mon, 18 Aug 2014 19:24:15 +0000 (+0300)
Subject: Ignore No value passed for parameter 'dml' message
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=5a86bbaa1e516b8bd85d9984cdd9516b49de5f47;p=openstack-build%2Fcinder-build.git

Ignore No value passed for parameter 'dml' message

There are a lot false positive PyLint errors on
DB migrations for table update method

Change-Id: Id033a1f5af04cde4d8bd888e01bfcc48e656a252
---

diff --git a/tools/lintstack.py b/tools/lintstack.py
index 1a79ef3b4..22e267f65 100755
--- a/tools/lintstack.py
+++ b/tools/lintstack.py
@@ -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.