{Pylint} Remove some invalid disable=unused-argument

Author: atombrella

src/azure-cli-core/azure/cli/core/aaz/_arg_action.py

@@ -233,7 +233,7 @@ def get_schema_by_key_items(key_items, schema):
         return tuple(valid_key_items), schema
 
     @classmethod
-    def _decode_value(cls, schema, value):  # pylint: disable=unused-argument
+    def _decode_value(cls, schema, value):
         from ._arg import AAZSimpleTypeArg
         from azure.cli.core.util import get_file_json, shell_safe_json_parse, get_file_yaml
 

src/azure-cli/azure/cli/command_modules/acs/custom.py

@@ -2686,7 +2686,7 @@ def aks_agentpool_operation_abort(cmd,
     return sdk_no_wait(no_wait, client.begin_abort_latest_operation, resource_group_name, cluster_name, nodepool_name)
 
 
-def aks_operation_abort(cmd,   # pylint: disable=unused-argument
+def aks_operation_abort(cmd,
                         client,
                         resource_group_name,
                         name,
@@ -2705,7 +2705,7 @@ def aks_operation_abort(cmd,   # pylint: disable=unused-argument
     return sdk_no_wait(no_wait, client.begin_abort_latest_operation, resource_group_name, name)
 
 
-def aks_agentpool_delete_machines(cmd,   # pylint: disable=unused-argument
+def aks_agentpool_delete_machines(cmd,
                                   client,
                                   resource_group_name,
                                   cluster_name,
@@ -2805,7 +2805,7 @@ def aks_nodepool_snapshot_create(cmd,    # pylint: disable=too-many-locals,too-m
     return client.create_or_update(resource_group_name, snapshot_name, snapshot, headers=aks_custom_headers)
 
 
-def aks_nodepool_snapshot_update(cmd, client, resource_group_name, snapshot_name, tags):   # pylint: disable=unused-argument
+def aks_nodepool_snapshot_update(cmd, client, resource_group_name, snapshot_name, tags):
     TagsObject = cmd.get_models(
         "TagsObject",
         resource_type=ResourceType.MGMT_CONTAINERSERVICE,
@@ -3266,7 +3266,6 @@ def aks_approuting_zone_list(
     raise CLIError('App routing addon is not enabled')
 
 
-# pylint: disable=unused-argument
 def _aks_approuting_update(
         cmd,
         client,

src/azure-cli/azure/cli/command_modules/aro/custom.py

@@ -44,8 +44,8 @@ def aro_create(cmd,  # pylint: disable=too-many-locals
                resource_name,
                master_subnet,
                worker_subnet,
-               vnet=None,  # pylint: disable=unused-argument
-               vnet_resource_group_name=None,  # pylint: disable=unused-argument
+               vnet=None,
+               vnet_resource_group_name=None,
                enable_preconfigured_nsg=False,
                location=None,
                pull_secret=None,
@@ -301,21 +301,21 @@ def __init__(self, disk_encryption_id, master_subnet_id, worker_subnet_id, preco
 
 
 def aro_validate(cmd,  # pylint: disable=too-many-locals,too-many-statements
-                 client,  # pylint: disable=unused-argument
-                 resource_group_name,  # pylint: disable=unused-argument
-                 resource_name,  # pylint: disable=unused-argument
+                 client,
+                 resource_group_name,
+                 resource_name,
                  master_subnet,
                  worker_subnet,
                  vnet=None,
-                 cluster_resource_group=None,  # pylint: disable=unused-argument
+                 cluster_resource_group=None,
                  client_id=None,
-                 client_secret=None,  # pylint: disable=unused-argument
-                 vnet_resource_group_name=None,  # pylint: disable=unused-argument
+                 client_secret=None,
+                 vnet_resource_group_name=None,
                  disk_encryption_set=None,
-                 location=None,  # pylint: disable=unused-argument
+                 location=None,
                  version=None,
-                 pod_cidr=None,  # pylint: disable=unused-argument
-                 service_cidr=None,  # pylint: disable=unused-argument
+                 pod_cidr=None,
+                 service_cidr=None,
                  ):
 
     validate(cmd,

src/azure-cli/azure/cli/command_modules/keyvault/custom.py

@@ -921,7 +921,7 @@ def set_policy(cmd, client, resource_group_name, vault_name,
 
 
 def add_network_rule(cmd, client, resource_group_name, vault_name, ip_address=None, subnet=None,
-                     vnet_name=None, no_wait=False):  # pylint: disable=unused-argument
+                     vnet_name=None, no_wait=False):
     """ Add a network rule to the network ACLs for a Key Vault. """
 
     VirtualNetworkRule = cmd.get_models('VirtualNetworkRule', resource_type=ResourceType.MGMT_KEYVAULT)
@@ -983,7 +983,7 @@ def add_network_rule(cmd, client, resource_group_name, vault_name, ip_address=No
 
 
 def remove_network_rule(cmd, client, resource_group_name, vault_name, ip_address=None, subnet=None,
-                        vnet_name=None, no_wait=False):  # pylint: disable=unused-argument
+                        vnet_name=None, no_wait=False):
     """ Remove a network rule from the network ACLs for a Key Vault. """
 
     VaultCreateOrUpdateParameters = cmd.get_models('VaultCreateOrUpdateParameters',

src/azure-cli/azure/cli/command_modules/role/_validators.py

@@ -74,7 +74,7 @@ def validate_cert(namespace):
         namespace.cert = x509
 
 
-def process_assignment_namespace(cmd, namespace):  # pylint: disable=unused-argument
+def process_assignment_namespace(cmd, namespace):
     # Make sure these arguments are non-empty strings.
     # When they are accidentally provided as an empty string "", they won't take effect when filtering the role
     # assignments, causing all matched role assignments to be listed/deleted. For example,

src/azure-cli/azure/cli/command_modules/vm/azure_stack/_validators.py

@@ -1879,7 +1879,8 @@ def process_vmss_create_namespace(cmd, namespace):
     _validate_community_gallery_legal_agreement_acceptance(cmd, namespace)
 
 
-def validate_vmss_update_namespace(cmd, namespace):  # pylint: disable=unused-argument
+def validate_vmss_update_namespace(cmd, namespace):
+def validate_vmss_update_namespace(cmd, namespace):
     if not namespace.instance_id:
         if namespace.protect_from_scale_in is not None or namespace.protect_from_scale_set_actions is not None:
             raise CLIError("usage error: protection policies can only be applied to VM instances within a VMSS."
@@ -2202,7 +2203,8 @@ def process_remove_identity_namespace(cmd, namespace):
                                                            'Microsoft.ManagedIdentity')
 
 
-def process_set_applications_namespace(cmd, namespace):  # pylint: disable=unused-argument
+def process_set_applications_namespace(cmd, namespace):
+def process_set_applications_namespace(cmd, namespace):
     _validate_vm_vmss_set_applications(cmd, namespace)
 
 
@@ -2540,15 +2542,15 @@ def _validate_vmss_terminate_notification(cmd, namespace):  # pylint: disable=un
         namespace.terminate_notification_time = 'PT' + namespace.terminate_notification_time + 'M'
 
 
-def _validate_vmss_create_automatic_repairs(cmd, namespace):  # pylint: disable=unused-argument
+def _validate_vmss_create_automatic_repairs(cmd, namespace):
     if namespace.automatic_repairs_grace_period is not None or namespace.automatic_repairs_action is not None:
         if namespace.load_balancer is None or namespace.health_probe is None:
             raise ArgumentUsageError("usage error: --load-balancer and --health-probe are required "
                                      "when creating vmss with automatic repairs")
     _validate_vmss_automatic_repairs(cmd, namespace)
 
 
-def _validate_vmss_update_automatic_repairs(cmd, namespace):  # pylint: disable=unused-argument
+def _validate_vmss_update_automatic_repairs(cmd, namespace):
     if namespace.enable_automatic_repairs is False and \
             (namespace.automatic_repairs_grace_period is not None or namespace.automatic_repairs_action is not None):
         raise ArgumentUsageError("usage error: please enable --enable-automatic-repairs")