From 0d86f39b15f50155e131276ce6c8f2f07841fb84 Mon Sep 17 00:00:00 2001 From: Victor Perron Date: Mon, 2 Sep 2024 09:17:27 +0200 Subject: [PATCH] chore(pipeline) : Add the int__union__contacts build step When introducing the new contacts from mediation_numerique and mes_aides, I did not verify that the "top-level" model was also built. --- pipeline/dags/dag_utils/dbt.py | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/pipeline/dags/dag_utils/dbt.py b/pipeline/dags/dag_utils/dbt.py index a202df6e9..a5b3a21c6 100644 --- a/pipeline/dags/dag_utils/dbt.py +++ b/pipeline/dags/dag_utils/dbt.py @@ -104,6 +104,14 @@ def get_before_geocoding_tasks(): command="build", select=" ".join( [ + # NOTE(vperron, 2024-09-02) : The contacts union is build here, + # out of coherence but I don't really like that now the "brevo" + # machinery depends on 3 DAGs to run correctly: import_brevo, + # main and notify_rgpd_contacts. Maybe it should be concentrated + # into a single DAG. Another way to see it is that it depended on + # main since the beginning as it required intermediate data to be + # present ? + "path:models/intermediate/int__union_contacts.sql", "path:models/intermediate/int__union_adresses.sql", "path:models/intermediate/int__union_services.sql", "path:models/intermediate/int__union_structures.sql",