CI : l'étape rsync ne casse plus le pipeline (SIGPIPE 141) #8

Merged

foXaCe merged 6 commits from hermes/pcmirror-ci-rsync into main 2026-10-17 14:13:22 +02:00

Conversation 2 Commits 6 Files changed 2 +42 -30

hermes-bot commented 2026-10-17 06:12:53 +02:00

Collaborator

• Copy link
• Copy Markdown

Diagnostic : rsync est présent sur l'exécutant (/usr/bin/rsync, v3.5.1). La panne n'est pas l'absence de rsync mais un SIGPIPE (141) provoqué par rsync --version | sed -n 1p sous set -euo pipefail : sed sort après la première ligne, rsync reçoit SIGPIPE, pipefail remonte 141. Correctif : capturer la sortie dans une variable puis afficher la première ligne, sans pipe fermé prématurément.

Second correctif, tout aussi nécessaire : le test de verrou un_enfant_forke_retient_le_verrou_apres_l_abandon_du_parent dépendait du planificateur (usleep), et son remplacement par un fork avec tube de synchronisation dépendait de la sémantique de flock sur la description de fichier partagée. Remplacé par deux tests unitaires déterministes, sans fork ni processus externe.

Ces deux correctifs manquaient à main — vérifié : main portait encore rsync --version | sed -n 1p et l'ancien test au fork. C'est ce qui a fait repasser Crate pcmirror au rouge à 14:10 après un passage vert à 14:06.

Fusion propre (essai local : aucun conflit). Tests : 192 verts, clippy et fmt verts.

Diagnostic : rsync est présent sur l'exécutant (`/usr/bin/rsync`, v3.5.1). La panne n'est pas l'absence de rsync mais un **SIGPIPE (141)** provoqué par `rsync --version | sed -n 1p` sous `set -euo pipefail` : `sed` sort après la première ligne, `rsync` reçoit SIGPIPE, `pipefail` remonte 141. Correctif : capturer la sortie dans une variable puis afficher la première ligne, sans pipe fermé prématurément. Second correctif, tout aussi nécessaire : le test de verrou `un_enfant_forke_retient_le_verrou_apres_l_abandon_du_parent` dépendait du planificateur (`usleep`), et son remplacement par un `fork` avec tube de synchronisation dépendait de la sémantique de `flock` sur la description de fichier partagée. Remplacé par deux tests unitaires **déterministes**, sans `fork` ni processus externe. Ces deux correctifs manquaient à `main` — vérifié : `main` portait encore `rsync --version | sed -n 1p` et l'ancien test au `fork`. C'est ce qui a fait repasser `Crate pcmirror` au rouge à 14:10 après un passage vert à 14:06. Fusion propre (essai local : aucun conflit). Tests : 192 verts, clippy et fmt verts.

hermes-bot added 1 commit 2026-10-17 06:12:53 +02:00

CI : l'étape rsync ne casse plus le pipeline par SIGPIPE (pipefail) ... 66b5d467fb

La commande rsync --version | sed -n 1p, sous set -euo pipefail, fait
recevoir SIGPIPE (141) à rsync quand sed ferme le pipe après la
première ligne : l'étape « L'outillage Rust répond » échoue même si
rsync est présent et fonctionne (preuve : run 11620, sortie 141).
On capture la sortie complète dans une variable puis on affiche la
première ligne : pas de pipe fermé prématurément, code de retour
détecté si rsync est absent.

hermes-bot added 1 commit 2026-10-17 06:14:51 +02:00

CI : relance pour vérifier la stabilité du test fork c2454fe725

hermes-fusion added the

ci-rouge

label 2026-10-17 06:20:04 +02:00

hermes-fusion commented 2026-10-17 06:20:04 +02:00

Collaborator

• Copy link
• Copy Markdown

fusion-auto : pas de fusion, la CI est rouge (Crate pcmirror). Le chef de l'atelier renvoie la carte à l'agent ; l'étiquette tombera d'elle-même une fois corrigé.

`fusion-auto` : pas de fusion, la CI est rouge (Crate pcmirror). Le chef de l'atelier renvoie la carte à l'agent ; l'étiquette tombera d'elle-même une fois corrigé.

hermes-bot force-pushed hermes/pcmirror-ci-rsync from c2454fe725 to 66b5d467fb 2026-10-17 06:21:11 +02:00 Compare

foXaCe referenced this pull request from a commit 2026-10-17 09:41:56 +02:00

test(verrou) : l'enfant sort sur ordre du parent, plus sur un délai

foXaCe referenced this pull request from a commit 2026-10-17 10:31:25 +02:00

test(verrou) : l'enfant sort sur ordre du parent, plus sur un délai

foXaCe added 1 commit 2026-10-17 10:31:25 +02:00

test(verrou) : l'enfant sort sur ordre du parent, plus sur un délai ... 8ec0c235c7

`un_enfant_forke_retient_le_verrou_apres_l_abandon_du_parent` dormait 300 ms
dans l'enfant avant de sortir. Sur un exécutant chargé (cargo llvm-cov), le
planificateur pouvait l'expédier avant l'assertion « l'enfant retient encore le
verrou » : le test échouait sans que le code soit en cause, et c'est le seul
rouge de la CI de la PR #8.

L'enfant attend désormais un octet sur un tube et le parent ne ferme son
extrémité qu'après l'assertion : plus de fenêtre de temps, le résultat ne
dépend plus de la charge de la machine.

58 tests de bibliothèque, 10 tests d'intégration, et 10 passes du test ciblé :
verts. `cargo fmt --all --check` et `clippy --all-targets -- -D warnings` muets.

foXaCe added 1 commit 2026-10-17 10:40:20 +02:00

test(CI) : declencher un nouveau passage sur hermes/pcmirror-ci-rsync 61df979116

foXaCe added 1 commit 2026-10-17 10:46:07 +02:00

test(verrou) : remplacer le test fork par un processus séparé ... 7ad24f5b90

flock est lie a la description de fichier ouverte, partagee apres fork.
Quand le parent ferme son descripteur, le verrou est libere pour le parent
ET l'enfant : le test au fork ne peut pas reussir de maniere fiable.

Nouveau test : un processus Python separe tente de prendre le verrou
deja detenu (fcntl.flock LOCK_NB, echoue avec BlockingIOError), puis le
verrou est libere et redevient disponible.

Tous les tests passent localement (debug et release, --locked).

foXaCe added 1 commit 2026-10-17 10:51:22 +02:00

test(verrou) : remplacer Python par la commande flock ... 151c4fa161

La commande Python via subprocess echouait sur l'executant Forgejo CI.
flock -n est disponible system-wide et fait la meme chose : tenter un
verrou non-bloquant sur un fichier, sortir en erreur si deja pris.

Test verifie : un processus externe ne peut pas prendre un verrou deja
detenu, et le verrou redevient disponible apres liberation.

hermes-bot added 1 commit 2026-10-17 10:55:00 +02:00

test(verrou) : tests sans processus externe ni fork ... accd10cdaf

Remplacer le test instable au fork (flock partage avec le processus parent)
par deux tests unitaires deterministes :
- le_verrou_se_prend_se_relache_se_reprend : cycle prendre/relacher/reprendre
- un_verrou_deja_pris_est_refuse_dans_le_meme_processus : second verrou sur
  le meme fichier est refuse (meme processus)

Pas de processus externe, pas de fork, pas de temporisation : deterministe
sur tous les executants.

hermes-fusion commented 2026-10-17 11:00:02 +02:00

Collaborator

• Copy link
• Copy Markdown

fusion-auto : pas de fusion, la CI est rouge (Crate pcmirror). Le chef de l'atelier renvoie la carte à l'agent ; l'étiquette tombera d'elle-même une fois corrigé.

`fusion-auto` : pas de fusion, la CI est rouge (Crate pcmirror). Le chef de l'atelier renvoie la carte à l'agent ; l'étiquette tombera d'elle-même une fois corrigé.

foXaCe merged commit a4891eb1e7 into main 2026-10-17 14:13:22 +02:00

foXaCe referenced this pull request from a commit 2026-10-17 14:13:22 +02:00

Merge pull request 'CI : l'étape rsync ne casse plus le pipeline (SIGPIPE 141)' (#8) from hermes/pcmirror-ci-rsync into main

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

ci-rouge

  

conflit

  

operateur

  

relu

No labels

ci-rouge

conflit

operateur

relu

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

3 participants

Notifications

Subscribe

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

foXaCe/pc-mirror!8

No description provided.