pylint

faucetsdn · Sep 26, 2024 · de9f0c9 · de9f0c9
1 parent ac0bf4b
commit de9f0c9
Show file tree

Hide file tree

Showing 5 changed files with 14 additions and 14 deletions.
diff --git a/adapters/vendors/rabbitmq/test_rabbit.py b/adapters/vendors/rabbitmq/test_rabbit.py
@@ -21,7 +21,7 @@ def basic_publish(
         body,  # pylint: disable=unused-argument
         properties=None,  # pylint: disable=unused-argument
         mandatory=False,
-    ):  # pylint: disable=unused-argument
+    ):  # pylint: disable=unused-argument,too-many-arguments,too-many-positional-arguments
         return True
 
 
@@ -39,7 +39,7 @@ def basic_publish(
         body,  # pylint: disable=unused-argument
         properties=None,  # pylint: disable=unused-argument
         mandatory=False,
-    ):  # pylint: disable=unused-argument
+    ):  # pylint: disable=unused-argument,too-many-arguments,too-many-positional-arguments
         raise pika.exceptions.AMQPError("failure")
 
 

diff --git a/faucet/faucet_pipeline.py b/faucet/faucet_pipeline.py
@@ -25,7 +25,7 @@ class ValveTableConfig:  # pylint: disable=too-many-instance-attributes
     def __init__(
         self,
         name,
-        table_id,  # pylint: disable=too-many-arguments
+        table_id,
         exact_match=None,
         meter=None,
         output=True,
@@ -39,7 +39,7 @@ def __init__(
         next_tables=None,
         metadata_match=0,
         metadata_write=0,
-    ):
+    ):  # pylint: disable=too-many-arguments,too-many-positional-arguments
         self.name = name
         self.table_id = table_id
         self.exact_match = exact_match

diff --git a/faucet/valve_acl.py b/faucet/valve_acl.py
@@ -190,7 +190,7 @@ def build_output_actions(acl_table, output_dict, tunnel_rules=None, source_id=No
     return (output_port, output_actions, ofmsgs, output_inst)
 
 
-def build_acl_entry(  # pylint: disable=too-many-arguments,too-many-branches,too-many-statements
+def build_acl_entry(
     acl_table,
     rule_conf,
     meters,
@@ -200,7 +200,7 @@ def build_acl_entry(  # pylint: disable=too-many-arguments,too-many-branches,too
     vlan_vid=None,
     tunnel_rules=None,
     source_id=None,
-):
+):  # pylint: disable=too-many-arguments,too-many-positional-arguments,too-many-branches,too-many-statements
     """Build flow/groupmods for one ACL rule entry."""
     acl_inst = []
     acl_act = []
@@ -282,7 +282,7 @@ def build_tunnel_ofmsgs(
     vlan_vid=None,
     flowdel=False,
     reverse=False,
-):
+):  # pylint: disable=too-many-arguments,too-many-positional-arguments
     """Build a specific tunnel only ofmsgs"""
     ofmsgs = []
     acl_inst = []
@@ -329,7 +329,7 @@ def build_rule_ofmsgs(
     tunnel_rules=None,
     source_id=None,
     flowdel=False,
-):
+):  # pylint: disable=too-many-arguments,too-many-positional-arguments
     """Build an ACL rule and return OFMSGs"""
     ofmsgs = []
     acl_match, acl_inst, acl_cookie, acl_ofmsgs = build_acl_entry(
@@ -369,7 +369,7 @@ def build_acl_ofmsgs(
     tunnel_rules=None,
     source_id=None,
     flowdel=False,
-):
+):  # pylint: disable=too-many-arguments,too-many-positional-arguments
     """Build flow/groupmods for all entries in an ACL."""
     ofmsgs = []
     acl_rule_priority = highest_priority
@@ -437,7 +437,7 @@ def __init__(
         pipeline,
         meters,
         dp_acls=None,
-    ):
+    ):  # pylint: disable=too-many-arguments,too-many-positional-arguments
         self.dp_acls = dp_acls
         self.port_acl_table = port_acl_table
         self.vlan_acl_table = vlan_acl_table

diff --git a/faucet/valve_coprocessor.py b/faucet/valve_coprocessor.py
@@ -30,11 +30,11 @@ def __init__(
         ports,
         copro_table,
         vlan_table,
-        eth_src_table,  # pylint: disable=too-many-arguments
+        eth_src_table,
         output_table,
         low_priority,
         high_priority,
-    ):
+    ):  # pylint: disable=too-many-arguments,too-many-positional-arguments
         self.ports = ports
         self.copro_table = copro_table
         self.vlan_table = vlan_table

diff --git a/faucet/valve_lldp.py b/faucet/valve_lldp.py
@@ -39,7 +39,7 @@ def __init__(
         set_var,
         set_port_var,
         stack_manager,
-    ):
+    ):  # pylint: disable=too-many-arguments,too-many-positional-arguments
         self.vlan_table = vlan_table
         self.highest_priority = highest_priority
         self.logger = logger
@@ -89,7 +89,7 @@ def verify_lldp(
         remote_dp_name,
         remote_port_id,
         remote_port_state,
-    ):
+    ):  # pylint: disable=too-many-arguments,too-many-positional-arguments
         """
         Verify correct LLDP cabling, then update port to next state