a worker was handed no note at all under a loaded full-tree run
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 52/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- Active
- Stack technique
- go
- Domaine
- testing-qa
Piste de recherche
Start with TestTwoWorkersLiveAtOnceEachGetOnlyItsOwnNote in internal/run/note_channel_test.go and reproduce the package with -race, then compare it with the full-tree run described at commit 2f94ca1f0. Trace the two workers' note delivery decisions and improve the failure output to identify each worker's notes and timing. Done means the runtime or fixture issue is fixed, the change explains which it was, and package -race covers it in CI.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
What happened
TestTwoWorkersLiveAtOnceEachGetOnlyItsOwnNote in internal/run failed once,
inside a run of 112 packages at once on a loaded build box.
--- FAIL: TestTwoWorkersLiveAtOnceEachGetOnlyItsOwnNote (0.66s)
note_channel_test.go:326: alpha was handed its own note 0 times, want once
FAIL github.com/Agent-Field/codeaf/internal/run 29.998s
The dump the assertion prints shows three requests to alpha, each carrying the
work order and the ask, and none of them carrying the note. So the note was not
delivered late or delivered to the wrong worker in that run. It was not there at
all across every request the worker made.
It does not reproduce, and that is the report
Sixteen attempts on the same tree at the same commit, on a box at load 0.33:
| what | result |
|---|---|
-count=5 -run TestTwoWorkersLiveAtOnce ./internal/run/ |
ok 2.758s |
-count=3 ./internal/run/ (whole package, so the other tests run beside it) |
ok 75.353s |
GOMAXPROCS=1 -count=8 -run TestTwoWorkersLiveAtOnce ./internal/run/ |
ok 4.151s |
Filing rather than rerunning is deliberate. The repository's own rule is that a
test failing only when other suites run beside it is a bug report and not a
known shape, and this test is about two workers being live at once, which is the
condition a loaded box actually supplies and an isolated rerun does not.
It is NOT in .github/known-red.txt and must not be added to it.
Why it matters more than an ordinary flake
The behaviour under test is the one #1356 shipped: a note is a channel into a
running worker rather than a log nobody reads. A worker that never receives its
note is that feature not working, and the failing case renders exactly like a
run where nobody sent a note. If this is real, the product symptom is silence.
Replication
Not reliable. What produced it once: go test -count=1 over every package
except internal/tui3 and internal/session on one box, roughly 112 packages,
during a full-tree check. The commit was 2f94ca1f0.
Acceptance
- Whatever the cause, the test's failure output says which of the two workers
got which notes and when, not only that alpha got none. The current dump
shows the requests but not the delivery decisions, so a reader cannot tell a
note that was never written from one written to the wrong worker. - If it is a race in the runtime,
-raceon this package in CI names it. - If it is a race in the fixture, the fixture is fixed, and the change says
which of the two it was. Not "made deterministic".
- Langage dominant
- Go
- Étoiles
- 115
- Forks
- 14
- Merge moyen
- 9 h 38 min
- PR mergées (30 j)
- 749
Préparer son environnement
Nous n'avons pas encore vérifié les fichiers d'installation de ce projet. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de Agent-Field/CodeAF
-
area:chat feature
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
Agent-Field/CodeAF#1510 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
Agent-Field/CodeAF#1489 ·
Les mainteneurs répondent en général sous 1 jour
-
area:chat bug good first issue sev:papercut
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Agent-Field/CodeAF#1470 ·
Les mainteneurs répondent en général sous 1 jour
-
area:chat bug good first issue sev:papercut
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
Agent-Field/CodeAF#1469 ·
Les mainteneurs répondent en général sous 1 jour
-
area:chat bug sev:papercut
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
Agent-Field/CodeAF#1468 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de Agent-Field/CodeAF
Issues similaires
-
enhancement low priority
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
eugenioenko/ttt#674 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
Les mainteneurs répondent en général sous 1 jour
-
Remove obsolete ipAllocationPolicy field from containernodepool-subnetworkref test dependencyOuverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 92/100
GoogleCloudPlatform/k8s-config-connector#13462 ·
Les mainteneurs répondent en général sous 1 jour
-
bug good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
vavallee/bindery#2793 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour