Skip to content

Conversation

maxcorbeau
Copy link
Contributor

@maxcorbeau maxcorbeau commented Apr 28, 2025

🚪 Acteurs fermés: pas créer de parent

🗺️ contexte: on a changé d'avis sur la façon de gérer les acteurs à fermer, voir discussion

💡 quoi: modification du DAG de suggestion des acteurs fermés

🎯 pourquoi:

  • 🚫 pas de nouveau parent créé = parent pas strictement nécessaire car aurait juste 1 nouvel enfant => 1 nouveau parent
    • 🟢 avantage: on ne va pas abimer les clusters existant, cf ci-dessous
  • 🆔 identifiant_externe = mettre le même que l'enfant d'origine (une technicité pour @nicoo )
  • 🧑‍🤝‍🧑 parent_id = mettre le même que l'enfant d'origine
    • 🟢 avantage: préservation du clustering

🤔 comment: subtilités liés aux diff de source:

  • changer l'odre de création: d'abord l'ancien parent à désactiver, après créer le nouveau parent, pour éviter la contrainte seulement 1 acteur ACTIF par identifiant_externe
  • 2 nouveaux modèles pydantic (les 2 sont testés) :
    • ChangeActeurUpdateStatut -> inactive un acteur et sa révision si elle existe
    • ChangeActeurCreateAsCopy -> Crée une copy d'un acteur inactif, permet de créer une nouvelle instance ACTIVE avec le nouveau SIREN/SIRET
  • un peu de doc pour illustrer dans quel ordre les DAG sont a exécuter

@maxcorbeau maxcorbeau added enhancement New feature or request airflow labels Apr 28, 2025
@kolok kolok force-pushed the ae_replaced_dont_create_parent branch from a7232d3 to cbc1e64 Compare May 7, 2025 11:45
@kolok kolok marked this pull request as ready for review May 12, 2025 16:38
@kolok kolok requested a review from a team as a code owner May 12, 2025 16:38
@kolok kolok requested review from kolok and fabienheureux and removed request for a team May 12, 2025 16:38
@kolok kolok removed their assignment May 12, 2025
@kolok kolok enabled auto-merge (squash) May 15, 2025 09:14
@kolok kolok merged commit 6874144 into main May 15, 2025
8 of 11 checks passed
@kolok kolok deleted the ae_replaced_dont_create_parent branch May 15, 2025 11:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
airflow enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants