Hacktoberfest 2026:メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

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

オープン
#733 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

まだ誰も着手していません。

  • #774 @Amperstrand による — マージされずにクローズ
  • #887 @Amperstrand による — マージされずにクローズ

評価

難易度
3/5
見積もり時間
1〜2日
初心者へのやさしさ
48/100
issue の種類
バグ
明瞭さ
おおむね明確
活発さ
活発
技術スタック
go
領域
testing

調査の方向性

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.

索引モデルが issue の本文から書いたものです。

説明

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
主要言語
Go
スター
12
フォーク
14
平均マージ
3日 21時間
マージ済み PR(30日)
269

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

OpenTollGate/tollgate-module-basic-go のほかの issue

OpenTollGate/tollgate-module-basic-go の issue をすべて見る

似ている issue

Go の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。