TECH: refonte CI Heimdall — gate Sonar bloquant, Integration tests actifs, Dependency-Check non filtrable #143

Closed
opened 2026-08-18 14:06:52 +02:00 by nonobis · 0 comments
Owner

Contexte

La PR #137 (feature télémétrie) embarque une refonte de .forgejo/workflows/ci.yml (filtrage des jobs par fichiers modifiés, nightly 4h, détection PR étendue). Trois changements de politique CI non validés y étaient introduits :

  1. Gate SonarQube passé NON bloquant : exit 0 systématique après quality gate = ERROR (avant : exit 1 bloquant, sauf si échec limité à new_coverage/new_violations).
  2. Integration tests (PostgreSQL/Testcontainers) désactivés : steps if: false + continue-on-error: true.
  3. OWASP Dependency-Check skippé quand aucun .csproj ne change : les CVE NuGet ne sont plus scannées sur les PRs de code pur.

Décision (the-devops, 2026-08-18)

La refonte est sortie de la PR #137 (revert) et déplacée ici avec corrections :

  • Gate Sonar bloquant restauré (exit 1 ; skip uniquement si conditions d'échec = new_coverage/new_violations uniquement — hérité du main, documenté)
  • test-db toujours actif (plus de filtre db_changed) : les tests DB font partie du gate
  • Integration tests réactivés (plus de if:false ni continue-on-error)
  • Dependency-Check toujours actif (plus de filtre deps_changed) : politique CVE bloquantes conservée
  • Filtrages conservés et assumés : test-web (bUnit) si Web change, docker+Trivy si code/Dockerfile change (nightly 4h couvre le reste)

Livrable

PR CI dédiée avec ces corrections, CI verte, gate Sonar exigeable au merge.

## Contexte La PR #137 (feature télémétrie) embarque une refonte de `.forgejo/workflows/ci.yml` (filtrage des jobs par fichiers modifiés, nightly 4h, détection PR étendue). Trois changements de politique CI non validés y étaient introduits : 1. **Gate SonarQube passé NON bloquant** : exit 0 systématique après quality gate = ERROR (avant : exit 1 bloquant, sauf si échec limité à new_coverage/new_violations). 2. **Integration tests (PostgreSQL/Testcontainers) désactivés** : steps if: false + continue-on-error: true. 3. **OWASP Dependency-Check skippé quand aucun .csproj ne change** : les CVE NuGet ne sont plus scannées sur les PRs de code pur. ## Décision (the-devops, 2026-08-18) La refonte est sortie de la PR #137 (revert) et déplacée ici avec corrections : - Gate Sonar **bloquant** restauré (exit 1 ; skip uniquement si conditions d'échec = new_coverage/new_violations uniquement — hérité du main, documenté) - test-db **toujours actif** (plus de filtre db_changed) : les tests DB font partie du gate - Integration tests **réactivés** (plus de if:false ni continue-on-error) - Dependency-Check **toujours actif** (plus de filtre deps_changed) : politique CVE bloquantes conservée - Filtrages conservés et assumés : test-web (bUnit) si Web change, docker+Trivy si code/Dockerfile change (nightly 4h couvre le reste) ## Livrable PR CI dédiée avec ces corrections, CI verte, gate Sonar exigeable au merge.
Sign in to join this conversation.
No milestone
No project
No assignees
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.

Reference
nonobis/Heimdall#143
No description provided.