[P2][ci] Faire échouer installcheck dès qu'un contrôle intermédiaire échoue #58
Labels
No labels
Agent/Autonomous OK
Agent/Demeter
Agent/Hermes
Agent/Human
Agent/Needs Review
Area/API
Area/Auth
Area/CLI
Area/Data
Area/Docs
Area/Infra
Area/UI
CI/Failing
CI/Flaky
CI/Green
CI/Needs Runner
CI/Needs Workflow
Compat/Backward Compatible
Compat/Breaking
Compat/Migration
Deploy/Ansible
Deploy/Blocked
Deploy/Homelab
Deploy/Needs Config
Deploy/Needs Secret
Deploy/Ready
Deploy/Rollback
Kind/Bug
Kind/Chore
Kind/Design
Kind/Documentation
Kind/Enhancement
Kind/Feature
Kind/Refactor
Kind/Security
Kind/Spike
Kind/Testing
Ops/Backup
Ops/Incident
Ops/Maintenance
Ops/Monitoring
Ops/Upgrade
Priority
Critical
Priority
High
Priority
Low
Priority
Medium
Priority
Someday
Reviewed
Confirmed
Reviewed
Duplicate
Reviewed
Invalid
Reviewed
Won't Fix
Risk
Critical
Risk
High
Risk
Low
Risk
Medium
Security/Auth
Security/Disclosure
Security/Hardening
Security/Secret
Size
L
Size
M
Size
S
Size
XL
Size
XS
Stack/Ansible
Stack/Docker
Stack/Forgejo
Stack/Hermes
Stack/Node
Stack/Postgres
Stack/Python
Stack/Systemd
Status
Abandoned
Status
Blocked
Status
In Progress
Status
In Review
Status
Need More Info
Status
Ready
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
2027a/murmur#58
Loading…
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Gravité et périmètre
P2 / medium. Makefile et validation CI de l'installation, commit
65442d2.Constat
La recette
Makefile:112et suivantes enchaîne installation, assertions et désinstallation avec des points-virgules sans fail-fast. Le statut du dernier test peut masquer un échec antérieur, y compris l'impossibilité d'exécuter le binaire installé. La CI invoque cette cible et hérite donc de ce faux vert.Reproduction
Dans une copie temporaire :
make installcheck MAKE=true. Les installations/désinstallations deviennent des no-op, plusieurs assertions échouent, l'exécution du binaire installé indique qu'il n'existe pas, mais la cible termine avec le code 0. Aucun fichier système installé.Impact
Une installation cassée peut être présentée comme valide. Ne bloque pas directement la dictée, mais invalide une garantie importante du contrôle de livraison.
Correction et acceptation
Rendre la recette fail-fast en gardant le trap de nettoyage. Ajouter un contrôle de régression ou une injection reproductible prouvant que l'échec d'une installation, d'une assertion intermédiaire ou de l'exécution du binaire produit un statut non nul. Vérifier aussi le chemin nominal, les sentinelles étrangères et le nettoyage du DESTDIR temporaire.