hyperledger / fabric-x-committer / 21826218458

89%

main: 91%

Build Type

Pull #304

github

Committed by cendhu

Commit Message

Fix flaky coordinator loadgen test by waiting for setup TX commits

CoordinatorAdapter.Progress() was returning the send counter (nextBlockNum)
instead of the commit counter (TransactionsCommitted). This caused
submitWorkloadSetupTXs to proceed before the config TX was actually committed,
so subsequent namespace TXs could arrive before policies were established.
Without the sidecar (which serializes the config TX), the coordinator test
relies on Progress() to gate setup TX ordering.

Also fix error swallowing in coordinator's sendTxStatus goroutine, which
could cause cascading deadlocks on stream errors.

Fix flaky TestLoadGenForLoadGen by explicitly stopping the sub-client
before test cleanup tears down the main server. Test cleanup runs in
LIFO order, so the main server's gRPC Stop ran before the sub-client
was cancelled, causing in-flight AppendBatch RPCs to hit "connection
reset by peer"

Signed-off-by: Senthilnathan <cendhu@gmail.com>

Pull Request Pull Request #304: Fix flaky coordinator loadgen test by waiting for setup TX commits

Coverage Stats

0 of 1 new or added line in 1 file covered. (0.0%)

16 existing lines in 5 files now uncovered.

9081 of 10210 relevant lines covered (88.94%)

1.29 hits per line