qontract-reconcile 0.10.2.dev263__py3-none-any.whl → 0.10.2.dev264__py3-none-any.whl

{qontract_reconcile-0.10.2.dev263.dist-info → qontract_reconcile-0.10.2.dev264.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: qontract-reconcile
-Version: 0.10.2.dev263
+Version: 0.10.2.dev264
 Summary: Collection of tools to reconcile services with their desired state as defined in the app-interface DB.
 Project-URL: homepage, https://github.com/app-sre/qontract-reconcile
 Project-URL: repository, https://github.com/app-sre/qontract-reconcile

{qontract_reconcile-0.10.2.dev263.dist-info → qontract_reconcile-0.10.2.dev264.dist-info}/RECORD

@@ -8,7 +8,7 @@ reconcile/aws_iam_password_reset.py,sha256=FpAqAXngmLFqkOtOjrz_i90qteUyLHJL0GMjo
 reconcile/aws_support_cases_sos.py,sha256=PDhilxQ4TBxVnxUPIUdTbKEaNUI0wzPiEsB91oHT2fY,3384
 reconcile/blackbox_exporter_endpoint_monitoring.py,sha256=O1wFp52EyF538c6txaWBs8eMtUIy19gyHZ6VzJ6QXS8,3512
 reconcile/checkpoint.py,sha256=gjtS8g6KIyKFYlHMSZjAqDUOlVh83nh4go-9yNrhWZU,5016
-reconcile/cli.py,sha256=GJ15k4DljB0aMHvtsX7zxPI-tY1j4DTHpd9FUBPSkKk,112289
+reconcile/cli.py,sha256=C4x30JxgSmKfqhOliWlJaBMsJjtLkd7UpWC9c5wjB4A,112236
 reconcile/closedbox_endpoint_monitoring_base.py,sha256=_OKz7K7HHw0-gzxeEma8PcUCtd70pRBy7JMoaAm8IVU,4940
 reconcile/cluster_deployment_mapper.py,sha256=5gumAaRCcFXsabUJ1dnuUy9WrP_FEEM5JnOnE8ch9sE,2326
 reconcile/dashdotdb_base.py,sha256=83ZWIf5JJk3P_D69y2TmXRcQr6ELJGlv10OM0h7fJVs,4767
@@ -43,21 +43,21 @@ reconcile/jenkins_webhooks_cleaner.py,sha256=tFbAzsFGvJ6UrHRZFdIuLdqG-Ocd5_Oknfs
 reconcile/jenkins_worker_fleets.py,sha256=azjrrZ2mJyheuh96Cpv8eGkKO_86uKVlQzcSq0NQ2TI,5382
 reconcile/jira_permissions_validator.py,sha256=nVHZg7kNn04Q-ryNM20wthMrhXos28g3O9b0ahzxAKc,14690
 reconcile/ldap_users.py,sha256=oP1CAxmgSi3zDJ3vKTPySjap6WmEX1U469FmFrov5l4,4599
-reconcile/mr_client_gateway.py,sha256=WhjMd-sIXDFCV8-rt8CEjurJ5OYB1pOD0K3o0tZRXQg,1885
-reconcile/ocm_additional_routers.py,sha256=KfcFDVbNoc6n5dHWjYdAf1_DiVqVG6Tw23WLKoV8cdg,3306
-reconcile/ocm_addons.py,sha256=qqAyqRBRbdZQvAcjb-QlSVyRAyQBZk6iVlgnI4jyi7s,3353
+reconcile/mr_client_gateway.py,sha256=3L21YncbetuUI3HYvDAEb5JX5HO5KG2CfUyjapX3w8E,2063
+reconcile/ocm_additional_routers.py,sha256=e5P_OJAaV0jp0626dJ0Hp1-xUsEKXRCnrJzFTIQBdJg,3854
+reconcile/ocm_addons.py,sha256=b59NWAMmd_zW4pxAH7hm8jzAkis0VZrUBmPYgduVlJc,3684
 reconcile/ocm_addons_upgrade_tests_trigger.py,sha256=A9zXeYG-_52DsS1dz47yDSnHz62du5XpPBlaeRa6zxY,3975
-reconcile/ocm_aws_infrastructure_access.py,sha256=rrlcQxvRTmKyYOUR1C-Nlxj_o_iCDSxc7PNoELUDiew,6892
-reconcile/ocm_clusters.py,sha256=1a6J1xYgfVLGZJe0Ob-lBFN9R6HKZ38xRjuWHQDamCo,17013
-reconcile/ocm_external_configuration_labels.py,sha256=imEpDv1RBpCSj8tHDv0R76hmNCFtcUzVNgS1yOVl8vs,3870
-reconcile/ocm_groups.py,sha256=nz6n4a0ZVDClpm1HbPfcwJ8fw3AYg7o4DCH-SBItVbs,3389
-reconcile/ocm_machine_pools.py,sha256=iiqReJMdzmvl9Ae0X1nT48WyROAoLS4pcI4Wk9NdWE0,16637
+reconcile/ocm_aws_infrastructure_access.py,sha256=f-zVCEf0ucdslhWvfeLcXxQ6Y4xtfCXUG6pxswGgWnU,7251
+reconcile/ocm_clusters.py,sha256=_nfrI9EeJ4PtLt7RWCHODqaA3G23sKkHMRCQK13twTA,17045
+reconcile/ocm_external_configuration_labels.py,sha256=dGE2-WeuLdz9kmkuRLmpbHlOkwI3GyY6Jd_cvRpsyJo,4342
+reconcile/ocm_groups.py,sha256=m-1-ho_pYadaldULF3ogCmC5GZ6nxEbbaLuME4UsMKw,4089
+reconcile/ocm_machine_pools.py,sha256=KnTYAOw25N-QRb3Y7zbNY8w8xCR55WyqVjycYz24Yxk,17243
 reconcile/ocm_update_recommended_version.py,sha256=Vi3Y2sX-OQxx1mv_xiPQXnmrpsZzGIE38No0yBcTaD4,4204
 reconcile/ocm_upgrade_scheduler_org_updater.py,sha256=aLgyInt9oIWAg0XtCiwJRUSwdPx3masKV8kHzkyEEOQ,4282
 reconcile/openshift_base.py,sha256=S9CUvTaJhO_ibfqiIJ11ezMfSxiji-CEaMvCwK48OFk,52831
 reconcile/openshift_cluster_bots.py,sha256=Dgh7QhmPn1OKAHnzvmRI9kAvzdmuW7INNj11VmcT4Os,10934
 reconcile/openshift_clusterrolebindings.py,sha256=sDgHi_t2ayE3O6zZ5CLao7uBmihxRK8K70w2GSADz-w,5822
-reconcile/openshift_groups.py,sha256=2crqdhTOMw-21VMNhsOnKySyAWLxqAwGfiVLxL7ejx8,9329
+reconcile/openshift_groups.py,sha256=zAXAF2Eej-TsUcOZqPe-dmlfXDlM8MD3AUsOC1-7Ltg,9188
 reconcile/openshift_limitranges.py,sha256=UvCGo_OQ4XoDK55TJmn55qEhhlkhLzhU12tX8nT5kPQ,3442
 reconcile/openshift_namespace_labels.py,sha256=4pC6uZICldzID0L2-cUUTC7UQawfmVXf8AEa2p24RZo,15588
 reconcile/openshift_namespaces.py,sha256=xG2scrmsi7VFeGv1Kc1d5kPY_r_JO6s5P85Ty4rRoys,5806
@@ -797,7 +797,7 @@ tools/saas_promotion_state/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJ
 tools/saas_promotion_state/saas_promotion_state.py,sha256=oF7C4hpIgyMTwTRm3Aun3cDCHIjVar65JoLp6NcJHlU,3909
 tools/sre_checkpoints/__init__.py,sha256=CDaDaywJnmRCLyl_NCcvxi-Zc0hTi_3OdwKiFOyS39I,145
 tools/sre_checkpoints/util.py,sha256=zEDbGr18ZeHNQwW8pUsr2JRjuXIPz--WAGJxZo9sv_Y,894
-qontract_reconcile-0.10.2.dev263.dist-info/METADATA,sha256=gu4n2dE6D3KKJFgAM7Cilxe7jLMAgq0LZP9GfZL5K94,24501
-qontract_reconcile-0.10.2.dev263.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
-qontract_reconcile-0.10.2.dev263.dist-info/entry_points.txt,sha256=5i9l54La3vQrDLAdwDKQWC0iG4sV9RRfOb1BpvzOWLc,698
-qontract_reconcile-0.10.2.dev263.dist-info/RECORD,,
+qontract_reconcile-0.10.2.dev264.dist-info/METADATA,sha256=6PUt4AX0OmDwYV7hjsuzytGQngumyFCkFFozm3YiFyw,24501
+qontract_reconcile-0.10.2.dev264.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
+qontract_reconcile-0.10.2.dev264.dist-info/entry_points.txt,sha256=5i9l54La3vQrDLAdwDKQWC0iG4sV9RRfOb1BpvzOWLc,698
+qontract_reconcile-0.10.2.dev264.dist-info/RECORD,,

reconcile/cli.py

@@ -2944,12 +2944,11 @@ def ocm_update_recommended_version(
 
 
 @integration.command(short_help="Manages cluster Addons in OCM.")
-@threaded()
 @click.pass_context
-def ocm_addons(ctx: click.Context, thread_pool_size: int) -> None:
+def ocm_addons(ctx: click.Context) -> None:
     import reconcile.ocm_addons
 
-    run_integration(reconcile.ocm_addons, ctx, thread_pool_size)
+    run_integration(reconcile.ocm_addons, ctx)
 
 
 @integration.command(

reconcile/mr_client_gateway.py

@@ -27,14 +27,19 @@ MR_GW_QUERY = """
 def get_mr_gateway_settings() -> Mapping[str, Any]:
     """Returns SecretReader settings"""
     gqlapi = gql.get_api()
-    settings = gqlapi.query(MR_GW_QUERY)["settings"]
+    data = gqlapi.query(MR_GW_QUERY)
+    if not data:
+        raise ValueError("no app-interface-settings found from gql query")
+    settings = data.get("settings")
     if settings:
         # assuming a single settings file for now
         return settings[0]
     raise ValueError("no app-interface-settings found")
 
 
-def init(gitlab_project_id=None, sqs_or_gitlab=None):
+def init(
+    gitlab_project_id: str | int | None = None, sqs_or_gitlab: str | None = None
+) -> GitLabApi | SQSGateway:
     """
     Creates the Merge Request client to of a given type.
 

reconcile/ocm_additional_routers.py

@@ -1,7 +1,7 @@
 import json
 import logging
 import sys
-from collections.abc import Mapping
+from collections.abc import Iterable, Mapping, MutableMapping
 from typing import Any
 
 from reconcile import queries
@@ -17,7 +17,9 @@ QONTRACT_INTEGRATION = "ocm-additional-routers"
 SUPPORTED_OCM_PRODUCTS = [OCM_PRODUCT_OSD]
 
 
-def fetch_current_state(clusters):
+def fetch_current_state(
+    clusters: list[Mapping[str, Any]],
+) -> tuple[OCMMap, list[dict[str, Any]]]:
     settings = queries.get_app_interface_settings()
     ocm_map = OCMMap(
         clusters=clusters, integration=QONTRACT_INTEGRATION, settings=settings
@@ -35,13 +37,13 @@ def fetch_current_state(clusters):
     return ocm_map, current_state
 
 
-def fetch_desired_state(clusters):
+def fetch_desired_state(clusters: Iterable[Mapping[str, Any]]) -> list[dict[str, Any]]:
     desired_state = []
     for cluster in clusters:
         cluster_name = cluster["name"]
         for router in cluster["additionalRouters"]:
             listening = "internal" if router["private"] else "external"
-            item = {"listening": listening, "cluster": cluster_name}
+            item: dict[str, Any] = {"listening": listening, "cluster": cluster_name}
             selectors = router.get("route_selectors", None)
             if selectors:
                 item["route_selectors"] = json.loads(selectors)
@@ -50,31 +52,34 @@ def fetch_desired_state(clusters):
     return desired_state
 
 
-def calculate_diff(current_state, desired_state):
+def calculate_diff(
+    current_state: Iterable[MutableMapping[str, Any]],
+    desired_state: Iterable[MutableMapping[str, Any]],
+) -> list[MutableMapping[str, Any]]:
     diffs = []
-    for d in desired_state:
-        c = [c for c in current_state if d.items() <= c.items()]
-        if not c:
-            d["action"] = "create"
-            diffs.append(d)
-
-    for c in current_state:
-        d = [d for d in desired_state if d.items() <= c.items()]
-        if not d:
-            c["action"] = "delete"
-            diffs.append(c)
+    for d_item in desired_state:
+        c_items = [c for c in current_state if d_item.items() <= c.items()]
+        if not c_items:
+            d_item["action"] = "create"
+            diffs.append(d_item)
+
+    for c_item in current_state:
+        d_items = [d for d in desired_state if d.items() <= c_item.items()]
+        if not d_items:
+            c_item["action"] = "delete"
+            diffs.append(c_item)
 
     return diffs
 
 
-def sort_diffs(diff):
+def sort_diffs(diff: Mapping[str, Any]) -> int:
     """Sort diffs so we delete first and create later"""
     if diff["action"] == "delete":
         return 1
     return 2
 
 
-def act(dry_run, diffs, ocm_map):
+def act(dry_run: bool, diffs: list[MutableMapping[str, Any]], ocm_map: OCMMap) -> None:
     diffs.sort(key=sort_diffs)
     for diff in diffs:
         action = diff.pop("action")
@@ -82,6 +87,9 @@ def act(dry_run, diffs, ocm_map):
         logging.info([action, cluster])
         if not dry_run:
             ocm = ocm_map.get(cluster)
+            if ocm is None:
+                logging.error(f"OCM client for cluster {cluster} not found.")
+                continue
             if action == "create":
                 ocm.create_additional_router(cluster, diff)
             elif action == "delete":
@@ -96,11 +104,10 @@ def _cluster_is_compatible(cluster: Mapping[str, Any]) -> bool:
     )
 
 
-def run(dry_run):
-    clusters = queries.get_clusters()
+def run(dry_run: bool) -> None:
     clusters = [
         c
-        for c in clusters
+        for c in queries.get_clusters()
         if integration_is_enabled(QONTRACT_INTEGRATION, c) and _cluster_is_compatible(c)
     ]
     if not clusters:

reconcile/ocm_addons.py

@@ -1,6 +1,6 @@
 import logging
 import sys
-from collections.abc import Mapping
+from collections.abc import Iterable, Mapping, MutableMapping
 from operator import itemgetter
 from typing import Any
 
@@ -12,7 +12,9 @@ from reconcile.utils.ocm import OCMMap
 QONTRACT_INTEGRATION = "ocm-addons"
 
 
-def fetch_current_state(clusters):
+def fetch_current_state(
+    clusters: Iterable[Mapping[str, Any]],
+) -> tuple[OCMMap, list[dict[str, Any]]]:
     settings = queries.get_app_interface_settings()
     ocm_map = OCMMap(
         clusters=clusters,
@@ -35,7 +37,7 @@ def fetch_current_state(clusters):
     return ocm_map, current_state
 
 
-def fetch_desired_state(clusters):
+def fetch_desired_state(clusters: Iterable[Mapping[str, Any]]) -> list[dict[str, Any]]:
     desired_state = []
     for cluster in clusters:
         cluster_name = cluster["name"]
@@ -49,13 +51,16 @@ def fetch_desired_state(clusters):
     return desired_state
 
 
-def sort_parameters(addon):
+def sort_parameters(addon: MutableMapping[str, Any]) -> None:
     addon_parameters = addon.get("parameters")
     if addon_parameters:
         addon["parameters"] = sorted(addon_parameters, key=itemgetter("id"))
 
 
-def calculate_diff(current_state, desired_state):
+def calculate_diff(
+    current_state: Iterable[Mapping[str, Any]],
+    desired_state: Iterable[MutableMapping[str, Any]],
+) -> list[MutableMapping[str, Any]]:
     diffs = []
     for d in desired_state:
         c = [c for c in current_state if d.items() <= c.items()]
@@ -66,7 +71,9 @@ def calculate_diff(current_state, desired_state):
     return diffs
 
 
-def act(dry_run, diffs, ocm_map):
+def act(
+    dry_run: bool, diffs: Iterable[MutableMapping[str, Any]], ocm_map: OCMMap
+) -> bool:
     err = False
     for diff in diffs:
         action = diff.pop("action")
@@ -95,7 +102,7 @@ def _cluster_is_compatible(cluster: Mapping[str, Any]) -> bool:
     return cluster.get("ocm") is not None and cluster.get("addons") is not None
 
 
-def run(dry_run, gitlab_project_id=None, thread_pool_size=10):
+def run(dry_run: bool) -> None:
     clusters = queries.get_clusters()
     clusters = [
         c

reconcile/ocm_aws_infrastructure_access.py

@@ -1,6 +1,6 @@
 import logging
 import sys
-from collections.abc import Mapping
+from collections.abc import Iterable, Mapping
 from typing import Any
 
 from reconcile import queries
@@ -22,7 +22,9 @@ QONTRACT_INTEGRATION = "ocm-aws-infrastructure-access"
 SUPPORTED_OCM_PRODUCTS = [OCM_PRODUCT_OSD]
 
 
-def fetch_current_state(clusters):
+def fetch_current_state(
+    clusters: Iterable[Mapping[str, Any]],
+) -> tuple[OCMMap, list[dict[str, Any]], list[dict[str, Any]], list[dict[str, Any]]]:
     current_state = []
     current_failed = []
     current_deleting = []
@@ -52,7 +54,7 @@ def fetch_current_state(clusters):
     return ocm_map, current_state, current_failed, current_deleting
 
 
-def fetch_desired_state(clusters):
+def fetch_desired_state(clusters: Iterable[Mapping[str, Any]]) -> list[dict[str, Any]]:
     desired_state = []
 
     for cluster_info in clusters:
@@ -129,8 +131,13 @@ def fetch_desired_state(clusters):
 
 
 def act(
-    dry_run, ocm_map, current_state, current_failed, desired_state, current_deleting
-):
+    dry_run: bool,
+    ocm_map: OCMMap,
+    current_state: list[dict[str, Any]],
+    current_failed: Iterable[dict[str, Any]],
+    desired_state: Iterable[dict[str, Any]],
+    current_deleting: list[dict[str, Any]],
+) -> None:
     to_delete = [c for c in current_state if c not in desired_state]
     to_delete += current_failed
     for item in to_delete:
@@ -173,7 +180,7 @@ def _cluster_is_compatible(cluster: Mapping[str, Any]) -> bool:
     )
 
 
-def get_clusters():
+def get_clusters() -> list[dict[str, Any]]:
     return [
         c
         for c in queries.get_clusters(aws_infrastructure_access=True)
@@ -181,7 +188,7 @@ def get_clusters():
     ]
 
 
-def run(dry_run):
+def run(dry_run: bool) -> None:
     clusters = get_clusters()
     if not clusters:
         logging.debug(

reconcile/ocm_clusters.py

@@ -40,7 +40,7 @@ QONTRACT_INTEGRATION = "ocm-clusters"
 QONTRACT_INTEGRATION_VERSION = make_semver(0, 1, 0)
 
 
-def _set_rosa_ocm_attrs(cluster: Mapping[str, Any]):
+def _set_rosa_ocm_attrs(cluster: Mapping[str, Any]) -> None:
     """Cluster account (aws) attribute from app-interface differs from the OCMSpec.
     app-interface's account includes the details for all the OCM environments
     but the cluster only needs the target OCM environment where it belongs.
@@ -78,8 +78,7 @@ def _set_rosa_ocm_attrs(cluster: Mapping[str, Any]):
 
     # doing this allows to exclude account fields which can be queried in graphql
     rosa_cluster_aws_account = ROSAClusterAWSAccount(
-        uid=uid,
-        rosa=rosa,
+        uid=uid, rosa=rosa, billing_account_id=None
     )
     if billing_account := account.get("billingAccount"):
         rosa_cluster_aws_account.billing_account_id = billing_account["uid"]
@@ -293,7 +292,7 @@ def get_cluster_ocm_update_spec(
 
 def _app_interface_updates_mr(
     clusters_updates: Mapping[str, Any], gitlab_project_id: str | None, dry_run: bool
-):
+) -> None:
     """Creates an MR to app-interface with the necessary cluster manifest updates
 
     :param clusters_updates: Updates to perform. Format required by the MR utils code

reconcile/ocm_external_configuration_labels.py

@@ -1,7 +1,7 @@
 import json
 import logging
 import sys
-from collections.abc import Mapping
+from collections.abc import Iterable, Mapping
 from typing import Any
 
 from reconcile import queries
@@ -12,14 +12,16 @@ from reconcile.utils.ocm import OCMMap
 QONTRACT_INTEGRATION = "ocm-external-configuration-labels"
 
 
-def get_allowed_labels_for_cluster(cluster: dict[str, Any]) -> set[str]:
+def get_allowed_labels_for_cluster(cluster: Mapping[str, Any]) -> set[str]:
     allowed_labels = cluster.get("ocm", {}).get(
         "allowedClusterExternalConfigLabels", []
     )
     return set(allowed_labels)
 
 
-def fetch_current_state(clusters):
+def fetch_current_state(
+    clusters: Iterable[Mapping[str, Any]],
+) -> tuple[OCMMap, list[dict[str, Any]]]:
     settings = queries.get_app_interface_settings()
     ocm_map = OCMMap(
         clusters=clusters, integration=QONTRACT_INTEGRATION, settings=settings
@@ -34,13 +36,16 @@ def fetch_current_state(clusters):
         for key, value in labels.items():
             if key not in allowed_labels:
                 continue
-            item = {"label": {"key": key, "value": value}, "cluster": cluster_name}
+            item = {
+                "label": {"key": key, "value": value},
+                "cluster": cluster_name,
+            }
             current_state.append(item)
 
     return ocm_map, current_state
 
 
-def fetch_desired_state(clusters):
+def fetch_desired_state(clusters: Iterable[Mapping[str, Any]]) -> list[dict[str, Any]]:
     desired_state = []
     for cluster in clusters:
         cluster_name = cluster["name"]
@@ -57,32 +62,35 @@ def fetch_desired_state(clusters):
     return desired_state
 
 
-def calculate_diff(current_state, desired_state):
+def calculate_diff(
+    current_state: Iterable[dict[str, Any]],
+    desired_state: Iterable[dict[str, Any]],
+) -> tuple[list[dict[str, Any]], bool]:
     diffs = []
     err = False
-    for d in desired_state:
-        c = [c for c in current_state if d == c]
-        if not c:
-            d["action"] = "create"
-            diffs.append(d)
-
-    for c in current_state:
-        d = [d for d in desired_state if c == d]
-        if not d:
-            c["action"] = "delete"
-            diffs.append(c)
+    for d_item in desired_state:
+        c_items = [c for c in current_state if d_item == c]
+        if not c_items:
+            d_item["action"] = "create"
+            diffs.append(d_item)
+
+    for c_item in current_state:
+        d_items = [d for d in desired_state if c_item == d]
+        if not d_items:
+            c_item["action"] = "delete"
+            diffs.append(c_item)
 
     return diffs, err
 
 
-def sort_diffs(diff):
+def sort_diffs(diff: Mapping[str, Any]) -> int:
     """Sort diffs so we delete first and create later"""
     if diff["action"] == "delete":
         return 1
     return 2
 
 
-def act(dry_run, diffs, ocm_map):
+def act(dry_run: bool, diffs: list[dict[str, Any]], ocm_map: OCMMap) -> None:
     diffs.sort(key=sort_diffs)
     for diff in diffs:
         action = diff["action"]
@@ -104,7 +112,11 @@ def _cluster_is_compatible(cluster: Mapping[str, Any]) -> bool:
     )
 
 
-def run(dry_run, gitlab_project_id=None, thread_pool_size=10):
+def run(
+    dry_run: bool,
+    gitlab_project_id: str | None = None,
+    thread_pool_size: int = 10,
+) -> None:
     clusters = queries.get_clusters()
     clusters = [
         c

reconcile/ocm_groups.py

@@ -1,7 +1,10 @@
 import itertools
 import logging
 import sys
-from collections.abc import Mapping
+from collections.abc import (
+    Iterable,
+    Mapping,
+)
 from typing import Any
 
 from sretoolbox.utils import threaded
@@ -18,7 +21,20 @@ from reconcile.utils.ocm.base import OCMClusterGroupId
 QONTRACT_INTEGRATION = "ocm-groups"
 
 
-def get_cluster_state(group_items, ocm_map):
+def create_groups_list(clusters: Iterable[Mapping[str, Any]]) -> list[dict[str, str]]:
+    groups_list: list[dict[str, str]] = []
+    for cluster_info in clusters:
+        cluster = cluster_info["name"]
+        groups = cluster_info["managedGroups"] or []
+        groups_list.extend(
+            {"cluster": cluster, "group_name": group_name} for group_name in groups
+        )
+    return groups_list
+
+
+def get_cluster_state(
+    group_items: Mapping[str, str], ocm_map: OCMMap
+) -> list[dict[str, str]]:
     cluster = group_items["cluster"]
     ocm = ocm_map.get(cluster)
     group_name = group_items["group_name"]
@@ -27,17 +43,18 @@ def get_cluster_state(group_items, ocm_map):
         return []
     return [
         {"cluster": cluster, "group": group_name, "user": user}
-        for user in group["users"] or []
+        for user in group.get("users") or []
     ]
 
 
-def fetch_current_state(clusters, thread_pool_size):
-    current_state = []
+def fetch_current_state(
+    clusters: Iterable[Mapping[str, Any]], thread_pool_size: int
+) -> tuple[OCMMap, list[dict[str, str]]]:
     settings = queries.get_app_interface_settings()
     ocm_map = OCMMap(
         clusters=clusters, integration=QONTRACT_INTEGRATION, settings=settings
     )
-    groups_list = openshift_groups.create_groups_list(clusters, oc_map=ocm_map)
+    groups_list = create_groups_list(clusters)
     results = threaded.run(
         get_cluster_state, groups_list, thread_pool_size, ocm_map=ocm_map
     )
@@ -46,7 +63,7 @@ def fetch_current_state(clusters, thread_pool_size):
     return ocm_map, current_state
 
 
-def act(diff, ocm_map):
+def act(diff: Mapping[str, Any], ocm_map: OCMMap) -> None:
     cluster = diff["cluster"]
     group = diff["group"]
     user = diff["user"]
@@ -63,8 +80,12 @@ def _cluster_is_compatible(cluster: Mapping[str, Any]) -> bool:
     return cluster.get("ocm") is not None
 
 
-def run(dry_run, thread_pool_size=10):
+def run(dry_run: bool, thread_pool_size: int = 10) -> None:
     clusters = queries.get_clusters()
+    if not clusters:
+        logging.debug("No clusters found in app-interface")
+        sys.exit(ExitCodes.SUCCESS)
+
     clusters = [
         c
         for c in clusters
@@ -97,10 +118,10 @@ def run(dry_run, thread_pool_size=10):
             act(diff, ocm_map)
 
 
-def early_exit_desired_state(*args, **kwargs) -> dict[str, Any]:
+def early_exit_desired_state(*args: Any, **kwargs: Any) -> dict[str, Any]:
     clusters = [
         c["name"]
-        for c in queries.get_clusters()
+        for c in queries.get_clusters() or []
         if integration_is_enabled(QONTRACT_INTEGRATION, c) and _cluster_is_compatible(c)
     ]
     desired_state = openshift_groups.fetch_desired_state(clusters=clusters)

reconcile/ocm_machine_pools.py

@@ -5,6 +5,7 @@ from abc import (
 )
 from collections.abc import Iterable, Mapping
 from enum import Enum
+from typing import Any, Self
 
 from pydantic import (
     BaseModel,
@@ -48,11 +49,11 @@ class AbstractAutoscaling(BaseModel):
         )
 
     @abstractmethod
-    def get_min(self):
+    def get_min(self) -> int:
         pass
 
     @abstractmethod
-    def get_max(self):
+    def get_max(self) -> int:
         pass
 
 
@@ -61,9 +62,12 @@ class MachinePoolAutoscaling(AbstractAutoscaling):
     max_replicas: int
 
     @root_validator()
-    @classmethod
-    def max_greater_min(cls, field_values):
-        if field_values.get("min_replicas") > field_values.get("max_replicas"):
+    def max_greater_min(cls, field_values: Mapping[str, Any]) -> Mapping[str, Any]:
+        min_replicas = field_values.get("min_replicas")
+        max_replicas = field_values.get("max_replicas")
+        if min_replicas is None or max_replicas is None:
+            raise ValueError("min_replicas and max_replicas must be set")
+        if min_replicas > max_replicas:
             raise ValueError("max_replicas must be greater than min_replicas")
         return field_values
 
@@ -79,10 +83,13 @@ class NodePoolAutoscaling(AbstractAutoscaling):
     max_replica: int
 
     @root_validator()
-    @classmethod
-    def max_greater_min(cls, field_values):
-        if field_values.get("min_replica") > field_values.get("max_replica"):
-            raise ValueError("max_replicas must be greater than min_replicas")
+    def max_greater_min(cls, field_values: Mapping[str, Any]) -> Mapping[str, Any]:
+        min_replica = field_values.get("min_replica")
+        max_replica = field_values.get("max_replica")
+        if min_replica is None or max_replica is None:
+            raise ValueError("min_replica and max_replica must be set")
+        if min_replica > max_replica:
+            raise ValueError("max_replica must be greater than min_replica")
         return field_values
 
     def get_min(self) -> int:
@@ -104,8 +111,7 @@ class AbstractPool(ABC, BaseModel):
     autoscaling: AbstractAutoscaling | None
 
     @root_validator()
-    @classmethod
-    def validate_scaling(cls, field_values):
+    def validate_scaling(cls, field_values: Mapping[str, Any]) -> Mapping[str, Any]:
         if field_values.get("autoscaling") and field_values.get("replicas"):
             raise ValueError("autoscaling and replicas are mutually exclusive")
         return field_values
@@ -134,14 +140,12 @@ class AbstractPool(ABC, BaseModel):
     def deletable(self) -> bool:
         pass
 
-    def _has_diff_autoscale(self, pool):
-        match (self.autoscaling, pool.autoscale):
-            case (None, None):
-                return False
-            case (None, _) | (_, None):
-                return True
-            case _:
-                return self.autoscaling.has_diff(pool.autoscale)
+    def _has_diff_autoscale(self, pool: ClusterMachinePoolV1) -> bool:
+        if self.autoscaling is None and pool.autoscale is None:
+            return False
+        if self.autoscaling is None or pool.autoscale is None:
+            return True
+        return self.autoscaling.has_diff(pool.autoscale)
 
 
 class MachinePool(AbstractPool):
@@ -198,7 +202,7 @@ class MachinePool(AbstractPool):
         pool: ClusterMachinePoolV1,
         cluster: str,
         cluster_type: ClusterType,
-    ):
+    ) -> Self:
         autoscaling: MachinePoolAutoscaling | None = None
         if pool.autoscale:
             autoscaling = MachinePoolAutoscaling(
@@ -278,7 +282,7 @@ class NodePool(AbstractPool):
         pool: ClusterMachinePoolV1,
         cluster: str,
         cluster_type: ClusterType,
-    ):
+    ) -> Self:
         autoscaling: NodePoolAutoscaling | None = None
         if pool.autoscale:
             autoscaling = NodePoolAutoscaling(
@@ -369,7 +373,7 @@ def _classify_cluster_type(cluster: ClusterV1) -> ClusterType:
             raise ValueError(f"unknown cluster type for cluster {cluster.name}")
 
 
-def fetch_current_state_for_cluster(cluster, ocm):
+def fetch_current_state_for_cluster(cluster: ClusterV1, ocm: OCM) -> list[AbstractPool]:
     cluster_type = _classify_cluster_type(cluster)
     if cluster_type == ClusterType.ROSA_HCP:
         return [
@@ -505,14 +509,15 @@ def act(dry_run: bool, diffs: Iterable[PoolHandler], ocm_map: OCMMap) -> None:
         logging.info([diff.action, diff.pool.cluster, diff.pool.id])
         if not dry_run:
             ocm = ocm_map.get(diff.pool.cluster)
-            diff.act(dry_run, ocm)
+            if ocm is not None:
+                diff.act(dry_run, ocm)
 
 
 def _cluster_is_compatible(cluster: ClusterV1) -> bool:
     return cluster.ocm is not None and cluster.machine_pools is not None
 
 
-def run(dry_run: bool):
+def run(dry_run: bool) -> None:
     clusters = get_clusters()
 
     filtered_clusters = [

reconcile/openshift_groups.py

@@ -5,11 +5,11 @@ from collections.abc import (
     Iterable,
     Mapping,
 )
-from typing import Any
 
 from sretoolbox.utils import threaded
 
 import reconcile.openshift_base as ob
+from reconcile.gql_definitions.common.clusters import ClusterV1
 from reconcile.gql_definitions.openshift_groups.managed_groups import (
     query as query_managed_groups,
 )
@@ -62,20 +62,16 @@ def get_cluster_state(
 
 
 def create_groups_list(
-    clusters: Iterable[Mapping[str, Any]], oc_map: ClusterMap
+    clusters: Iterable[ClusterV1], oc_map: ClusterMap
 ) -> list[dict[str, str]]:
-    """
-    Also used by ocm-groups integration and thus requires to work with dict for now
-    """
     groups_list: list[dict[str, str]] = []
-    for cluster_info in clusters:
-        cluster = cluster_info["name"]
-        oc = oc_map.get(cluster)
+    for cluster in clusters:
+        oc = oc_map.get(cluster.name)
         if isinstance(oc, OCLogMsg):
             logging.log(level=oc.log_level, msg=oc.message)
-        groups = cluster_info["managedGroups"] or []
+        groups = cluster.managed_groups or []
         groups_list.extend(
-            {"cluster": cluster, "group_name": group_name} for group_name in groups
+            {"cluster": cluster.name, "group_name": group_name} for group_name in groups
         )
     return groups_list
 
@@ -97,7 +93,7 @@ def fetch_current_state(
         thread_pool_size=thread_pool_size,
     )
 
-    groups_list = create_groups_list([c.dict(by_alias=True) for c in clusters], oc_map)
+    groups_list = create_groups_list(clusters, oc_map)
     results = threaded.run(
         get_cluster_state, groups_list, thread_pool_size, oc_map=oc_map
     )