Merge "python: Pylint 1.8.2 code conformity"

4 files changed, 4 insertions, 5 deletions

diff --git a/src/__init__.py b/src/__init__.py
index 9293b4f8..c784ea3c 100644
--- a/src/__init__.py
+++ b/src/__init__.py
@@ -18,4 +18,3 @@ No functionality is expected for this package and its purpose is just to
 keep Python package structure intact without extra requirements for
 PYTHONPATH.
 """
-
diff --git a/src/dpdk/dpdk.py b/src/dpdk/dpdk.py
index 2f120129..c2e656ef 100644
--- a/src/dpdk/dpdk.py
+++ b/src/dpdk/dpdk.py
@@ -140,7 +140,7 @@ def _vhost_user_cleanup():
 def _bind_nics():
     """Bind NICs using the bind tool specified in the configuration.
     """
-    if not len(_NICS_PCI):
+    if not _NICS_PCI:
         _LOGGER.info('NICs are not configured - nothing to bind')
         return
     try:
@@ -171,7 +171,7 @@ def _bind_nics():
 def _unbind_nics():
     """Unbind NICs using the bind tool specified in the configuration.
     """
-    if not len(_NICS_PCI):
+    if not _NICS_PCI:
         _LOGGER.info('NICs are not configured - nothing to unbind')
         return
     try:
diff --git a/src/ovs/dpctl.py b/src/ovs/dpctl.py
index 015fb38c..5030223e 100644
--- a/src/ovs/dpctl.py
+++ b/src/ovs/dpctl.py
@@ -60,5 +60,5 @@ class DPCtl(object):
 
         :return: None
         """
-        self.logger.debug('delete datapath ' + dp_name)
+        self.logger.debug('delete datapath %s', dp_name)
         self.run_dpctl(['del-dp', dp_name])
diff --git a/src/ovs/ofctl.py b/src/ovs/ofctl.py
index 110940c8..b023e080 100644
--- a/src/ovs/ofctl.py
+++ b/src/ovs/ofctl.py
@@ -469,4 +469,4 @@ def flow_match(flow_dump, flow_src):
     for rule in flow_src_list:
         if rule in flow_dump_list:
             flow_src_ctrl.remove(rule)
-    return True if not len(flow_src_ctrl) else False
+    return True if not flow_src_ctrl else False