diff --git a/adapters/vendors/rabbitmq/test_rabbit.py b/adapters/vendors/rabbitmq/test_rabbit.py index c4e6cd40e1..2ff76631ca 100644 --- 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 index 2eb18cb984..013f53450d 100644 --- 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 index 65aca4abbd..0ebf269e4a 100644 --- 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 index e20104ceda..10b960cd06 100644 --- 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