23799238831
91%

Ran 31 Mar 2026 01:20PM UTC

Jobs 0

Files 0

Run time –

Badge

Embed ▾

pending completion

Build # 23799238831

Build Type

push

github

Committed by

web-flow

Commit Message

[test] fix flaky TestSignatureVerifierManagerPolicyUpdateAndRecover (#501)

#### Type of change

- Test update
 
#### Description

Root cause: requireAllUpdate submits many batches (3 per 10ms tick),
filling the outputValidatedTxs channel (capacity 10). When the channel
is full, signVerifier goroutines block on Write() instead of being in
Recv(). Both goroutines in the errgroup deadlock: receiveStatus blocks
on Write (channel full), sendTransactions blocks on Read (no batches),
and neither can cancel gCtx. The drain goroutine was started after
requireAllUpdate, so the channel was already full by the time it ran. On
busy CI machines, the drain goroutine's scheduling delay combined with
buffered responses meant signVerifier[0] could remain stuck on Write for
the entire 30-second EventuallyIntMetric timeout, never reaching Recv()
to detect the broken stream from server.Stop().

Fix: move the drain goroutine before requireAllUpdate so the output
channel never fills up. This ensures signVerifiers are always in their
idle state (blocked on Recv) when the server stops, allowing immediate
stream break detection. Use channel.Ready to guarantee the drain
goroutine is running before proceeding.

#### Related issues

  - resolves #492

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

Run Details

Source Files on build 23799238831

Detailed source file information is not available for this build.

hyperledger / fabric-x-committer / 23799238831
91%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on build 23799238831

hyperledger / fabric-x-committer / 23799238831 91%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on build 23799238831

hyperledger / fabric-x-committer / 23799238831
91%

README BADGES
x