nonobis/Heimdall
Watch
1
0
Fork
You've already forked Heimdall
 0
Code Issues 24 Pull requests Projects Releases Packages Activity Actions 1

fix(ci): action ci-full par URL Forgejo directe (@main) — suppression du checkout .ci manuel et de son cache #111

Closed
nonobis wants to merge 22 commits from fix/ci-url-forgejo-directe into main
pull from: fix/ci-url-forgejo-directe
merge into: nonobis:main
Conversation 3 Commits 22 Files changed 1 +6
nonobis commented 2026-08-12 23:08:25 +02:00
Owner
Copy link

Contexte

Bascule de l'action composite ci-full sur une référence URL Forgejo directe (@main) pour éviter le cache du checkout .ci manuel qui causait l'exécution d'anciennes versions du workflow (tests E2E non filtrés, inputs SonarQube non passés).

Changements

  • Suppression du checkout nonobis/templates vers .ci
  • uses: https://forgejo.tordais.cc/nonobis/templates/.forgejo/actions/ci-full@main

Tickets liés

Aucun ticket dédié (fix infra CI).

## Contexte Bascule de l'action composite `ci-full` sur une référence URL Forgejo directe (`@main`) pour éviter le cache du checkout `.ci` manuel qui causait l'exécution d'anciennes versions du workflow (tests E2E non filtrés, inputs SonarQube non passés). ## Changements - [x] Suppression du checkout `nonobis/templates` vers `.ci` - [x] `uses: https://forgejo.tordais.cc/nonobis/templates/.forgejo/actions/ci-full@main` ## Tickets liés Aucun ticket dédié (fix infra CI).
ci: rerun après passage templates en public
Some checks failed
CI / ci (pull_request) Failing after 10m56s
Details
535a9e43b7
nonobis self-assigned this 2026-08-12 23:14:45 +02:00
fix(ci): actions templates @v1 (cache runner par URL non invalidé)
Some checks failed
CI / ci (pull_request) Failing after 18m3s
Details
0f30a3ba31
chore(ci): bascule ci-full@v2 (purge caches applicatifs)
Some checks failed
CI / ci (pull_request) Failing after 25m5s
Details
6149c05a32
ci: relance pour validation (cache runner)
Some checks failed
CI / ci (pull_request) Failing after 6m56s
Details
8007b06559
ci: relance après purge du cache runner
Some checks failed
CI / ci (pull_request) Failing after 7m46s
Details
18cf34c85c
ci: relance après purge + restart runners
Some checks failed
CI / ci (pull_request) Failing after 5m45s
Details
8540627e0e
ci: bascule ci-full@v2 → @v3 (purge env complète en début de build)
Some checks failed
CI / ci (pull_request) Failing after 44m18s
Details
2cf9dd0ba5
ci: re-trigger pour tester la purge ~/.cache/act (templates b66383d)
Some checks failed
CI / ci (pull_request) Failing after 6m43s
Details
f40007f06e
ci: re-trigger 2 — vérifier la purge ~/.cache/act (templates 2640a13)
Some checks failed
CI / ci (pull_request) Failing after 5m23s
Details
2b0f822f51
ci: re-trigger post-purge hôte 6h (vérif cache actions)
Some checks failed
CI / ci (pull_request) Failing after 2s
Details
1b00eda4cf
ci: re-trigger post --force-full (purge totale caches + vérif)
Some checks failed
CI / ci (pull_request) Failing after 10s
Details
1087ad8bfb
ci: re-trigger post --force-full v2 (clone frais checkout@v4)
Some checks failed
CI / ci (pull_request) Failing after 39s
Details
5927bd1522
ci: re-trigger post runner-cleanup corrigé (stop→purge→start, section 6)
Some checks failed
CI / ci (pull_request) Failing after 10m8s
Details
16e152dbfd
ci: re-trigger post réparation runners (reboot LXC 2026-08-14)
Some checks failed
CI / ci (pull_request) Failing after 39m35s
Details
42c337d210
nonobis commented 2026-08-14 15:09:15 +02:00
Author
Owner
Copy link

🔍 Revue the-reviewer — Passe 1 (2026-08-14)

Statut : ❌ PR à corriger / probablement à fermer (titre ≠ contenu, pas de ticket, CI rouge)

Code — ⚠️ titre ne correspond pas au contenu

  • Le titre annonce « action ci-full par URL Forgejo directe (@main) — suppression du checkout .ci manuel ». Or le diff actuel (22 commits de re-trigger) fait l'inverse : retour au checkout .ci local (PAT) + action locale ./.ci/.forgejo/actions/ci-full (commentaire explicite ligne 2 : « Retour au système pré-bascule »).
  • L'historique de 22 commits « ci: re-trigger » est du bruit de debug : la branche doit être nettoyée (rebase interactif ou nouvelle branche propre).

CI — ❌ bloquant

  • Run #1929 : échec E2E (Chromium not found) + Infrastructure.Tests (PostgreSQL KO) + ci: command not found → cache runner corrompu (infra, pas cette PR en soi).

Conformité — ❌ bloquant

  • Aucun ticket lié (violation du process : PR par ticket ou groupe de tickets même thème — fix infra = ticket TECH).
  • Template PR non respecté (manquent les sections Tests / SonarQube / CVE).
  • Branche 2 commits derrière main.
  • Référence @main dans le titre : le process exige un tag versionné ci-full@vN (voir forgejo-pr-unified) — le contenu ne l'applique de toute façon pas.

Sécurité — ✅

  • RAS (fichier workflow uniquement).

Actions requises

  1. the-po : créer le ticket TECH pour ce fix infra CI.
  2. the-developer : reconstruire la branche sur le vrai objectif (URL Forgejo directe + tag @vN, suppression du checkout .ci), ou fermer la PR si l'approche est abandonnée.
  3. the-devops : réparer le cache runner (purge /opt/forgejo-runner/work/).
## 🔍 Revue the-reviewer — Passe 1 (2026-08-14) **Statut : ❌ PR à corriger / probablement à fermer** (titre ≠ contenu, pas de ticket, CI rouge) ### Code — ⚠️ titre ne correspond pas au contenu - Le titre annonce « action ci-full par URL Forgejo directe (@main) — suppression du checkout .ci manuel ». Or le diff actuel (22 commits de re-trigger) fait l'**inverse** : retour au checkout `.ci` local (PAT) + action locale `./.ci/.forgejo/actions/ci-full` (commentaire explicite ligne 2 : « Retour au système pré-bascule »). - L'historique de 22 commits « ci: re-trigger » est du bruit de debug : la branche doit être nettoyée (rebase interactif ou nouvelle branche propre). ### CI — ❌ bloquant - Run #1929 : échec E2E (`Chromium not found`) + Infrastructure.Tests (PostgreSQL KO) + `ci: command not found` → **cache runner corrompu** (infra, pas cette PR en soi). ### Conformité — ❌ bloquant - **Aucun ticket lié** (violation du process : PR par ticket ou groupe de tickets même thème — fix infra = ticket TECH). - Template PR non respecté (manquent les sections Tests / SonarQube / CVE). - Branche **2 commits derrière main**. - Référence `@main` dans le titre : le process exige un **tag versionné** `ci-full@vN` (voir forgejo-pr-unified) — le contenu ne l'applique de toute façon pas. ### Sécurité — ✅ - RAS (fichier workflow uniquement). ### Actions requises 1. **the-po** : créer le ticket TECH pour ce fix infra CI. 2. **the-developer** : reconstruire la branche sur le vrai objectif (URL Forgejo directe + tag `@vN`, suppression du checkout `.ci`), ou **fermer la PR** si l'approche est abandonnée. 3. **the-devops** : réparer le cache runner (purge `/opt/forgejo-runner/work/`).
nonobis commented 2026-08-14 15:30:31 +02:00
Author
Owner
Copy link

Process « PR par ticket » — ticket créé (passe de revue 2026-08-14)

Cette PR n'était liée à aucun ticket. Ticket créé pour porter le correctif :

  • Refs #126 — TECH: CI — référencer ci-full par URL Forgejo directe taguée (@vN) au lieu de @main

À faire côté développeur :

  1. Référencer le ticket dans la PR : Refs #126 (la PR en l'état référence @main, le ticket demande le tag versionné).
  2. Aligner sur la recommandation du skill forgejo-pr-unified : uses: https://forgejo.tordais.cc/nonobis/templates/.forgejo/actions/ci-full@vN (dernier tag connu : v5) — à chaque bump de templates, publier un nouveau tag vN (nouvelle URL = nouveau hash de cache runner).
## Process « PR par ticket » — ticket créé (passe de revue 2026-08-14) Cette PR n'était liée à aucun ticket. Ticket créé pour porter le correctif : - **Refs #126** — TECH: CI — référencer ci-full par URL Forgejo directe taguée (@vN) au lieu de @main À faire côté développeur : 1. Référencer le ticket dans la PR : `Refs #126` (la PR en l'état référence `@main`, le ticket demande le tag versionné). 2. Aligner sur la recommandation du skill `forgejo-pr-unified` : `uses: https://forgejo.tordais.cc/nonobis/templates/.forgejo/actions/ci-full@vN` (dernier tag connu : `v5`) — à chaque bump de templates, publier un nouveau tag `vN` (nouvelle URL = nouveau hash de cache runner).
nonobis commented 2026-08-14 16:15:10 +02:00
Author
Owner
Copy link

PR fermée — le fix CI réel est passé par nonobis/templates (78cddf9).

Diagnostic des runs 1978/1999 : le step « Run .NET tests » du ci-full construisait la liste des projets avec des espaces (TEST_PROJECTS="$TEST_PROJECTS $proj") → toute la liste tenait sur une ligne → les greps en aval matchaient la ligne entière → OTHERS vide, exclude-test-projects (E2E) inopérant, et Heimdall.E2E.Tests partait en série dans la boucle Domain (32 échecs, Chromium absent). PostgreSQL LXC (tmpfs) fonctionnait déjà (Infrastructure/Integration verts au run 1999).

Correctif livré dans nonobis/templates (main, 78cddf9) : séparateur newline dans TEST_PROJECTS + strip final — simulation vérifiée, E2E exclu, Domain/Application corrects. Les PR #117 et #118 ont été re-trigger dessus.

Cette PR (fix/ci-url-forgejo-directe) est fermée car :

  • son contenu (checkout .ci + PAT) est déjà sur main depuis le retour au système pré-bascule (32840f0) ;
  • l'objectif du titre (URL Forgejo directe @main/@vN) est incompatible avec templates en PRIVÉ (403 User permission denied au premier step, piège documenté) — pas de bascule tant que templates n'est pas repassé public ;
  • la branche ne contient que des commits de debug/re-trigger (22) sans changement utile pour Heimdall.

→ Traité via le ticket TECH #126 (fermé avec la même justification).

**PR fermée — le fix CI réel est passé par nonobis/templates (78cddf9).** Diagnostic des runs 1978/1999 : le step « Run .NET tests » du ci-full construisait la liste des projets avec des **espaces** (`TEST_PROJECTS="$TEST_PROJECTS $proj"`) → toute la liste tenait sur une ligne → les greps en aval matchaient la ligne entière → `OTHERS` vide, `exclude-test-projects` (E2E) inopérant, et Heimdall.E2E.Tests partait en série dans la boucle Domain (32 échecs, Chromium absent). PostgreSQL LXC (tmpfs) fonctionnait déjà (Infrastructure/Integration verts au run 1999). Correctif livré dans `nonobis/templates` (main, 78cddf9) : séparateur **newline** dans TEST_PROJECTS + strip final — simulation vérifiée, E2E exclu, Domain/Application corrects. Les PR #117 et #118 ont été re-trigger dessus. Cette PR (fix/ci-url-forgejo-directe) est fermée car : - son contenu (checkout `.ci` + PAT) est **déjà sur main** depuis le retour au système pré-bascule (32840f0) ; - l'objectif du titre (URL Forgejo directe @main/@vN) est **incompatible** avec `templates` en PRIVÉ (403 `User permission denied` au premier step, piège documenté) — pas de bascule tant que templates n'est pas repassé public ; - la branche ne contient que des commits de debug/re-trigger (22) sans changement utile pour Heimdall. → Traité via le ticket TECH #126 (fermé avec la même justification).
nonobis closed this pull request 2026-08-14 16:15:11 +02:00
Some checks failed
CI / ci (pull_request) Failing after 39m35s
Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug
Comportement incorrect ou risque fonctionnel

  
chore
Maintenance, dépendances, CI, outillage

  
design
Ticket avec composante UI — maquette the-designer requise/intégrée

  
dx:ci

   
dx:database

   
dx:dotnet

   
dx:infra

   
dx:security

   
dx:tests

   
dx:ui

   
etape-1
Étape 1 — MVP Multi-Site (phases 1-12)

  
etape-2
Étape 2 — Caméras + Temps réel (phases 13-14)

  
etape-3
Étape 3 — Vidéo + IA (phases 15-16)

  
etape-4
Étape 4 — Événements + Notifications (phases 17-19)

  
etape-5
Étape 5 — UI Blazor complète (phases 20-23)

  
etape-6
Étape 6 — Production Hardening (phase 24)

  
etape-7
Étape 7 — Mobile (phase 25)

  
feature
Nouvelle fonctionnalité / implémentation spec manquante

  
priority-high
Priorité haute — bloque le cœur métier

  
priority-low
Priorité faible / cosmétique

  
priority-medium
Priorité moyenne

  
security
Vulnérabilités CVE, scans Trivy/OWASP, security hardening

  
spec
Référence à MASTER_SPECIFICATION.md

  
tech-debt
Dette technique, obsolescence, drift documentation

No labels
bug
chore
design
dx:ci
dx:database
dx:dotnet
dx:infra
dx:security
dx:tests
dx:ui
etape-1
etape-2
etape-3
etape-4
etape-5
etape-6
etape-7
feature
priority-high
priority-low
priority-medium
security
spec
tech-debt
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
nonobis
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
nonobis/Heimdall!111
No description provided.