Skip to content

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

Closed
Feature
cloud-pi-native/documentation
#127
Closed
💡 [REQUEST] - Porter le pont zone GitLab vers server-nestjs#2864
Feature
cloud-pi-native/documentation
#127
Assignees
shikanime
Labels
enhancementNew feature or request
Milestone
9.28.0

Description

@shikanime
shikanime
opened on Oct 15, 2026

Description

À la bascule des routes zone vers server-nestjs (#2770), le provisioning GitLab des zones cesse silencieusement si le pont n'est pas porté : le legacy plugins/gitlab crée et supprime le repo infra de zone sur hook.zone.upsert / hook.zone.delete, et aucun consommateur zone.* GitLab n'existe côté server-nestjs.

PRs liées

Issues liées

Exemples simples

À la création d'une zone, GitlabService garantit l'existence du repo infra de la zone ; à la suppression, il le supprime définitivement (permanentlyRemove), parité avec plugins/gitlab/src/functions.ts:189.

Spécifications techniques

  • Consommateurs @OnEvent('zone.upsert') / @OnEvent('zone.delete') dans apps/server-nestjs/src/modules/gitlab/gitlab.service.ts
  • ensureInfraGroupRepo / deleteInfraGroupRepo dans gitlab-client.service.ts : suppression non créante (no-op si le repo est absent), 404 toléré sur l'étape permanentlyRemove pour l'idempotence
  • Factorisation unlessNotFound depuis les 7 sites catch-404 gitbeaker existants

Définition du fini

  • La fonctionnalité est terminée
  • Les tests liés à cette fonctionnalité ont été ajoutés
  • La documentation liée a été ajoutée

Activity

  1. shikanime
    added this to the 9.28.0 milestone on Oct 15, 2026
  2. shikanime
    self-assigned this
    on Oct 15, 2026
  3. shikanime
    added
    enhancementNew feature or request
    on Oct 15, 2026

  4. shikanime commented on Oct 15, 2026

    @shikanime
    shikanime
    on Oct 15, 2026
    MemberAuthor

    Ledger vérifié contre #2807 head 118a336f (CI verte, 32 checks pass).

    La fonctionnalité est terminée — GitlabService.handleUpsertZone / handleDeleteZone (@OnEvent('zone.upsert') / @OnEvent('zone.delete')) dans apps/server-nestjs/src/modules/gitlab/gitlab.service.ts ; ensureInfraGroupRepo / deleteInfraGroupRepo dans gitlab-client.service.ts avec suppression non créante (no-op si repo absent) et 404 toléré sur permanentlyRemove :

    +  @OnEvent('zone.upsert')
+  async handleUpsertZone(payload: ZoneEventPayload): Promise<RequiredPluginResult<'gitlab'>> {
+  @OnEvent('zone.delete')
+  async handleDeleteZone(payload: ZoneEventPayload): Promise<RequiredPluginResult<'gitlab'>> {

    Les 6 fils de review sont résolus, y compris les bloquants (chemin de suppression absolu deletion_scheduled, idempotence, parseGroupPaths restauré).

    Les tests liés à cette fonctionnalité ont été ajoutés — gitlab.service.spec.ts (commit 9b5f0b1b) : 4 specs handleUpsertZone / handleDeleteZone (succès + KO plugin result) ; job unit-tests / Unit tests vert sur le head courant.

    La documentation reste ouverte : décision post-merge (sks-doc).

  5. shikanime
    added a commit that references this issue on Oct 15, 2026

    feat(zone): port the gitlab zone bridge to server-nestjs

    d9c6b5b

  6. shikanime commented on Oct 16, 2026

    @shikanime
    shikanime
    on Oct 16, 2026
    MemberAuthor

    Critère « La documentation liée a été ajoutée » traité : PR cloud-pi-native/documentation#117 — la liste des dépôts system-managed de docs/services/gitlab.md documente désormais le dépôt infra de zone (provisionné à la création de la zone, supprimé avec elle, porteur des manifests Argo CD).

    Ledger 3/3 une fois cette PR mergée.

  7. shikanime
    added a commit that references this issue on Oct 16, 2026

    docs(server-nestjs): record the gitlab zone consumers in the cartogra…

    d8f52e9

  8. shikanime commented on Oct 16, 2026

    @shikanime
    shikanime
    on Oct 16, 2026
    MemberAuthor

    La documentation liée a été ajoutée — commit d8f52e99 sur #2807 met à jour la table des dépendances de bascule de la cartographie (MODULARISATION-CARTOGRAPHIE.md) :

    - | `zone.upsert`, `zone.delete` | Aucun émetteur migré | Vault | La zone reste legacy. |
+ | `zone.upsert`, `zone.delete` | Aucun émetteur migré | Vault, GitLab | Consommateurs présents ; la zone reste legacy. |

    Ledger 3/3.

  9. shikanime
    modified the milestones: 9.28.0, on Oct 19, 2026
  10. shikanime
    added the
    Feature
    issue type on Oct 20, 2026
  11. shikanime
    modified the milestones: , 9.28.0 on Oct 21, 2026
  12. shikanime
    linked a pull request that will close this issuedoc(gitlab): document the zone infra repository lifecycle #127on Oct 21, 2026

  13. shikanime commented on Oct 21, 2026

    @shikanime
    shikanime
    on Oct 21, 2026
    MemberAuthor

    Requalification : réalisé sur main. Le pont zone GitLab est porté dans apps/server-nestjs : consommateurs @OnEvent('zone.upsert') et @OnEvent('zone.delete') dans gitlab.service.ts (création/suppression du repo infra de zone), avec specs. Vérifié sur main.

  14. shikanime commented on Oct 21, 2026

    @shikanime
    shikanime
    on Oct 21, 2026
    MemberAuthor

    Terminé : consommateurs zone.upsert / zone.delete présents dans le module gitlab de server-nestjs (main).

  15. shikanime
    closed this as completedon Oct 21, 2026
  16. StephaneTrebel
    added 2 commits that reference this issue on Oct 23, 2026

    feat(zone): port the gitlab zone bridge to server-nestjs

    301f27e

    docs(server-nestjs): record the gitlab zone consumers in the cartogra…

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

Metadata

Metadata

Assignees

Labels

enhancementNew feature or request

Type

Feature

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions