qontract-reconcile 0.10.1rc513__py3-none-any.whl → 0.10.1rc515__py3-none-any.whl

{qontract_reconcile-0.10.1rc513.dist-info → qontract_reconcile-0.10.1rc515.dist-info}/METADATA

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

@@ -9,7 +9,7 @@ reconcile/aws_iam_password_reset.py,sha256=NwErtrqgBiXr7eGCAHdtGGOx0S7-4JnSc29Ie
 reconcile/aws_support_cases_sos.py,sha256=Jk6_XjDeJSYxgRGqcEAOcynt9qJF2r5HPIPcSKmoBv8,2974
 reconcile/blackbox_exporter_endpoint_monitoring.py,sha256=W_VJagnsJR1v5oqjlI3RJJE0_nhtJ0m81RS8zWA5u5c,3538
 reconcile/checkpoint.py,sha256=R2WFXUXLTB4sWMi4GeA4eegsuf_1-Q4vH8M0Toh3Ij4,5036
-reconcile/cli.py,sha256=mtVX-Ws_9Bw3xgmjZuIQ9P4ZQZkaZszaJsxw1qa3xXY,84464
+reconcile/cli.py,sha256=ek511mirANFglUXz_lrnIjmHZ48ZDJOx-53OFf-fSO4,84491
 reconcile/closedbox_endpoint_monitoring_base.py,sha256=SMhkcQqprWvThrIJa3U_3uh5w1h-alleW1QnCJFY4Qw,4909
 reconcile/cluster_deployment_mapper.py,sha256=2Ah-nu-Mdig0pjuiZl_XLrmVAjYzFjORR3dMlCgkmw0,2352
 reconcile/dashdotdb_base.py,sha256=a5aPLVxyqPSbjdB0Ty-uliOtxwvEbbEljHJKxdK3-Zk,4813
@@ -96,6 +96,7 @@ reconcile/queries.py,sha256=ZGV4CZzzYz5whcoHk8X9gwfgJ9GnHxz6K07NkGpoBBo,50361
 reconcile/query_validator.py,sha256=BAjGrU8_VhzTOv5k0-uz0hY9ziZyconv8VAhgre1Auc,1497
 reconcile/requests_sender.py,sha256=914iluuF4UVgG3VyxxtnHOu4yf6YKS2fIy6PViSsFTQ,3875
 reconcile/resource_scraper.py,sha256=vo1N9vLJCYWvXlTwFRIpEuWjx_39ZV9zxJlpoPq4g3U,2330
+reconcile/resource_template_tester.py,sha256=DsKvBuNLPxm4Fa-e1YHHySnhThm5i_j-nF3G4b02Mz0,2416
 reconcile/saas_file_validator.py,sha256=WdRVwAdEWO3uH3atefZhZRiQEZje7SQBfVZI6L6c7cA,1955
 reconcile/sendgrid_teammates.py,sha256=oO8QbLb4s1o8A6CGiCagN9CmS05BSS_WLztuY0Ym9D8,4773
 reconcile/service_dependencies.py,sha256=PMKP9vc6oL-78rzyF_RE8DzLSQMSqN8vCqt9sWpBLAM,4470
@@ -105,7 +106,6 @@ reconcile/slack_usergroups.py,sha256=4Xikz_zKPHE5Q_hf2vOOs9vlItxgCqZPn7YJgn1Brik
 reconcile/sql_query.py,sha256=qLTUU2Hc0BmJoSl9cCUsIfdEaEaOQp7y__E4KLjjcaA,25732
 reconcile/status.py,sha256=cY4IJFXemhxptRJqR4qaaOWqei9e4jgLXuVSGajMsjg,544
 reconcile/status_board.py,sha256=xOKicOelyGv3h2sFJxa9FBj8HDroZyGhMaRWaNRGpQQ,8100
-reconcile/template_tester.py,sha256=vZz8GM46waQUGd3OVnhW5OLTqctFMH_Hh1QXxT5hduM,2384
 reconcile/terraform_aws_route53.py,sha256=R8eZHlvP368nvtmLd_cMSK3RGxD7jso9qE7NP92iniU,9986
 reconcile/terraform_cloudflare_dns.py,sha256=auU4bzeLwd4S8D8oqpqJbrCUoEdELXrgi7vHOedjYFk,13332
 reconcile/terraform_cloudflare_resources.py,sha256=EbQQaoDnZ7brvRCpbFtwlD7KLk2hDVNcjhrJGaAywEk,15023
@@ -345,14 +345,16 @@ reconcile/rhidp/sso_client/base.py,sha256=EfQ2ewcOKh5idg46UKAkY6z0m_nGQfvnQKffa2
 reconcile/rhidp/sso_client/integration.py,sha256=kA8g7c38ZBSdrRtyfEqy_WgSreD1PbwY7ZIN-3tZRPc,2221
 reconcile/rhidp/sso_client/metrics.py,sha256=Tq7tSOsqL3XdcPUdozxqzSPIodUeOV87UCTqpuuqqhw,1013
 reconcile/saas_auto_promotions_manager/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-reconcile/saas_auto_promotions_manager/integration.py,sha256=5hSvC3JUM4r63UFuC1IbeG6YbmdlzQrekwRqF5ggvCY,5667
+reconcile/saas_auto_promotions_manager/integration.py,sha256=1nm3aV5a0frEPTSww4ue1mfQ_Px6XaRZoaBRhPgJiAU,6472
 reconcile/saas_auto_promotions_manager/publisher.py,sha256=4_M9Oykhj-kEZPUn05E2DY5gD6-x32Dgf7K3NPOkGEg,2029
 reconcile/saas_auto_promotions_manager/subscriber.py,sha256=cLhPlkT71J2LIice3SLmH1WpsqzV46gd0peMxrnqyRw,7452
 reconcile/saas_auto_promotions_manager/merge_request_manager/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 reconcile/saas_auto_promotions_manager/merge_request_manager/merge_request.py,sha256=PNu7sE-tDUY61E03z5w0b93fdowZ8auCl0S4_vhYOKQ,1333
-reconcile/saas_auto_promotions_manager/merge_request_manager/merge_request_manager.py,sha256=CpTbL2wX4s6iQVZJzu-i8kTJOezCwPxyJTMoNdltqgc,7525
-reconcile/saas_auto_promotions_manager/merge_request_manager/mr_parser.py,sha256=gyEn7yp03XDf0P8QEDfXWTcGGehNQ2Ln_sQ3cUq00Bc,6945
-reconcile/saas_auto_promotions_manager/merge_request_manager/renderer.py,sha256=Sj3mwmBcml6t-49bPnXdq7snT9N0WKrPk2chU_0ylyE,7188
+reconcile/saas_auto_promotions_manager/merge_request_manager/merge_request_manager.py,sha256=P4RH6y-qC9RKJxVguyOfOAe42a6AP1Ez-NlyMSH3j8I,7492
+reconcile/saas_auto_promotions_manager/merge_request_manager/merge_request_manager_v2.py,sha256=AwCyvnNAKwRbESodHN2-fWG8Wpxua0nzQkcIsz4Gevg,5411
+reconcile/saas_auto_promotions_manager/merge_request_manager/mr_parser.py,sha256=x8Gg-YjEFWEeDPJH3Y8SrfcJbwhLuAqCz4kIhfEyaaA,7060
+reconcile/saas_auto_promotions_manager/merge_request_manager/reconciler.py,sha256=DgMw24exjuor9s4Y8y0Ftsjs5l7rl8D7dksMoJ1RF0A,7639
+reconcile/saas_auto_promotions_manager/merge_request_manager/renderer.py,sha256=VSKCwY8jZ6NYwIT6edmVnpzY7cgjhn5fhX9xtlw2LD8,7341
 reconcile/saas_auto_promotions_manager/utils/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 reconcile/saas_auto_promotions_manager/utils/saas_files_inventory.py,sha256=mihuWynroB1Cea1Lsvf6V8Nb8PGiBdcLC0uhCX_52Y0,6966
 reconcile/skupper_network/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
@@ -452,14 +454,14 @@ reconcile/test/test_vpc_peerings_validator.py,sha256=dFSmjc_dMN2GqMbntCFpa7PUZmy
 reconcile/test/test_wrong_region.py,sha256=7KzL7OaICQ9Z3DW27zt_ykMN7_87owAFC-2CYjvGoyA,2138
 reconcile/test/saas_auto_promotions_manager/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 reconcile/test/saas_auto_promotions_manager/conftest.py,sha256=tZNs35EuWulP53Cqt61RteOGY_uH4gfhtXfGQ8-awwM,3081
-reconcile/test/saas_auto_promotions_manager/test_integration_test.py,sha256=4LPWN7YWr2t_MRpwGf7pQ4AMSBEMYvTqTe5hY1rJyS0,4078
+reconcile/test/saas_auto_promotions_manager/test_integration_test.py,sha256=UIafVpeof9Zd3Q17mEDIUwgdBGf8oFxkf7h8tLfmwCE,2077
 reconcile/test/saas_auto_promotions_manager/merge_request_manager/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 reconcile/test/saas_auto_promotions_manager/merge_request_manager/merge_request_manager/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-reconcile/test/saas_auto_promotions_manager/merge_request_manager/merge_request_manager/conftest.py,sha256=Ky5IAi0EzrHqP0eimV4Mq0FMVdrFEN-7P-Nst4wqB34,4276
-reconcile/test/saas_auto_promotions_manager/merge_request_manager/merge_request_manager/data_keys.py,sha256=ZTGXlpgAECZnNqsJbkYGr6fBreC_KejYz4tCJIg8iDM,573
-reconcile/test/saas_auto_promotions_manager/merge_request_manager/merge_request_manager/test_merge_request_manager.py,sha256=c0-c9wfd-3mOMFsCJUDOCx-EfoiT75hJypz72xwdMb8,5978
-reconcile/test/saas_auto_promotions_manager/merge_request_manager/merge_request_manager/test_mr_parser.py,sha256=9VLJKOW80NMQ3WCKDqtaOa0-StoFxaCzKN-NCPfeklo,9845
-reconcile/test/saas_auto_promotions_manager/merge_request_manager/merge_request_manager/test_unbatching.py,sha256=IqmVqWw98Gegwz5johzh13cqCONk7D_VbgK1y8VqRP4,3004
+reconcile/test/saas_auto_promotions_manager/merge_request_manager/merge_request_manager/conftest.py,sha256=7O3lbk1EmEtUofqGncfiwMYvDPXrkQNPB59zlQ_zXkM,4588
+reconcile/test/saas_auto_promotions_manager/merge_request_manager/merge_request_manager/data_keys.py,sha256=Z1IV51OUuzhd-3S8W-k7ixC-fkaglCokn0eakK0Z73s,606
+reconcile/test/saas_auto_promotions_manager/merge_request_manager/merge_request_manager/test_merge_request_manager.py,sha256=ryROiiQNtZvuG820CB-RQ9FMmLAGshezyMDNDiJNA_E,2369
+reconcile/test/saas_auto_promotions_manager/merge_request_manager/merge_request_manager/test_mr_parser.py,sha256=dcGHzxuafKSxmswSO1qF2WlKaqsmEvtERC6Lb8kDAN0,10019
+reconcile/test/saas_auto_promotions_manager/merge_request_manager/merge_request_manager/test_reconciler.py,sha256=-rSv-C-JFaY0gsjSFgcXz9ASNQQOzwizaQEbVYJtLHg,14906
 reconcile/test/saas_auto_promotions_manager/merge_request_manager/renderer/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 reconcile/test/saas_auto_promotions_manager/merge_request_manager/renderer/conftest.py,sha256=2rCSstewp4LPoEJHm5N7dGJexEtY8ndLHvoGZYjmpsc,1678
 reconcile/test/saas_auto_promotions_manager/merge_request_manager/renderer/data_keys.py,sha256=beHYQ9kgDLeBZgC2FvxQA3tHx1PO-RAMN8_kVcSdikI,90
@@ -666,8 +668,8 @@ tools/test/test_app_interface_metrics_exporter.py,sha256=SX7qL3D1SIRKFo95FoQztvf
 tools/test/test_qontract_cli.py,sha256=d18KrdhtUGqoC7_kWZU128U0-VJEj-0rjFkLVufcI6I,2755
 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.1rc513.dist-info/METADATA,sha256=TTUNsBsaH5Ftp3hD6tDYHiEwEfjMcxwz7H5SShiJHM8,2349
-qontract_reconcile-0.10.1rc513.dist-info/WHEEL,sha256=oiQVh_5PnQM0E3gPdiz09WCNmwiHDMaGer_elqB3coM,92
-qontract_reconcile-0.10.1rc513.dist-info/entry_points.txt,sha256=rTjAv28I_CHLM8ID3OPqMI_suoQ9s7tFbim4aYjn9kk,376
-qontract_reconcile-0.10.1rc513.dist-info/top_level.txt,sha256=l5ISPoXzt0SdR4jVdkfa7RPSKNc8zAHYWAnR-Dw8Ey8,24
-qontract_reconcile-0.10.1rc513.dist-info/RECORD,,
+qontract_reconcile-0.10.1rc515.dist-info/METADATA,sha256=Bgwt_oRZlcfCQ6ycYlopR_xt-fRVu3kWrlOW7wMuaqo,2349
+qontract_reconcile-0.10.1rc515.dist-info/WHEEL,sha256=oiQVh_5PnQM0E3gPdiz09WCNmwiHDMaGer_elqB3coM,92
+qontract_reconcile-0.10.1rc515.dist-info/entry_points.txt,sha256=rTjAv28I_CHLM8ID3OPqMI_suoQ9s7tFbim4aYjn9kk,376
+qontract_reconcile-0.10.1rc515.dist-info/top_level.txt,sha256=l5ISPoXzt0SdR4jVdkfa7RPSKNc8zAHYWAnR-Dw8Ey8,24
+qontract_reconcile-0.10.1rc515.dist-info/RECORD,,

reconcile/cli.py

@@ -2628,10 +2628,10 @@ def prometheus_rules_tester(ctx, thread_pool_size, cluster_name):
 
 @integration.command(short_help="Tests templating of resources.")
 @click.pass_context
-def template_tester(ctx):
-    import reconcile.template_tester
+def resource_template_tester(ctx):
+    import reconcile.resource_template_tester
 
-    run_integration(reconcile.template_tester, ctx.obj)
+    run_integration(reconcile.resource_template_tester, ctx.obj)
 
 
 @integration.command(

reconcile/{template_tester.py → resource_template_tester.py}

@@ -11,13 +11,13 @@ from reconcile.status import ExitCodes
 from reconcile.utils import gql
 from reconcile.utils.semver_helper import make_semver
 
-QONTRACT_INTEGRATION = "template-tester"
+QONTRACT_INTEGRATION = "resource-template-tester"
 QONTRACT_INTEGRATION_VERSION = make_semver(0, 1, 0)
 
 
 TEMPLATE_TESTS_QUERY = """
 {
-  tests: template_tests_v1 {
+  tests: resource_template_tests_v1 {
     name
     resourcePath
     expectedResult
@@ -30,7 +30,7 @@ def load_resource(path: str) -> dict:
     return yaml.safe_load(gql.get_resource(path)["content"])
 
 
-def run(dry_run):
+def run(dry_run: bool) -> None:
     gqlapi = gql.get_api()
     template_tests = gqlapi.query(TEMPLATE_TESTS_QUERY)["tests"]
     settings = queries.get_app_interface_settings()

reconcile/saas_auto_promotions_manager/integration.py

@@ -9,9 +9,15 @@ from reconcile.openshift_saas_deploy import (
 from reconcile.saas_auto_promotions_manager.merge_request_manager.merge_request_manager import (
     MergeRequestManager,
 )
+from reconcile.saas_auto_promotions_manager.merge_request_manager.merge_request_manager_v2 import (
+    MergeRequestManagerV2,
+)
 from reconcile.saas_auto_promotions_manager.merge_request_manager.mr_parser import (
     MRParser,
 )
+from reconcile.saas_auto_promotions_manager.merge_request_manager.reconciler import (
+    Reconciler,
+)
 from reconcile.saas_auto_promotions_manager.merge_request_manager.renderer import (
     Renderer,
 )
@@ -45,6 +51,7 @@ class SaasAutoPromotionsManager:
         deployment_state: PromotionState,
         vcs: VCS,
         saas_file_inventory: SaasFilesInventory,
+        merge_request_manager_v2: MergeRequestManagerV2,
         merge_request_manager: MergeRequestManager,
         thread_pool_size: int,
         dry_run: bool,
@@ -52,6 +59,7 @@ class SaasAutoPromotionsManager:
         self._deployment_state = deployment_state
         self._vcs = vcs
         self._saas_file_inventory = saas_file_inventory
+        self._merge_request_manager_v2 = merge_request_manager_v2
         self._merge_request_manager = merge_request_manager
         self._thread_pool_size = thread_pool_size
         self._dry_run = dry_run
@@ -88,11 +96,14 @@ class SaasAutoPromotionsManager:
         self._merge_request_manager.create_promotion_merge_requests(
             subscribers=subscribers_with_diff
         )
+        self._merge_request_manager_v2.reconcile(subscribers=subscribers_with_diff)
 
 
 def init_external_dependencies(
     dry_run: bool,
-) -> tuple[PromotionState, VCS, SaasFilesInventory, MergeRequestManager]:
+) -> tuple[
+    PromotionState, VCS, SaasFilesInventory, MergeRequestManagerV2, MergeRequestManager
+]:
     """
     Lets initialize everything that involves calls to external dependencies:
     - VCS -> Gitlab / Github queries
@@ -120,17 +131,29 @@ def init_external_dependencies(
         allow_opening_mrs=allow_opening_mrs,
     )
     mr_parser = MRParser(vcs=vcs)
-    merge_request_manager = MergeRequestManager(
+    merge_request_manager_v2 = MergeRequestManagerV2(
         vcs=vcs,
+        reconciler=Reconciler(),
         mr_parser=mr_parser,
         renderer=Renderer(),
     )
+    merge_request_manager = MergeRequestManager(
+        mr_parser=mr_parser,
+        renderer=Renderer(),
+        vcs=vcs,
+    )
     saas_files = get_saas_files()
     saas_inventory = SaasFilesInventory(saas_files=saas_files)
     deployment_state = PromotionState(
         state=init_state(integration=OPENSHIFT_SAAS_DEPLOY, secret_reader=secret_reader)
     )
-    return deployment_state, vcs, saas_inventory, merge_request_manager
+    return (
+        deployment_state,
+        vcs,
+        saas_inventory,
+        merge_request_manager_v2,
+        merge_request_manager,
+    )
 
 
 @defer
@@ -143,6 +166,7 @@ def run(
         deployment_state,
         vcs,
         saas_inventory,
+        merge_request_manager_v2,
         merge_request_manager,
     ) = init_external_dependencies(dry_run=dry_run)
     if defer:
@@ -153,6 +177,7 @@ def run(
         vcs=vcs,
         saas_file_inventory=saas_inventory,
         merge_request_manager=merge_request_manager,
+        merge_request_manager_v2=merge_request_manager_v2,
         thread_pool_size=thread_pool_size,
         dry_run=dry_run,
     )

reconcile/saas_auto_promotions_manager/merge_request_manager/merge_request_manager.py

@@ -16,7 +16,6 @@ from reconcile.saas_auto_promotions_manager.merge_request_manager.renderer impor
     CHANNELS_REF,
     CONTENT_HASHES,
     IS_BATCHABLE,
-    SAPM_LABEL,
     VERSION_REF,
     Renderer,
 )
@@ -25,6 +24,8 @@ from reconcile.utils.vcs import VCS
 
 ITEM_SEPARATOR = ","
 
+SAPM_LABEL = "SAPM"
+
 
 class MergeRequestManager:
     """
@@ -68,14 +69,13 @@ class MergeRequestManager:
                     "Closing this MR because it failed MR check and isn't marked un-batchable yet.",
                 )
                 # Remember these hashes as unbatchable
-                content_hashes = mr.content_hashes.split(ITEM_SEPARATOR)
-                self._unbatchable_hashes.update(content_hashes)
+                self._unbatchable_hashes.update(mr.content_hashes)
             else:
                 open_mrs_after_unbatching.append(mr)
         self._open_mrs = open_mrs_after_unbatching
 
     def housekeeping(self) -> None:
-        self._open_mrs = self._mr_parser.retrieve_open_mrs()
+        self._open_mrs = self._mr_parser.retrieve_open_mrs(label=SAPM_LABEL)
         self._unbatch_failed_mrs()
 
     def _aggregate_subscribers_per_channel_combo(
@@ -160,7 +160,7 @@ class MergeRequestManager:
                 channels=channel_combo,
                 is_batchable=combined_content_hash not in self._unbatchable_hashes,
             )
-            title = self._renderer.render_title(channels=channel_combo)
+            title = self._renderer.render_title(is_draft=False, channels=channel_combo)
             logging.info(
                 "Open MR for update in channel(s) %s",
                 channel_combo,

reconcile/saas_auto_promotions_manager/merge_request_manager/merge_request_manager_v2.py

@@ -0,0 +1,144 @@
+import logging
+from collections import defaultdict
+from collections.abc import Iterable
+
+from gitlab.exceptions import GitlabGetError
+
+from reconcile.saas_auto_promotions_manager.merge_request_manager.merge_request import (
+    SAPMMR,
+)
+from reconcile.saas_auto_promotions_manager.merge_request_manager.mr_parser import (
+    MRParser,
+)
+from reconcile.saas_auto_promotions_manager.merge_request_manager.reconciler import (
+    Addition,
+    Promotion,
+    Reconciler,
+)
+from reconcile.saas_auto_promotions_manager.merge_request_manager.renderer import (
+    Renderer,
+)
+from reconcile.saas_auto_promotions_manager.subscriber import Subscriber
+from reconcile.utils.vcs import VCS
+
+BATCH_SIZE_LIMIT = 5
+SAPM_LABEL = "SAPMCanary"
+
+
+class MergeRequestManagerV2:
+    """
+    Manager for SAPM merge requests.
+
+    This class uses MRParser to fetch current state (currently open MRs).
+    This class calculates the desired state (i.e., desired promotions).
+    Desired state and current state are given to the Reconciler, which will
+    then determine a Diff (Additions and Deletions of MRs).
+
+    This class interacts with VCS to realize the result of the Diff.
+    """
+
+    def __init__(
+        self, vcs: VCS, mr_parser: MRParser, reconciler: Reconciler, renderer: Renderer
+    ):
+        self._vcs = vcs
+        self._mr_parser = mr_parser
+        self._renderer = renderer
+        self._reconciler = reconciler
+        self._content_hash_to_subscriber: dict[str, list[Subscriber]] = {}
+        self._sapm_mrs: list[SAPMMR] = []
+
+    def _aggregate_desired_state(
+        self, subscribers: Iterable[Subscriber]
+    ) -> list[Promotion]:
+        subscribers_per_channel_combo: dict[str, list[Subscriber]] = defaultdict(list)
+        for subscriber in subscribers:
+            channel_combo = ",".join([c.name for c in subscriber.channels])
+            subscribers_per_channel_combo[channel_combo].append(subscriber)
+
+        desired_promotions: list[Promotion] = []
+        for channel_combo, subs in subscribers_per_channel_combo.items():
+            combined_content_hash = Subscriber.combined_content_hash(subscribers=subs)
+            self._content_hash_to_subscriber[combined_content_hash] = subs
+            desired_promotions.append(
+                Promotion(
+                    content_hashes={combined_content_hash},
+                    channels={channel_combo},
+                )
+            )
+        return desired_promotions
+
+    def _render_mr(self, addition: Addition) -> None:
+        subs: list[Subscriber] = []
+        for content_hash in addition.content_hashes:
+            subs.extend(self._content_hash_to_subscriber[content_hash])
+        content_by_path: dict[str, str] = {}
+        has_error = False
+        for sub in subs:
+            if sub.target_file_path not in content_by_path:
+                try:
+                    content_by_path[sub.target_file_path] = (
+                        self._vcs.get_file_content_from_app_interface_master(
+                            file_path=sub.target_file_path
+                        )
+                    )
+                except GitlabGetError as e:
+                    if e.response_code == 404:
+                        logging.error(
+                            "The saas file %s does not exist anylonger. Most likely qontract-server data not in synch. This should resolve soon on its own.",
+                            sub.target_file_path,
+                        )
+                        has_error = True
+                        break
+                    raise e
+            content_by_path[sub.target_file_path] = (
+                self._renderer.render_merge_request_content(
+                    subscriber=sub,
+                    current_content=content_by_path[sub.target_file_path],
+                )
+            )
+        if has_error:
+            return
+
+        description_hashes = ",".join(addition.content_hashes)
+        description_channels = ",".join(addition.channels)
+
+        description = self._renderer.render_description(
+            content_hashes=description_hashes,
+            channels=description_channels,
+            is_batchable=addition.batchable,
+        )
+        title = self._renderer.render_title(
+            is_draft=True, channels=description_channels
+        )
+        logging.info(
+            "Open MR for update in channel(s) %s",
+            description_channels,
+        )
+        self._sapm_mrs.append(
+            SAPMMR(
+                sapm_label=SAPM_LABEL,
+                content_by_path=content_by_path,
+                title=title,
+                description=description,
+            )
+        )
+
+    def reconcile(self, subscribers: Iterable[Subscriber]) -> None:
+        current_state = self._mr_parser.retrieve_open_mrs(label=SAPM_LABEL)
+        desired_state = self._aggregate_desired_state(subscribers=subscribers)
+        diff = self._reconciler.reconcile(
+            batch_limit=BATCH_SIZE_LIMIT,
+            desired_promotions=desired_state,
+            open_mrs=current_state,
+        )
+        for deletion in diff.deletions:
+            self._vcs.close_app_interface_mr(
+                mr=deletion.mr.raw,
+                comment=deletion.reason,
+            )
+
+        for addition in diff.additions:
+            self._render_mr(addition=addition)
+
+        for rendered_mr in self._sapm_mrs:
+            self._vcs.open_app_interface_merge_request(mr=rendered_mr)

reconcile/saas_auto_promotions_manager/merge_request_manager/mr_parser.py

@@ -9,7 +9,6 @@ from reconcile.saas_auto_promotions_manager.merge_request_manager.renderer impor
     CONTENT_HASHES,
     IS_BATCHABLE,
     PROMOTION_DATA_SEPARATOR,
-    SAPM_LABEL,
     SAPM_VERSION,
     VERSION_REF,
 )
@@ -19,12 +18,15 @@ from reconcile.utils.vcs import VCS, MRCheckStatus
 @dataclass
 class OpenMergeRequest:
     raw: ProjectMergeRequest
-    content_hashes: str
-    channels: str
+    content_hashes: set[str]
+    channels: set[str]
     failed_mr_check: bool
     is_batchable: bool
 
 
+ITEM_SEPARATOR = ","
+
+
 class MRParser:
     """
     We store state in MR descriptions.
@@ -49,11 +51,13 @@ class MRParser:
             return ""
         return groups[0]
 
-    def _fetch_sapm_managed_open_merge_requests(self) -> list[ProjectMergeRequest]:
+    def _fetch_sapm_managed_open_merge_requests(
+        self, label: str
+    ) -> list[ProjectMergeRequest]:
         all_open_mrs = self._vcs.get_open_app_interface_merge_requests()
-        return [mr for mr in all_open_mrs if SAPM_LABEL in mr.attributes.get("labels")]
+        return [mr for mr in all_open_mrs if label in mr.attributes.get("labels")]
 
-    def retrieve_open_mrs(self) -> list[OpenMergeRequest]:
+    def retrieve_open_mrs(self, label: str) -> list[OpenMergeRequest]:
         """
         This function parses the state and returns a list of valid, parsed open MRs (current state).
         If any issue is encountered during parsing, we consider this MR
@@ -66,7 +70,7 @@ class MRParser:
         """
         open_mrs: list[OpenMergeRequest] = []
         seen: set[tuple[str, str, str]] = set()
-        for mr in self._fetch_sapm_managed_open_merge_requests():
+        for mr in self._fetch_sapm_managed_open_merge_requests(label=label):
             attrs = mr.attributes
             desc = attrs.get("description")
             has_conflicts = attrs.get("has_conflicts", False)
@@ -177,8 +181,8 @@ class MRParser:
             open_mrs.append(
                 OpenMergeRequest(
                     raw=mr,
-                    content_hashes=content_hashes,
-                    channels=channels_refs,
+                    content_hashes=set(content_hashes.split(ITEM_SEPARATOR)),
+                    channels=set(channels_refs.split(ITEM_SEPARATOR)),
                     failed_mr_check=mr_check_status == MRCheckStatus.FAILED,
                     is_batchable=is_batchable_str == "True",
                 )