Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

merchant suite: TestKilledAttemptConvergesAfterRestart failed once under full-suite load (5.8 s convergence timeout) — not reproducible in isolation

Aperta
#733 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

  • #774 di @Amperstrand — chiusa senza merge
  • #887 di @Amperstrand — chiusa senza merge

Valutazione

Difficoltà
3/5
Tempo stimato
1-2 giorni
Idoneità per principianti
48/100
Tipo di issue
Bug
Chiarezza
Abbastanza chiara
Stato di attività
Attiva
Stack tecnologico
go
Ambito
testing

Direzione di ricerca

Start with src/merchant/receive_intent_test.go:152 and rerun the full package test under load with -v to capture which convergence assertion fails. The issue reports no reliable isolated reproduction, so investigate the grant retry and journal reconciliation waits only after identifying the expired deadline. Done means the intermittent failure is understood and the test passes reliably under the relevant load.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Summary

TestKilledAttemptConvergesAfterRestart (src/merchant/receive_intent_test.go:152, the #502 kill-boundary row) failed once in a heavily loaded full-package run (~587 s with -race), hitting a ~5 s convergence wait. It did not reproduce in 10 isolated repetitions, with or without the merchant ndsctl seam from #729. Thin evidence so far — filing so the intermittence is on the record rather than in one contributor's terminal; not urgent.

Found while validating #729.

Verified on

  • Commit: cb4db7f (main, 2026-10-07); bare linux host
  • 1 failure in 3 full-suite runs; 0/10 isolated runs (with seam), 0/10 (without)

Evidence (verbatim)

--- FAIL: TestKilledAttemptConvergesAfterRestart (5.80s)

(The failure detail lines scrolled past in the suite log; the 5.80 s duration points at one of the test's 5 s convergence deadlines. Re-running the suite with -v and capturing the assertion line is the first step of any investigation.)

Reproduce

Not reliably reproducible yet:

cd src/merchant
go test -race -count=10 -tags testenv . -run 'TestKilledAttemptConvergesAfterRestart'   # 0/10 both with and without the #729 seam
go test -race -count=1 -tags testenv .                                                  # one occurrence in three attempts, under load

Troubleshooting guide

  1. The test installs its own fake ndsctl (installRenewalNdsctl), so the #729 package seam cannot change which binary it execs — the identical 0/10 rate with and without the seam supports "load-dependent timing", not interference.
  2. If it recurs, capture -v output first; the interesting question is which convergence wait expired (grant retry cadence vs. journal reconciliation after the simulated restart).

References

  • Test added by #502 (09ea3af)
  • Surfaced during validation of #729
Lingua principale
Go
Stelle
12
Fork
14
Merge medio
2g 22h
PR unite (30g)
267

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di OpenTollGate/tollgate-module-basic-go

Tutte le issue di OpenTollGate/tollgate-module-basic-go

Issue simili

Altre issue su Go

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.