qontract-reconcile 0.10.1rc1014__py3-none-any.whl → 0.10.1rc1015__py3-none-any.whl

{qontract_reconcile-0.10.1rc1014.dist-info → qontract_reconcile-0.10.1rc1015.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: qontract-reconcile
-Version: 0.10.1rc1014
+Version: 0.10.1rc1015
 Summary: Collection of tools to reconcile services with their desired state as defined in the app-interface DB.
 Home-page: https://github.com/app-sre/qontract-reconcile
 Author: Red Hat App-SRE Team

{qontract_reconcile-0.10.1rc1014.dist-info → qontract_reconcile-0.10.1rc1015.dist-info}/RECORD

@@ -181,7 +181,7 @@ reconcile/cna/assets/asset_factory.py,sha256=7T7X_J6xIsoGETqBRI45_EyIKEdQcnRPt_G
 reconcile/cna/assets/null.py,sha256=85mVh97atCoC0aLuX47poTZiyOthmziJeBsUw0c924w,1658
 reconcile/dynatrace_token_provider/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 reconcile/dynatrace_token_provider/dependencies.py,sha256=41q05A4C_eS3E8-MR4veeMxtQNsPoGdxmEa3d-OKxq4,2814
-reconcile/dynatrace_token_provider/integration.py,sha256=ffH4BpMNb3AafwengwzurZ-aiztGP1MUlnVU4FnO3IY,21540
+reconcile/dynatrace_token_provider/integration.py,sha256=QY-k5vsbBOm80yW_RB6G2JZD5NY21zCyqHigos7GjRM,20876
 reconcile/dynatrace_token_provider/metrics.py,sha256=xiKkl8fTEBQaXJelGCPNTZhHAWdO1M3pCXNr_Tei63c,1285
 reconcile/dynatrace_token_provider/model.py,sha256=gkpqo5rRRueBXnIMjp4EEHqBUBuU65TRI8zpdb8GJ0A,241
 reconcile/dynatrace_token_provider/ocm.py,sha256=iHMsgbsLs-dlrB9UXmWNDF7E4UDe49JOsLa9rnowKfo,4282
@@ -744,7 +744,7 @@ reconcile/utils/clusterhealth/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5N
 reconcile/utils/clusterhealth/providerbase.py,sha256=DXomGYogckBLqWtXn0PXU0hWYxB6K0F7ernldrkHhVY,1140
 reconcile/utils/clusterhealth/telemeter.py,sha256=PllSLsJXvGNatmTF4mxCNPVbDrpr_MPk0m5pWj-LT6g,1534
 reconcile/utils/dynatrace/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-reconcile/utils/dynatrace/client.py,sha256=siVDdCQeLaWm4W1T-QKnFwlotPmXmZfpyoe3pXowBt8,2760
+reconcile/utils/dynatrace/client.py,sha256=dVaTwIlLhutv_2sxL_ISIQkN5br1Tfm14dhVP_hwePI,2834
 reconcile/utils/glitchtip/__init__.py,sha256=FT6iBhGqoe7KExFdbgL8AYUb64iW_4snF5__Dcl7yt0,258
 reconcile/utils/glitchtip/client.py,sha256=ovh4tx-ajlihjvcq6nyY4chulbuMJYvzDPv9j9CuAKM,7867
 reconcile/utils/glitchtip/models.py,sha256=uHbCK9-RWgxNYUAkEHXRAZRDDZW7jkOFFt9MdlqN4bU,6481
@@ -859,8 +859,8 @@ tools/test/test_qontract_cli.py,sha256=_D61RFGAN5x44CY1tYbouhlGXXABwYfxKSWSQx3Jr
 tools/test/test_saas_promotion_state.py,sha256=dy4kkSSAQ7bC0Xp2CociETGN-2aABEfL6FU5D9Jl00Y,6056
 tools/test/test_sd_app_sre_alert_report.py,sha256=v363r9zM7__0kR5K6mvJoGFcM9BvE33fWAayrqkpojA,2116
 tools/test/test_sre_checkpoints.py,sha256=SKqPPTl9ua0RFdSSofnoQX-JZE6dFLO3LRhfQzqtfh8,2607
-qontract_reconcile-0.10.1rc1014.dist-info/METADATA,sha256=_oYiLMwp1_SSIsoW1g1574AcwpLZcBG_NwIFvcUG_XU,2263
-qontract_reconcile-0.10.1rc1014.dist-info/WHEEL,sha256=GJ7t_kWBFywbagK5eo9IoUwLW6oyOeTKmQ-9iHFVNxQ,92
-qontract_reconcile-0.10.1rc1014.dist-info/entry_points.txt,sha256=GKQqCl2j2X1BJQ69een6rHcR26PmnxnONLNOQB-nRjY,491
-qontract_reconcile-0.10.1rc1014.dist-info/top_level.txt,sha256=l5ISPoXzt0SdR4jVdkfa7RPSKNc8zAHYWAnR-Dw8Ey8,24
-qontract_reconcile-0.10.1rc1014.dist-info/RECORD,,
+qontract_reconcile-0.10.1rc1015.dist-info/METADATA,sha256=SjLjAyfV0eoceoItqeaPqvVGbI2tMBP1Xst6__37_sM,2263
+qontract_reconcile-0.10.1rc1015.dist-info/WHEEL,sha256=GJ7t_kWBFywbagK5eo9IoUwLW6oyOeTKmQ-9iHFVNxQ,92
+qontract_reconcile-0.10.1rc1015.dist-info/entry_points.txt,sha256=GKQqCl2j2X1BJQ69een6rHcR26PmnxnONLNOQB-nRjY,491
+qontract_reconcile-0.10.1rc1015.dist-info/top_level.txt,sha256=l5ISPoXzt0SdR4jVdkfa7RPSKNc8zAHYWAnR-Dw8Ey8,24
+qontract_reconcile-0.10.1rc1015.dist-info/RECORD,,

reconcile/dynatrace_token_provider/integration.py

@@ -96,7 +96,7 @@ class DynatraceTokenProviderIntegration(
                         for cluster in clusters
                         if cluster.organization_id in self.params.ocm_organization_ids
                     ]
-                existing_dtp_tokens = {}
+                existing_dtp_tokens: dict[str, dict[str, str]] = {}
 
                 for cluster in clusters:
                     try:
@@ -139,7 +139,7 @@ class DynatraceTokenProviderIntegration(
                                 continue
                             if tenant_id not in existing_dtp_tokens:
                                 existing_dtp_tokens[tenant_id] = (
-                                    dt_client.get_token_ids_for_name_prefix(
+                                    dt_client.get_token_ids_map_for_name_prefix(
                                         prefix="dtp-"
                                     )
                                 )
@@ -172,7 +172,7 @@ class DynatraceTokenProviderIntegration(
         cluster: Cluster,
         dt_client: DynatraceClient,
         ocm_client: OCMClient,
-        existing_dtp_tokens: Iterable[str],
+        existing_dtp_tokens: Mapping[str, str],
         tenant_id: str,
         token_spec: DynatraceTokenProviderTokenSpecV1,
     ) -> None:
@@ -281,7 +281,7 @@ class DynatraceTokenProviderIntegration(
         dry_run: bool,
         current_k8s_secrets: Iterable[K8sSecret],
         desired_spec: DynatraceTokenProviderTokenSpecV1,
-        existing_dtp_tokens: Iterable[str],
+        existing_dtp_tokens: Mapping[str, str],
         dt_client: DynatraceClient,
         cluster_uuid: str,
     ) -> tuple[bool, Iterable[K8sSecret]]:
@@ -377,15 +377,12 @@ class DynatraceTokenProviderIntegration(
                 raise e
         return manifest
 
-    def get_secrets_from_syncset(
-        self, syncset: Mapping[str, Any], token_spec: DynatraceTokenProviderTokenSpecV1
+    def get_secrets_from_data(
+        self,
+        secret_data_by_name: Mapping[str, Any],
+        token_spec: DynatraceTokenProviderTokenSpecV1,
     ) -> list[K8sSecret]:
         secrets: list[K8sSecret] = []
-        secret_data_by_name = {
-            resource.get("metadata", {}).get("name"): resource.get("data", {})
-            for resource in syncset.get("resources", [])
-            if resource.get("kind") == "Secret"
-        }
         for secret in token_spec.secrets:
             secret_data = secret_data_by_name.get(secret.name)
             if secret_data:
@@ -414,42 +411,29 @@ class DynatraceTokenProviderIntegration(
                 )
         return secrets
 
+    def get_secrets_from_syncset(
+        self, syncset: Mapping[str, Any], token_spec: DynatraceTokenProviderTokenSpecV1
+    ) -> list[K8sSecret]:
+        secret_data_by_name = {
+            resource.get("metadata", {}).get("name"): resource.get("data", {})
+            for resource in syncset.get("resources", [])
+            if resource.get("kind") == "Secret"
+        }
+        return self.get_secrets_from_data(
+            secret_data_by_name=secret_data_by_name, token_spec=token_spec
+        )
+
     def get_secrets_from_manifest(
         self, manifest: Mapping[str, Any], token_spec: DynatraceTokenProviderTokenSpecV1
     ) -> list[K8sSecret]:
-        secrets: list[K8sSecret] = []
         secret_data_by_name = {
             resource.get("metadata", {}).get("name"): resource.get("data", {})
             for resource in manifest.get("workloads", [])
             if resource.get("kind") == "Secret"
         }
-        for secret in token_spec.secrets:
-            secret_data = secret_data_by_name.get(secret.name)
-            if secret_data:
-                tokens = []
-                for token in secret.tokens:
-                    token_id = self.base64_decode(
-                        secret_data.get(f"{token.key_name_in_secret}Id", "")
-                    )
-                    token_value = self.base64_decode(
-                        secret_data.get(token.key_name_in_secret, "")
-                    )
-                    tokens.append(
-                        DynatraceAPIToken(
-                            id=token_id,
-                            token=token_value,
-                            name=token.name,
-                            secret_key=token.key_name_in_secret,
-                        )
-                    )
-                secrets.append(
-                    K8sSecret(
-                        secret_name=secret.name,
-                        namespace_name=secret.namespace,
-                        tokens=tokens,
-                    )
-                )
-        return secrets
+        return self.get_secrets_from_data(
+            secret_data_by_name=secret_data_by_name, token_spec=token_spec
+        )
 
     def construct_secrets_data(
         self,

reconcile/utils/dynatrace/client.py

@@ -49,14 +49,16 @@ class DynatraceClient:
             ) from e
         return DynatraceAPITokenCreated(token=token.token, id=token.id)
 
-    def get_token_ids_for_name_prefix(self, prefix: str) -> list[str]:
+    def get_token_ids_map_for_name_prefix(self, prefix: str) -> dict[str, str]:
         try:
             dt_tokens = self._api.tokens.list()
         except Exception as e:
             raise DynatraceTokenRetrievalError(
                 f"{self._environment_url=} Failed to retrieve tokens for {prefix=}", e
             ) from e
-        return [token.id for token in dt_tokens if token.name.startswith(prefix)]
+        return {
+            token.id: token.name for token in dt_tokens if token.name.startswith(prefix)
+        }
 
     def get_token_by_id(self, token_id: str) -> DynatraceAPIToken:
         try:
@@ -67,10 +69,11 @@ class DynatraceClient:
             ) from e
         return DynatraceAPIToken(id=token.id, scopes=token.scopes)
 
-    def update_token_scopes(self, token_id: str, scopes: list[str]) -> None:
+    def update_token(self, token_id: str, name: str, scopes: list[str]) -> None:
         try:
             self._api.tokens.put(
                 token_id=token_id,
+                name=name,
                 api_token=ApiTokenUpdate(
                     scopes=scopes,
                 ),