Skip to content

doc(gitlab): document the zone infra repository lifecycle - #127

Open
shikanime wants to merge 2 commits into
mainfrom
doc/zone-infra-repo
Open

shikanime wants to merge 2 commits into
mainfrom
doc/zone-infra-repo

Conversation

@shikanime

Copy link
Copy Markdown
Member

Why

Console#2807 porte le pont zone GitLab vers server-nestjs : chaque zone reçoit un dépôt infra system-managed, supprimé avec la zone. La liste des dépôts gérés n'en parle pas — le critère documentation de Console#2864 reste décoché.

What

  • docs/services/gitlab.md, liste « Dépôts techniques » : ajoute le dépôt infra de zone avec son cycle de vie (provisionné à la création, supprimé avec la zone, manifests Argo CD).

References

  • Console#2864 (critère documentation)
  • Console#2807 (pont zone)

The zone bridge provisions a system-managed infra repository per zone; the managed-repositories list omitted it, so a reader could not tell zone repos apart from orphaned manual ones.

Refs #2864

Co-authored-by: Automata <automata@shikanime.studio>
Signed-off-by: William Phetsinorath <william.phetsinorath-open@interieur.gouv.fr>
Change-Id: I7666f0678250413ff26db570d57de5db6a6a6964

@StephaneTrebel StephaneTrebel left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Le cycle création/suppression de zone correspond bien au pont Console #2807 désormais fusionné, mais l’ajout confond le groupe infra avec le nom du dépôt et l’insère dans un périmètre de réconciliation projet qui ne le gère pas : correction demandée inline. Le Markdown, les liens locaux de la page et le contrôle CI Test app build sont valides ; aucun build applicatif local n’a été exécuté. Sans ledger direct dans ce dépôt ; le suivi Console #2864 existe et n’a pas été modifié par cette revue.

Comment thread docs/services/gitlab.md Outdated
@shikanime shikanime linked an issue Oct 21, 2026 that may be closed by this pull request
3 tasks done
@shikanime shikanime added the documentation Improvements or additions to documentation label Oct 26, 2026
@shikanime shikanime self-assigned this Oct 26, 2026

@StephaneTrebel StephaneTrebel left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Le constat important de ma précédente revue reste présent sur cette tête : le texte appelle « infra » un dépôt qui porte le slug de la zone et le place dans le périmètre des dépôts du projet ; je ne duplique pas ce fil toujours ouvert. J’ajoute une suggestion de formulation plus accessible pour cette explication ; liens locaux vérifiés et CI « Test app build » verte, sans build local, le suivi Console #2864 étant connu mais sans ledger direct dans cette MR.

Comment thread docs/services/gitlab.md Outdated
The zone infrastructure repository lives in the infra group as
<PROJECTS_ROOT_DIR>/infra/<zoneSlug>, named after the zone slug. It is
reconciled by zone.upsert and deleted by zone.delete, not by project
reprovisioning.

Co-authored-by: Automata <automata@shikanime.studio>
Signed-off-by: William Phetsinorath <william.phetsinorath@shikanime.studio>

@StephaneTrebel StephaneTrebel left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Les deux remarques précédentes sont corrigées sur cette tête : le groupe infra et le dépôt de zone sont distingués, et le contenu ainsi que le cycle de vie du dépôt sont expliqués plus directement. Aucun nouveau constat ; les quatre liens locaux de la page et la structure Markdown sont valides, avec « Test app build » vert (pas de build local ni de recette applicative exécutés ; suivi Console #2864 connu, sans ledger direct modifié).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

💡 [REQUEST] - Porter le pont zone GitLab vers server-nestjs

3 participants