qontract-reconcile 0.10.2.dev277__py3-none-any.whl → 0.10.2.dev279__py3-none-any.whl

{qontract_reconcile-0.10.2.dev277.dist-info → qontract_reconcile-0.10.2.dev279.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: qontract-reconcile
-Version: 0.10.2.dev277
+Version: 0.10.2.dev279
 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.dev277.dist-info → qontract_reconcile-0.10.2.dev279.dist-info}/RECORD

@@ -82,10 +82,10 @@ reconcile/openshift_tekton_resources.py,sha256=z9OidaI7Ju2O0O0PfIcdoyH62VA4TxLTD
 reconcile/openshift_upgrade_watcher.py,sha256=dGTQQdCOl95Bz0wOqg6JaMdBSSDgnAveH_hprUafhW0,6624
 reconcile/openshift_users.py,sha256=h4dH3gTCFFQID76PFuYeMWNzFQ9DgTUtsOcvxfj-3cs,5385
 reconcile/openshift_vault_secrets.py,sha256=Ax-_EBWWU1VRHYyKaUkGJkIjHGwWM3bZgjXL5CkPW8k,1883
-reconcile/quay_base.py,sha256=h5xNjb7EZm8L2JgpO42r6w0UA4im5dabZXJSIW69zKU,1987
-reconcile/quay_membership.py,sha256=1ajUJwPPudN7zmMZp81EipYljD0hAxzowaL-RbcroVA,6382
+reconcile/quay_base.py,sha256=GQkUpuEzC1V_QrHEu_PoELVGnlNRvgFnTqVKp-xkgC4,2070
+reconcile/quay_membership.py,sha256=No2sgEyTVj-hr5VPLy_xdrYAPvt-xo-CPpOt0X3x_6o,6623
 reconcile/quay_mirror.py,sha256=PBooiA0ShZpWYfO6oeKFqYYT6Syi7Q8JJD9kj0wRRLg,14030
-reconcile/quay_mirror_org.py,sha256=I-tEqRHLL6uFqbSi7qCfPuDNoae7EAI2U68NbDOhmv8,10809
+reconcile/quay_mirror_org.py,sha256=PG_WpzE4TymICs74mFt3lh7XA4Rg91y4EVREMEd8PFk,10815
 reconcile/quay_permissions.py,sha256=9KOutS1w4RFQqkvMSy54VtsKNx56-phzP6yI_rEW-B8,4244
 reconcile/quay_repos.py,sha256=cuEYG0HUe0ut5yvLdEwOF5-CmccpXQHRb_wDazvDrvQ,6895
 reconcile/queries.py,sha256=aaffpfMIlmHy73CUxos--NnUsTR5ms3VPeaUY3dnvIs,49879
@@ -117,7 +117,7 @@ reconcile/vpc_peerings_validator.py,sha256=aESqrhm1tpkc2iqSL1UV5to_HjNgjRSffD0cr
 reconcile/aus/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 reconcile/aus/advanced_upgrade_service.py,sha256=Cx-WthkvlcX58PrnnNeo_bQG17Xw3BGNu3f53Jc1VoA,23985
 reconcile/aus/aus_label_source.py,sha256=o0S2f0qwcII_8nzhHZhRQ83gEZ1DrSXyO4xzSwLebuU,4382
-reconcile/aus/base.py,sha256=RAtYkwCM2CIqv37XkgyzFyTiNNnImCD-yMQ698QYgAY,51451
+reconcile/aus/base.py,sha256=Gd76QvSO2PjINjsth7sEeJPrD1a0TLYtpgNzTWIzj2w,50016
 reconcile/aus/cluster_version_data.py,sha256=VZWbUEIbrDKO-sroMpQtiWCTqDraTMd8tssKV0HyTQ0,7140
 reconcile/aus/healthchecks.py,sha256=jR9c-syh9impnkV0fd6XW3Bnk7iRN5zv8oCRYM-yIRY,2700
 reconcile/aus/metrics.py,sha256=BhIvZVTn25fIzijz3xFynJngS2sXDBTxxprUUVWJcFo,4246
@@ -737,7 +737,7 @@ reconcile/utils/ocm/sre_capability_labels.py,sha256=nqh0imrYczNeeeC7ZNX3pEwuAIVk
 reconcile/utils/ocm/status_board.py,sha256=8DYeIrOsW8Bh5PCtKdvGGpaxb9Wugcc5rLxZJ8Z7b_s,4181
 reconcile/utils/ocm/subscriptions.py,sha256=hehKXsDXIhnhqvWOuiYvx6y2FGq3zt0APGYj7WiBIdI,2765
 reconcile/utils/ocm/syncsets.py,sha256=9IQm1l5BodOVZa2OFbQmow3afmh4nXe5pn-CCJ5LxTI,1169
-reconcile/utils/ocm/upgrades.py,sha256=UEgfSJuC8nH6jW9JjEx91_9QeeHR1UJ_9FVOX_PlvTU,4712
+reconcile/utils/ocm/upgrades.py,sha256=W8-sLgETI_418ftY9vBlXswyjx_KdhJTJO9cwBL3hfY,4162
 reconcile/utils/rosa/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 reconcile/utils/rosa/rosa_cli.py,sha256=zV8kKG5s3oFJwyW3E7xxNzqSE7JydrmBx169Jxm8sew,11331
 reconcile/utils/rosa/session.py,sha256=CwlQJ7eEfaMjVOt_6KoNJZc4oQj-iC-EOZq8eSGzjuI,6861
@@ -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.dev277.dist-info/METADATA,sha256=0r2ZpiyGn-ZqHRMYtgMTPoV7kOLnhwFRzHA0JQYlIsQ,24501
-qontract_reconcile-0.10.2.dev277.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
-qontract_reconcile-0.10.2.dev277.dist-info/entry_points.txt,sha256=5i9l54La3vQrDLAdwDKQWC0iG4sV9RRfOb1BpvzOWLc,698
-qontract_reconcile-0.10.2.dev277.dist-info/RECORD,,
+qontract_reconcile-0.10.2.dev279.dist-info/METADATA,sha256=4VMuEqWlD_8n_w3YF3iolN1wiU0tvQrGBovEfdKuYHE,24501
+qontract_reconcile-0.10.2.dev279.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
+qontract_reconcile-0.10.2.dev279.dist-info/entry_points.txt,sha256=5i9l54La3vQrDLAdwDKQWC0iG4sV9RRfOb1BpvzOWLc,698
+qontract_reconcile-0.10.2.dev279.dist-info/RECORD,,

reconcile/aus/base.py

@@ -83,8 +83,6 @@ from reconcile.utils.ocm.upgrades import (
     create_control_plane_upgrade_policy,
     create_node_pool_upgrade_policy,
     create_upgrade_policy,
-    delete_control_plane_upgrade_policy,
-    delete_upgrade_policy,
     get_control_plane_upgrade_policies,
     get_node_pool_upgrade_policies,
     get_upgrade_policies,
@@ -467,7 +465,7 @@ class AddonUpgradePolicy(AbstractUpgradePolicy):
 
 
 class ClusterUpgradePolicy(AbstractUpgradePolicy):
-    """Class to create and delete ClusterUpgradePolicies in OCM"""
+    """Class to create ClusterUpgradePolicies in OCM"""
 
     def create(self, ocm_api: OCMBaseClient) -> None:
         policy = {
@@ -478,11 +476,7 @@ class ClusterUpgradePolicy(AbstractUpgradePolicy):
         create_upgrade_policy(ocm_api, self.cluster.id, policy)
 
     def delete(self, ocm_api: OCMBaseClient) -> None:
-        if not self.id:
-            raise ValueError(
-                "Cannot delete cluster upgrade policy without id (not created yet)"
-            )
-        delete_upgrade_policy(ocm_api, self.cluster.id, self.id)
+        raise NotImplementedError("ClusterUpgradePolicy.delete() not implemented")
 
     def summarize(self) -> str:
         details = {
@@ -509,11 +503,7 @@ class ControlPlaneUpgradePolicy(AbstractUpgradePolicy):
         create_control_plane_upgrade_policy(ocm_api, self.cluster.id, policy)
 
     def delete(self, ocm_api: OCMBaseClient) -> None:
-        if not self.id:
-            raise ValueError(
-                "Cannot delete controlplane upgrade policy without id (not created yet)"
-            )
-        delete_control_plane_upgrade_policy(ocm_api, self.cluster.id, self.id)
+        raise NotImplementedError("ControlPlaneUpgradePolicy.delete() not implemented")
 
     def summarize(self) -> str:
         details = {
@@ -527,7 +517,7 @@ class ControlPlaneUpgradePolicy(AbstractUpgradePolicy):
 
 class NodePoolUpgradePolicy(AbstractUpgradePolicy):
     node_pool: str
-    """Class to create and delete NodePoolUpgradePolicies in OCM"""
+    """Class to create NodePoolUpgradePolicies in OCM"""
 
     def create(self, ocm_api: OCMBaseClient) -> None:
         policy = {
@@ -910,38 +900,21 @@ def upgradeable_version(
 def verify_current_should_skip(
     current_state: Sequence[AbstractUpgradePolicy],
     desired: ClusterUpgradeSpec,
-    now: datetime,
-    addon_id: str = "",
-) -> tuple[bool, UpgradePolicyHandler | None]:
+) -> bool:
     current_policies = [c for c in current_state if c.cluster.id == desired.cluster.id]
     if not current_policies:
-        return False, None
+        return False
 
     # there can only be one upgrade policy per cluster
     if len(current_policies) != 1:
         raise ValueError(
             f"[{desired.org.org_id}/{desired.cluster.name}] expected only one upgrade policy"
         )
-    current = current_policies[0]
-    version = current.version  # may not exist in automatic upgrades
-    if version and not addon_id and desired.version_blocked(version):
-        next_run = current.next_run
-        if next_run and datetime.strptime(next_run, "%Y-%m-%dT%H:%M:%SZ") < now:
-            logging.warning(
-                f"[{desired.org.org_id}/{desired.org.name}/{desired.cluster.name}] currently upgrading to blocked version '{version}'"
-            )
-            return True, None
-        logging.debug(
-            f"[{desired.org.org_id}/{desired.org.name}/{desired.cluster.name}] found planned upgrade policy "
-            + f"with blocked version {version}"
-        )
-        return False, UpgradePolicyHandler(action="delete", policy=current)
 
-    # else
     logging.debug(
         f"[{desired.org.org_id}/{desired.org.name}/{desired.cluster.name}] skipping cluster with existing upgrade policy"
     )
-    return True, None
+    return True
 
 
 def verify_schedule_should_skip(
@@ -1129,14 +1102,8 @@ def calculate_diff(
                 set_upgrading(spec.cluster.id, spec.effective_mutexes, sector_name)
                 continue
 
-        # ignore clusters with an existing upgrade policy
-        skip, delete_policy = verify_current_should_skip(
-            current_state, spec, now, addon_id
-        )
-        if skip:
+        if verify_current_should_skip(current_state, spec):
             continue
-        if delete_policy:
-            diffs.append(delete_policy)
 
         next_schedule = verify_schedule_should_skip(spec, now, addon_id)
         if not next_schedule:

reconcile/quay_base.py

@@ -1,13 +1,15 @@
 from collections import namedtuple
+from typing import Any
 
 from reconcile import queries
 from reconcile.utils.quay_api import QuayApi
 from reconcile.utils.secret_reader import SecretReader
 
 OrgKey = namedtuple("OrgKey", ["instance", "org_name"])
+QuayApiStore = dict[OrgKey, dict[str, Any]]
 
 
-def get_quay_api_store():
+def get_quay_api_store() -> QuayApiStore:
     """
     Returns a dictionary with a key for each Quay organization
     managed in app-interface.

reconcile/quay_membership.py

@@ -1,6 +1,7 @@
 import logging
 import sys
 from collections.abc import Sequence
+from typing import Any
 
 from reconcile.gql_definitions.quay_membership import quay_membership
 from reconcile.gql_definitions.quay_membership.quay_membership import (
@@ -9,13 +10,14 @@ from reconcile.gql_definitions.quay_membership.quay_membership import (
     PermissionQuayOrgTeamV1,
     UserV1,
 )
-from reconcile.quay_base import get_quay_api_store
+from reconcile.quay_base import QuayApiStore, get_quay_api_store
 from reconcile.status import ExitCodes
 from reconcile.utils import (
     expiration,
     gql,
 )
 from reconcile.utils.aggregated_list import (
+    Action,
     AggregatedDiffRunner,
     AggregatedList,
     RunnerError,
@@ -33,7 +35,7 @@ def get_permissions_for_quay_membership() -> list[PermissionQuayOrgTeamV1]:
     return [p for p in query_data.permissions if isinstance(p, PermissionQuayOrgTeamV1)]
 
 
-def process_permission(permission: PermissionQuayOrgTeamV1):
+def process_permission(permission: PermissionQuayOrgTeamV1) -> dict[str, Any]:
     """Returns a new permission object with the right keys
 
     State needs these fields: service, org, team.
@@ -52,7 +54,7 @@ def process_permission(permission: PermissionQuayOrgTeamV1):
     }
 
 
-def fetch_current_state(quay_api_store):
+def fetch_current_state(quay_api_store: QuayApiStore) -> AggregatedList:
     state = AggregatedList()
 
     for org_key, org_data in quay_api_store.items():
@@ -85,7 +87,7 @@ def get_usernames(users: Sequence[UserV1 | BotV1 | ExternalUserV1]) -> list[str]
     return [u.quay_username for u in users if u.quay_username]
 
 
-def fetch_desired_state():
+def fetch_desired_state() -> AggregatedList:
     permissions = get_permissions_for_quay_membership()
     state = AggregatedList()
 
@@ -105,14 +107,14 @@ def fetch_desired_state():
 
 
 class RunnerAction:
-    def __init__(self, dry_run, quay_api_store):
+    def __init__(self, dry_run: bool, quay_api_store: QuayApiStore):
         self.dry_run = dry_run
         self.quay_api_store = quay_api_store
 
-    def add_to_team(self):
+    def add_to_team(self) -> Action:
         label = "add_to_team"
 
-        def action(params, items):
+        def action(params: dict, items: list) -> bool:
             org = params["org"]
             team = params["team"]
 
@@ -134,7 +136,7 @@ class RunnerAction:
 
         return action
 
-    def create_team(self):
+    def create_team(self) -> Action:
         """
         Create an empty team in Quay. This method avoids adding users to the
         new team. add_to_team() will handle updating the member list the
@@ -142,7 +144,7 @@ class RunnerAction:
         """
         label = "create_team"
 
-        def action(params, items):
+        def action(params: dict, items: list) -> bool:
             org = params["org"]
             team = params["team"]
 
@@ -164,10 +166,10 @@ class RunnerAction:
 
         return action
 
-    def del_from_team(self):
+    def del_from_team(self) -> Action:
         label = "del_from_team"
 
-        def action(params, items):
+        def action(params: dict, items: list) -> bool:
             org = params["org"]
             team = params["team"]
 
@@ -185,7 +187,7 @@ class RunnerAction:
         return action
 
 
-def run(dry_run):
+def run(dry_run: bool) -> None:
     quay_api_store = get_quay_api_store()
 
     current_state = fetch_current_state(quay_api_store)

reconcile/quay_mirror_org.py

@@ -290,7 +290,7 @@ class QuayMirrorOrg:
 
 
 def run(
-    dry_run,
+    dry_run: bool,
     control_file_dir: str | None,
     compare_tags: bool | None,
     compare_tags_interval: int,

reconcile/utils/ocm/upgrades.py

@@ -77,15 +77,6 @@ def create_upgrade_policy(ocm_api: OCMBaseClient, cluster_id: str, spec: dict) -
     ocm_api.post(f"{build_cluster_url(cluster_id)}/upgrade_policies", spec)
 
 
-def delete_upgrade_policy(
-    ocm_api: OCMBaseClient, cluster_id: str, policy_id: str
-) -> None:
-    """
-    Deletes an existing Upgrade Policy
-    """
-    ocm_api.delete(f"{build_cluster_url(cluster_id)}/upgrade_policies/{policy_id}")
-
-
 #
 # CONTROL PLANE UPGRADE POLICIES
 #
@@ -120,17 +111,6 @@ def create_control_plane_upgrade_policy(
     )
 
 
-def delete_control_plane_upgrade_policy(
-    ocm_api: OCMBaseClient, cluster_id: str, upgrade_policy_id: str
-) -> None:
-    """
-    Deletes an existing Control Plane Upgrade Policy
-    """
-    ocm_api.delete(
-        f"{build_cluster_url(cluster_id)}/control_plane/upgrade_policies/{upgrade_policy_id}"
-    )
-
-
 #
 # NODE POOLUPGRADE POLICIES
 #