qontract-reconcile 0.10.2.dev14__py3-none-any.whl → 0.10.2.dev16__py3-none-any.whl

reconcile/rhidp/ocm_oidc_idp/base.py

@@ -1,4 +1,5 @@
 import logging
+import operator
 from collections.abc import (
     Iterable,
     Sequence,
@@ -38,10 +39,10 @@ class IDPState(BaseModel):
     cluster: Cluster
     idp: OCMOIdentityProvider | OCMOIdentityProviderOidc | OCMOIdentityProviderGithub
 
-    def __eq__(self, __value: object) -> bool:
-        if not isinstance(__value, IDPState):
+    def __eq__(self, value: object) -> bool:
+        if not isinstance(value, IDPState):
             raise NotImplementedError("Cannot compare to non IDPState objects.")
-        return self.idp == __value.idp
+        return self.idp == value.idp
 
 
 def run(
@@ -91,19 +92,13 @@ def fetch_current_state(
     ocm_api: OCMBaseClient, clusters: Iterable[Cluster]
 ) -> list[IDPState]:
     """Fetch all current configured OIDC identity providers."""
-    current_state: list[IDPState] = []
-    for cluster in clusters:
+    return [
+        IDPState(cluster=cluster, idp=idp)
+        for cluster in clusters
         for idp in get_identity_providers(
             ocm_api=ocm_api, ocm_cluster=cluster.ocm_cluster
-        ):
-            current_state.append(
-                IDPState(
-                    cluster=cluster,
-                    idp=idp,
-                ),
-            )
-
-    return current_state
+        )
+    ]
 
 
 def fetch_desired_state(
@@ -175,7 +170,7 @@ def act(
             idp_state.idp.type,
             idp_state.idp.name,
         )),
-        equal=lambda idp1, idp2: idp1 == idp2,
+        equal=operator.eq,
     )
 
     for idp_state in diff_result.delete.values():

reconcile/run_integration.py

@@ -28,8 +28,8 @@ from reconcile.utils.runtime.environment import (
     log_fmt,
 )
 
-SHARDS = int(os.environ.get("SHARDS", 1))
-SHARD_ID = int(os.environ.get("SHARD_ID", 0))
+SHARDS = int(os.environ.get("SHARDS", "1"))
+SHARD_ID = int(os.environ.get("SHARD_ID", "0"))
 SHARD_ID_LABEL = os.environ.get("SHARD_KEY", f"{SHARD_ID}-{SHARDS}")
 PREFIX_LOG_LEVEL = os.environ.get("PREFIX_LOG_LEVEL", "false")
 
@@ -44,14 +44,14 @@ DRY_RUN = (
 )
 INTEGRATION_EXTRA_ARGS = os.environ.get("INTEGRATION_EXTRA_ARGS")
 CONFIG = os.environ.get("CONFIG", "/config/config.toml")
-PROMETHEUS_PORT = os.environ.get("PROMETHEUS_PORT", 9090)
+PROMETHEUS_PORT = int(os.environ.get("PROMETHEUS_PORT", "9090"))
 
 LOG_FILE = os.environ.get("LOG_FILE")
 LOG_LEVEL = os.environ.get("LOG_LEVEL", "INFO")
-SLEEP_DURATION_SECS = os.environ.get("SLEEP_DURATION_SECS", 600)
-SLEEP_ON_ERROR = os.environ.get("SLEEP_ON_ERROR", 10)
+SLEEP_DURATION_SECS = int(os.environ.get("SLEEP_DURATION_SECS", "600"))
+SLEEP_ON_ERROR = int(os.environ.get("SLEEP_ON_ERROR", "10"))
 
-PUSHGATEWAY_ENABLED = os.environ.get("PUSHGATEWAY_ENABLED", False)
+PUSHGATEWAY_ENABLED = bool(os.environ.get("PUSHGATEWAY_ENABLED"))
 
 LOG = logging.getLogger(__name__)
 
@@ -88,7 +88,7 @@ def _parse_dry_run_flag(dry_run: str | None) -> str | None:
         )
         logging.error(msg)
         raise ValueError(msg)
-    return dry_run if dry_run else None
+    return dry_run or None
 
 
 def build_entry_point_args(

reconcile/saas_auto_promotions_manager/publisher.py

@@ -54,7 +54,7 @@ class Publisher:
         self.app_name = app_name
         self.namespace_name = namespace_name
         self.resource_template_name = resource_template_name
-        self.target_name = target_name if target_name else "None"
+        self.target_name = target_name or "None"
         self.cluster_name = cluster_name
         self.redeploy_on_config_change = bool(redeploy_on_config_change)
         self.has_subscriber = has_subscriber

reconcile/saas_auto_promotions_manager/utils/saas_files_inventory.py

@@ -92,21 +92,15 @@ class SaasFilesInventory:
                     blocked_versions.setdefault(code_component.url, set()).add(version)
             for resource_template in saas_file.resource_templates:
                 for target in resource_template.targets:
-                    file_path = target.path if target.path else saas_file.path
+                    file_path = target.path or saas_file.path
                     if target.disable or target.delete:
                         continue
                     if not target.promotion:
                         continue
                     if not target.promotion.auto:
                         continue
-                    soak_days = (
-                        target.promotion.soak_days if target.promotion.soak_days else 0
-                    )
-                    schedule = (
-                        target.promotion.schedule
-                        if target.promotion.schedule
-                        else "* * * * *"
-                    )
+                    soak_days = target.promotion.soak_days or 0
+                    schedule = target.promotion.schedule or "* * * * *"
                     subscriber = Subscriber(
                         uid=target.uid(
                             parent_saas_file_name=saas_file.name,

reconcile/service_dependencies.py

@@ -21,13 +21,8 @@ QONTRACT_INTEGRATION = "service-dependencies"
 
 
 def get_dependency_names(dependency_map: Mapping[Any, Any], dep_type: str) -> list[str]:
-    dep_names = []
-    for dm in dependency_map:
-        if dm["type"] != dep_type:
-            continue
-        for service in dm["services"]:
-            dep_names.append(service["name"])
-    return dep_names
+    dependency_maps = (dm for dm in dependency_map if dm["type"] == dep_type)
+    return [service["name"] for service in dependency_maps]
 
 
 def get_desired_dependency_names(

reconcile/skupper_network/reconciler.py

@@ -31,7 +31,7 @@ def delete_skupper_site(
     for kind in integration_managed_kinds:
         # delete everything labeled by us
         to_delete.update({
-            f'{item["kind"]}-{item["metadata"]["name"]}': item
+            f"{item['kind']}-{item['metadata']['name']}": item
             for item in oc.get_items(
                 kind=kind,
                 namespace=site.namespace.name,
@@ -40,7 +40,7 @@ def delete_skupper_site(
         })
         # delete everything else that starts with 'skupper-'
         to_delete.update({
-            f'{item["kind"]}-{item["metadata"]["name"]}': item
+            f"{item['kind']}-{item['metadata']['name']}": item
             for item in oc.get_items(kind=kind, namespace=site.namespace.name)
             if item["metadata"]["name"].startswith("skupper-")
         })
@@ -84,12 +84,12 @@ def _create_token(
     logging.info(f"{connected_site}: Creating new connection token for {site}")
     sc = get_site_controller(connected_site)
     if not dry_run:
-        _labels = copy.deepcopy(site.token_labels)
-        _labels.update(labels)
+        labels_ = copy.deepcopy(site.token_labels)
+        labels_.update(labels)
         oc.apply(
             connected_site.namespace.name,
             resource=OR(
-                body=sc.site_token(connected_site.unique_token_name(site), _labels),
+                body=sc.site_token(connected_site.unique_token_name(site), labels_),
                 integration=integration,
                 integration_version=integration_version,
             ),

reconcile/skupper_network/site_controller.py

@@ -28,14 +28,14 @@ class SiteController:
 
     def site_token(self, name: str, labels: MutableMapping[str, str]) -> dict[str, Any]:
         """Skupper site token secret."""
-        _labels = copy.deepcopy(labels)
-        _labels["skupper.io/type"] = "connection-token-request"
+        labels_ = copy.deepcopy(labels)
+        labels_["skupper.io/type"] = "connection-token-request"
         return {
             "apiVersion": "v1",
             "kind": "Secret",
             "metadata": {
                 "name": name,
-                "labels": _labels,
+                "labels": labels_,
             },
         }
 

reconcile/sql_query.py

@@ -277,17 +277,17 @@ def collect_queries(
         tf_resource_info = get_tf_resource_info(terrascript, namespace, identifier)
         if tf_resource_info is None:
             logging.error(
-                f'[sql-query:{name} (path: {sql_query["path"]})] Could not find rds identifier {identifier} in namespace {namespace["name"]}. '
+                f"[sql-query:{name} (path: {sql_query['path']})] Could not find rds identifier {identifier} in namespace {namespace['name']}. "
                 "If this is a removed read only instance, consider updating the identifier to the source replica or remove this file."
             )
             sys.exit(ExitCodes.ERROR)
 
-        _queries = []
+        queries_ = []
         if sql_query["query"] is not None:
-            _queries.append(sql_query["query"])
+            queries_.append(sql_query["query"])
 
         if sql_query["queries"] is not None:
-            _queries.extend(sql_query["queries"])
+            queries_.extend(sql_query["queries"])
 
         # building up the final query dictionary
         item = {
@@ -296,7 +296,7 @@ def collect_queries(
             "identifier": sql_query["identifier"],
             "db_conn": db_conn,
             "output": output,
-            "queries": _queries,
+            "queries": queries_,
             **tf_resource_info,
         }
 

reconcile/status_board.py

@@ -194,40 +194,40 @@ class StatusBoardExporterIntegration(QontractReconcileIntegration):
                 )
             )
             # new product, so it misses also the applications
-            for app_name in desired_product_apps[product_name]:
-                return_list.append(
-                    StatusBoardHandler(
-                        action="create",
-                        status_board_object=create_app(app_name, product),
-                    )
+            return_list.extend(
+                StatusBoardHandler(
+                    action="create",
+                    status_board_object=create_app(app_name, product),
                 )
+                for app_name in desired_product_apps[product_name]
+            )
 
         # existing product, only add/remove applications
         for product_name, apps in diff_result.change.items():
             product = current_products[product_name]
-            for app_name in apps.desired - apps.current:
-                return_list.append(
-                    StatusBoardHandler(
-                        action="create",
-                        status_board_object=create_app(app_name, product),
-                    )
+            return_list.extend(
+                StatusBoardHandler(
+                    action="create",
+                    status_board_object=create_app(app_name, product),
                 )
+                for app_name in apps.desired - apps.current
+            )
             to_delete = apps.current - apps.desired
-            for application in product.applications or []:
-                if application.name in to_delete:
-                    return_list.append(
-                        StatusBoardHandler(
-                            action="delete",
-                            status_board_object=application,
-                        )
-                    )
+            return_list.extend(
+                StatusBoardHandler(
+                    action="delete",
+                    status_board_object=application,
+                )
+                for application in product.applications or []
+                if application.name in to_delete
+            )
 
         # product is deleted entirely
         for product_name in diff_result.delete:
-            for application in current_products[product_name].applications or []:
-                return_list.append(
-                    StatusBoardHandler(action="delete", status_board_object=application)
-                )
+            return_list.extend(
+                StatusBoardHandler(action="delete", status_board_object=application)
+                for application in current_products[product_name].applications or []
+            )
             return_list.append(
                 StatusBoardHandler(
                     action="delete", status_board_object=current_products[product_name]

reconcile/terraform_cloudflare_users.py

@@ -331,8 +331,8 @@ def build_external_resource_spec_from_cloudflare_users(
     """
     specs: list[ExternalResourceSpec] = []
 
-    for _, v in cloudflare_users.items():
-        for _, cf_user in v.items():
+    for v in cloudflare_users.values():
+        for cf_user in v.values():
             data_source_cloudflare_account_roles = {
                 "identifier": safe_resource_id(cf_user.account_name),
                 "account_id": "${var.account_id}",

reconcile/terraform_repo.py

@@ -382,12 +382,12 @@ class TerraformRepoIntegration(
 
                 # construct diff urls
                 diff_urls: list[str] = []
-                for pair in diff_result.change.values():
-                    if pair.current.ref != pair.desired.ref:
-                        # gitlab specific syntax
-                        diff_urls.append(
-                            f"{pair.current.repository}/compare/{pair.current.ref}...{pair.desired.ref}"
-                        )
+                # gitlab specific syntax
+                diff_urls.extend(
+                    f"{pair.current.repository}/compare/{pair.current.ref}...{pair.desired.ref}"
+                    for pair in diff_result.change.values()
+                    if pair.current.ref != pair.desired.ref
+                )
 
                 if len(diff_urls) > 0:
                     comment_body = "tf-repo diffs:\n" + "\n".join([

reconcile/terraform_users.py

@@ -95,12 +95,15 @@ def _filter_participating_aws_accounts(
     accounts: list,
     roles: list[dict[str, Any]],
 ) -> list:
-    participating_aws_account_names = set()
+    participating_aws_account_names: set[str] = set()
     for role in roles:
-        for aws_group in role["aws_groups"] or []:
-            participating_aws_account_names.add(aws_group["account"]["name"])
-        for user_policy in role["user_policies"] or []:
-            participating_aws_account_names.add(user_policy["account"]["name"])
+        participating_aws_account_names.update(
+            aws_group["account"]["name"] for aws_group in role["aws_groups"] or []
+        )
+        participating_aws_account_names.update(
+            user_policy["account"]["name"]
+            for user_policy in role["user_policies"] or []
+        )
     return [a for a in accounts if a["name"] in participating_aws_account_names]
 
 

reconcile/terraform_vpc_peerings.py

@@ -702,7 +702,7 @@ def run(
             integration_version=QONTRACT_INTEGRATION_VERSION,
             dry_run=dry_run,
             cache_source=cache_source,
-            shard=account_name if account_name else "",
+            shard=account_name or "",
             ttl_seconds=extended_early_exit_cache_ttl_seconds,
             logger=logging.getLogger(),
             runner=runner,

reconcile/terraform_vpc_resources/integration.py

@@ -196,7 +196,7 @@ class TerraformVpcResources(QontractReconcileIntegration[TerraformVpcResourcesPa
         mr_manager._fetch_managed_open_merge_requests()
 
         # Create a MR for each vpc request if the MR don't exist yet
-        for _, outputs in handled_output.items():
+        for outputs in handled_output.values():
             template = gql_api.get_template(
                 path="/templating/templates/terraform-vpc-resources/vpc.yml"
             )["template"]

reconcile/typed_queries/app_interface_deadmanssnitch_settings.py

@@ -10,7 +10,7 @@ from reconcile.utils.gql import GqlApi
 def get_deadmanssnitch_settings(
     gql_api: GqlApi | None = None,
 ) -> DeadMansSnitchSettingsV1:
-    api = gql_api if gql_api else gql.get_api()
+    api = gql_api or gql.get_api()
     data = query(query_func=api.query)
     if data.settings and data.settings[0].dead_mans_snitch_settings is not None:
         return data.settings[0].dead_mans_snitch_settings

reconcile/typed_queries/app_quay_repos_escalation_policies.py

@@ -9,6 +9,6 @@ from reconcile.utils.gql import GqlApi
 def get_apps_quay_repos_escalation_policies(
     gql_api: GqlApi | None = None,
 ) -> list[AppV1]:
-    api = gql_api if gql_api else gql.get_api()
+    api = gql_api or gql.get_api()
     data = query(query_func=api.query)
     return list(data.apps or [])

reconcile/typed_queries/aws_vpc_requests.py

@@ -4,6 +4,6 @@ from reconcile.utils.gql import GqlApi
 
 
 def get_aws_vpc_requests(gql_api: GqlApi | None = None) -> list[VPCRequest]:
-    api = gql_api if gql_api else gql.get_api()
+    api = gql_api or gql.get_api()
     data = query(query_func=api.query)
     return list(data.vpc_requests or [])

reconcile/typed_queries/aws_vpcs.py

@@ -7,6 +7,6 @@ from reconcile.utils.gql import GqlApi
 
 
 def get_aws_vpcs(gql_api: GqlApi | None = None) -> list[AWSVPC]:
-    api = gql_api if gql_api else gql.get_api()
+    api = gql_api or gql.get_api()
     data = query(query_func=api.query)
     return list(data.vpcs or [])

reconcile/typed_queries/clusters.py

@@ -13,6 +13,6 @@ def get_clusters(
     variables = {}
     if name:
         variables["name"] = name
-    api = gql_api if gql_api else gql.get_api()
+    api = gql_api or gql.get_api()
     data = query(query_func=api.query, variables=variables)
     return list(data.clusters or [])

reconcile/typed_queries/clusters_minimal.py

@@ -12,6 +12,6 @@ def get_clusters_minimal(
     variables = {}
     if name:
         variables["name"] = name
-    api = gql_api if gql_api else gql.get_api()
+    api = gql_api or gql.get_api()
     data = query(query_func=api.query, variables=variables)
     return list(data.clusters or [])

reconcile/typed_queries/clusters_with_dms.py

@@ -11,6 +11,6 @@ def get_clusters_with_dms(
 ) -> list[ClusterV1]:
     # get the clusters containing the filed enableDeadMansSnitch
     variable = {"filter": {"enableDeadMansSnitch": {"ne": None}}}
-    api = gql_api if gql_api else gql.get_api()
+    api = gql_api or gql.get_api()
     data = query(query_func=api.query, variables=variable)
     return data.clusters or []

reconcile/typed_queries/dynatrace_environments.py

@@ -9,6 +9,6 @@ from reconcile.utils.gql import GqlApi
 def get_dynatrace_environments(
     api: GqlApi | None = None,
 ) -> list[DynatraceEnvironmentV1]:
-    api = api if api else gql.get_api()
+    api = api or gql.get_api()
     data = query(api.query)
     return list(data.environments or [])

reconcile/typed_queries/dynatrace_token_provider_token_specs.py

@@ -9,6 +9,6 @@ from reconcile.utils.gql import GqlApi
 def get_dynatrace_token_provider_token_specs(
     api: GqlApi | None = None,
 ) -> list[DynatraceTokenProviderTokenSpecV1]:
-    api = api if api else gql.get_api()
+    api = api or gql.get_api()
     data = query(api.query)
     return list(data.token_specs or [])

reconcile/typed_queries/reserved_networks.py

@@ -7,6 +7,6 @@ from reconcile.utils.gql import GqlApi
 
 
 def get_networks(gql_api: GqlApi | None = None) -> list[NetworkV1]:
-    api = gql_api if gql_api else gql.get_api()
+    api = gql_api or gql.get_api()
     data = query(query_func=api.query)
     return list(data.networks or [])

reconcile/typed_queries/saas_files.py

@@ -73,7 +73,7 @@ class SaasResourceTemplateTarget(ConfiguredBaseModel):
     ) -> str:
         """Returns a unique identifier for a target."""
         return hashlib.blake2s(
-            f"{parent_saas_file_name}:{parent_resource_template_name}:{self.name if self.name else 'default'}:{self.namespace.cluster.name}:{self.namespace.name}".encode(),
+            f"{parent_saas_file_name}:{parent_resource_template_name}:{self.name or 'default'}:{self.namespace.cluster.name}:{self.namespace.name}".encode(),
             digest_size=20,
         ).hexdigest()
 

reconcile/typed_queries/slo_documents.py

@@ -7,6 +7,6 @@ from reconcile.utils.gql import GqlApi
 
 
 def get_slo_documents(gql_api: GqlApi | None = None) -> list[SLODocumentV1]:
-    api = gql_api if gql_api else gql.get_api()
+    api = gql_api or gql.get_api()
     data = query(query_func=api.query)
     return data.slo_document_v1 or []

reconcile/typed_queries/status_board.py

@@ -51,8 +51,7 @@ def get_selected_app_names(
     for selector in selectors:
         apps_to_remove: set[str] = set()
         results = parser.parse(selector).find(apps)
-        for match in results:
-            apps_to_remove.add(match.value["name"])
+        apps_to_remove.update(match.value["name"] for match in results)
         selected_app_names -= apps_to_remove
 
     return selected_app_names

reconcile/utils/amtool.py

@@ -52,7 +52,7 @@ def version() -> AmtoolResult:
     """Returns the version parsed from amtool --version"""
     result = _run_cmd(["amtool", "--version"])
 
-    pattern = re.compile("^amtool, version (?P<version>[^ ]+) .+")
+    pattern = re.compile(r"^amtool, version (?P<version>[^ ]+) .+")
     if m := pattern.match(result.message):
         return AmtoolResult(True, m.group("version"))
 
@@ -63,7 +63,7 @@ def _run_cmd(cmd: list[str]) -> AmtoolResult:
     try:
         result = run(cmd, capture_output=True, check=True)
     except CalledProcessError as e:
-        msg = f'Error running amtool command [{" ".join(cmd)}]'
+        msg = f"Error running amtool command [{' '.join(cmd)}]"
         if e.stdout:
             msg += f" {e.stdout.decode()}"
         if e.stderr:

reconcile/utils/aws_api.py

@@ -1,4 +1,5 @@
 import logging
+import operator
 import os
 import re
 import time
@@ -191,7 +192,7 @@ class AWSApi:  # pylint: disable=too-many-public-methods
         service_name,
         region_name: str | None = None,
     ):
-        region = region_name if region_name else session.region_name
+        region = region_name or session.region_name
         client = session.client(
             service_name,
             region_name=region,
@@ -205,7 +206,7 @@ class AWSApi:  # pylint: disable=too-many-public-methods
     def _get_session_resource(
         session: Session, service_name, region_name: str | None = None
     ):
-        region = region_name if region_name else session.region_name
+        region = region_name or session.region_name
         return session.resource(service_name, region_name=region)
 
     def _account_ec2_client(
@@ -1000,12 +1001,13 @@ class AWSApi:  # pylint: disable=too-many-public-methods
         assumed_role_data = self._get_account_assume_data(account)
         assumed_ec2 = self._get_assumed_role_client(*assumed_role_data)
         nat_gateways = assumed_ec2.describe_nat_gateways()
-        egress_ips = set()
+        egress_ips: set[str] = set()
         for nat in nat_gateways.get("NatGateways") or []:
             if nat["VpcId"] != vpc_id:
                 continue
-            for address in nat["NatGatewayAddresses"]:
-                egress_ips.add(address["PublicIp"])
+            egress_ips.update(
+                address["PublicIp"] for address in nat["NatGatewayAddresses"]
+            )
 
         return egress_ips
 
@@ -1481,13 +1483,11 @@ class AWSApi:  # pylint: disable=too-many-public-methods
             client.delete_hosted_zone(Id=zone_id)
         except client.exceptions.NoSuchHostedZone:
             logging.error(
-                f"[{account_name}] Error trying to delete "
-                f"unknown DNS zone {zone_id}"
+                f"[{account_name}] Error trying to delete unknown DNS zone {zone_id}"
             )
         except client.exceptions.HostedZoneNotEmpty:
             logging.error(
-                f"[{account_name}] Cannot delete DNS zone that "
-                f"is not empty {zone_id}"
+                f"[{account_name}] Cannot delete DNS zone that is not empty {zone_id}"
             )
         except Exception as e:
             logging.error(f"[{account_name}] unhandled exception: {e}")
@@ -1697,7 +1697,7 @@ class AWSApi:  # pylint: disable=too-many-public-methods
         return [
             obj["Key"]
             for obj in sorted(
-                objects, key=lambda obj: obj["LastModified"], reverse=True
+                objects, key=operator.itemgetter("LastModified"), reverse=True
             )
         ]
 

reconcile/utils/aws_helper.py

@@ -74,7 +74,7 @@ def get_account(accounts: Iterable[Account], account_name: str) -> Account:
     raise AccountNotFoundError(account_name)
 
 
-def get_region_from_availability_zone(availability_zone: str) -> str:
+def get_region_from_availability_zone(availability_zone: str) -> str:  # noqa: FURB118
     return availability_zone[:-1]
 
 

reconcile/utils/binary.py

@@ -39,8 +39,7 @@ def binary_version(binary, version_args, search_regex, expected_versions):
                 result = subprocess.run(cmd, capture_output=True, check=True)
             except subprocess.CalledProcessError as e:
                 msg = (
-                    f"Could not execute binary '{binary}' "
-                    f"for binary version check: {e}"
+                    f"Could not execute binary '{binary}' for binary version check: {e}"
                 )
                 raise Exception(msg) from e
 

reconcile/utils/differ.py

@@ -1,3 +1,4 @@
+import operator
 from collections.abc import (
     Callable,
     Iterable,
@@ -30,10 +31,6 @@ class DiffResult(Generic[Current, Desired, Key]):
     identical: dict[Key, DiffPair[Current, Desired]]
 
 
-def _default_equal(current: Current, desired: Desired) -> bool:
-    return current == desired
-
-
 def _default_key(item: Any) -> Any:
     return item
 
@@ -41,7 +38,7 @@ def _default_key(item: Any) -> Any:
 def diff_mappings(
     current: Mapping[Key, Current],
     desired: Mapping[Key, Desired],
-    equal: Callable[[Current, Desired], bool] = _default_equal,
+    equal: Callable[[Current, Desired], bool] = operator.eq,
 ) -> DiffResult[Current, Desired, Key]:
     """
     Compare two mappings and return a `DiffResult` instance containing the differences between them.
@@ -91,7 +88,7 @@ def diff_any_iterables(
     desired: Iterable[Desired],
     current_key: Callable[[Current], Key] = _default_key,
     desired_key: Callable[[Desired], Key] = _default_key,
-    equal: Callable[[Current, Desired], bool] = _default_equal,
+    equal: Callable[[Current, Desired], bool] = operator.eq,
 ) -> DiffResult[Current, Desired, Key]:
     """
     Compare two iterables and return a `DiffResult` instance containing the differences between them.
@@ -152,7 +149,7 @@ def diff_iterables(
     current: Iterable[T],
     desired: Iterable[T],
     key: Callable[[T], Key] = _default_key,
-    equal: Callable[[T, T], bool] = _default_equal,
+    equal: Callable[[T, T], bool] = operator.eq,
 ) -> DiffResult[T, T, Key]:
     """
     Compare two iterables with same type and return a `DiffResult` instance containing the differences between them.