qontract-reconcile 0.10.1rc1077__py3-none-any.whl → 0.10.1rc1079__py3-none-any.whl

{qontract_reconcile-0.10.1rc1077.dist-info → qontract_reconcile-0.10.1rc1079.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: qontract-reconcile
-Version: 0.10.1rc1077
+Version: 0.10.1rc1079
 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.1rc1077.dist-info → qontract_reconcile-0.10.1rc1079.dist-info}/RECORD

@@ -161,7 +161,7 @@ reconcile/aws_version_sync/merge_request_manager/merge_request_manager.py,sha256
 reconcile/change_owners/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 reconcile/change_owners/approver.py,sha256=Z3_11vnK2WNOxjEEXVDh0224-_-qbt9d6mBeVE-7fsc,2259
 reconcile/change_owners/bundle.py,sha256=ZIlXRo6Z2raeWSCUqYsexBdol-q-r9kWJs5O_YPaEYk,5273
-reconcile/change_owners/change_log_tracking.py,sha256=9A_fJoahIckRF_qepfKYEwZmedrcq8KTJh_KFt4DuRQ,6542
+reconcile/change_owners/change_log_tracking.py,sha256=K8x59urjxMilwhmbQff9vinTGDblinvfRvdjilwjTJU,6677
 reconcile/change_owners/change_owners.py,sha256=0HRJhDm0oW3uYJFgzynqA1gA0lbhalhSkmWOiQmr-NM,17062
 reconcile/change_owners/change_types.py,sha256=TjVtvmkU0s8w2NA6qvWQccB6PwlCrChFySlsHLYZjpE,32027
 reconcile/change_owners/changes.py,sha256=pa3cNAL-Xawh700ARJJQjY0p09NA1J2329RcE0F0MHM,17224
@@ -728,7 +728,7 @@ reconcile/utils/state.py,sha256=W0_awkLAPX18hNOF_60o73tkPxDUylqbzYNHfl_sDsk,1638
 reconcile/utils/structs.py,sha256=LcbLEg8WxfRqM6nW7NhcWN0YeqF7SQzxOgntmLs1SgY,352
 reconcile/utils/template.py,sha256=wTvRU4AnAV_o042tD4Mwls2dwWMuk7MKnde3MaCjaYg,331
 reconcile/utils/terraform_client.py,sha256=LjX2U2E0Dglt2S_KA5jWQ_dVC8sPn4FEAh0xW_d6JTk,35953
-reconcile/utils/terrascript_aws_client.py,sha256=SMsJaOmpn_9QQHhKIN_5ps1zOa9zEcBSsxGOyvF8voU,280492
+reconcile/utils/terrascript_aws_client.py,sha256=YqMsqSJl5_P8kXPzcaWLAMxdIxhq1supwL94Xd5qPB4,281298
 reconcile/utils/three_way_diff_strategy.py,sha256=oQcHXd9LVhirJfoaOBoHUYuZVGfyL2voKr6KVI34zZE,4833
 reconcile/utils/throughput.py,sha256=iP4UWAe2LVhDo69mPPmgo9nQ7RxHD6_GS8MZe-aSiuM,344
 reconcile/utils/vault.py,sha256=9GSNHku8tw5KM2LKpZ1myWYDLtLGUJgpSnD0DxbzeO0,14956
@@ -870,8 +870,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.1rc1077.dist-info/METADATA,sha256=bJLx0VrFGijgrDJY3dSyvsOjYFXa2Eso9a29xaIXOXs,2213
-qontract_reconcile-0.10.1rc1077.dist-info/WHEEL,sha256=eOLhNAGa2EW3wWl_TU484h7q1UNgy0JXjjoqKoxAAQc,92
-qontract_reconcile-0.10.1rc1077.dist-info/entry_points.txt,sha256=GKQqCl2j2X1BJQ69een6rHcR26PmnxnONLNOQB-nRjY,491
-qontract_reconcile-0.10.1rc1077.dist-info/top_level.txt,sha256=l5ISPoXzt0SdR4jVdkfa7RPSKNc8zAHYWAnR-Dw8Ey8,24
-qontract_reconcile-0.10.1rc1077.dist-info/RECORD,,
+qontract_reconcile-0.10.1rc1079.dist-info/METADATA,sha256=DXIyJxZvXSTDEpfJMVm9-gHPlcs369V7T2STe6DHG9s,2213
+qontract_reconcile-0.10.1rc1079.dist-info/WHEEL,sha256=eOLhNAGa2EW3wWl_TU484h7q1UNgy0JXjjoqKoxAAQc,92
+qontract_reconcile-0.10.1rc1079.dist-info/entry_points.txt,sha256=GKQqCl2j2X1BJQ69een6rHcR26PmnxnONLNOQB-nRjY,491
+qontract_reconcile-0.10.1rc1079.dist-info/top_level.txt,sha256=l5ISPoXzt0SdR4jVdkfa7RPSKNc8zAHYWAnR-Dw8Ey8,24
+qontract_reconcile-0.10.1rc1079.dist-info/RECORD,,

reconcile/change_owners/change_log_tracking.py

@@ -135,7 +135,11 @@ class ChangeLogIntegration(QontractReconcileIntegration[ChangeLogIntegrationPara
                     case "/app-sre/app-1.yml":
                         changed_apps = {c["name"] for c in change_versions}
                         change_log_item.apps.extend(changed_apps)
-                    case "/app-sre/saas-file-2.yml" | "/openshift/namespace-1.yml":
+                    case (
+                        "/app-sre/saas-file-2.yml"
+                        | "/openshift/namespace-1.yml"
+                        | "/dependencies/jenkins-config-1.yml"
+                    ):
                         changed_apps = {
                             name
                             for c in change_versions

reconcile/utils/terrascript_aws_client.py

@@ -287,6 +287,10 @@ class OutputResourceNameNotUniqueException(Exception):
         )
 
 
+class RDSParameterGroupValidationError(Exception):
+    pass
+
+
 class StateInaccessibleException(Exception):
     pass
 
@@ -1670,6 +1674,16 @@ class TerrascriptClient:  # pylint: disable=too-many-public-methods
                 elif provider != provider_region:
                     raise ValueError("region does not match availability zone")
 
+        def validate_parameter_group(parameter_group: aws_db_parameter_group) -> None:
+            parameter_group_name = parameter_group.get("name")
+            for parameter in parameter_group.get("parameter", []):
+                if parameter.get("name") == "rds.logical_replication":
+                    apply_method = parameter.get("apply_method")
+                    if apply_method != "pending-reboot":
+                        raise RDSParameterGroupValidationError(
+                            f"{parameter_group_name=} rds.logical_replication {apply_method=} must be set to 'pending-reboot'"
+                        )
+
         def populate_parameter_group(name: str) -> aws_db_parameter_group:
             pg_values = self.get_values(name)
             # Parameter group name is not required by terraform.
@@ -1683,7 +1697,9 @@ class TerrascriptClient:  # pylint: disable=too-many-public-methods
             pg_values["parameter"] = pg_values.pop("parameters")
             if self._multiregion_account(account) and len(provider) > 0:
                 pg_values["provider"] = provider
-            return aws_db_parameter_group(pg_identifier, **pg_values)
+            parameter_group = aws_db_parameter_group(pg_identifier, **pg_values)
+            validate_parameter_group(parameter_group=parameter_group)
+            return parameter_group
 
         # 'deps' should contain a list of terraform resource names
         # (not full objects) that must be created