hyperledger-labs / fabric-smart-client / 24720167728 / 7
70%
main: 70%

DEFAULT BRANCH: main
Ran
Files 534
Run time 16s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 7.471%. Remained the same
24720167728.7

push

github

web-flow 
fix(fabricx): restore arma -> BFT broadcaster alias (#1249)

* fix(fabricx): restore arma -> BFT broadcaster alias

Commit dfdcbba6 (PR #1206 "support for GetConfigTransaction") refactored
the fabricx channel/committer initialization and dropped `NewCommitter`,
which used to register the arma consensus alias:

    os.Broadcasters["arma"] = os.Broadcasters[ordering.BFT]

That alias was not carried over to the new architecture, so v0.9.0+
fabricx can no longer talk to deployments that use fabric-x-orderer /
fabric-x-common, where `ConsensusType="arma"` is the standard (see the
SampleFabricX profile in fabric-x-common/sampleconfig/configtx.yaml and
the encoder ConsensusTypeArma constant).

Symptom in a real fabric-x deployment (FSC v0.10.0 + fabric-x-orderer
v0.0.24 + fabric-x-committer v0.1.9):

    failed to apply config update: failed to update ordering service:
    failed to set consensus type from channel config: no broadcaster
    found for consensus [arma]

The subsequent retries re-invoke membershipService.Update with the same
genesis envelope and trip the sequence-0 config validator, which masks
the root cause:

    failed validating config for channel [arma]: config currently at
    sequence 0, cannot validate config at sequence 0

Fix: translate consensusType "arma" to ordering.BFT inside the fabricx
channel wrapper's Configure adapter. This keeps the alias localized to
fabricx (the fabric generic ordering service is untouched), matches the
intent of the original registration in NewCommitter, and preserves the
"arma IS fabric-x's BFT implementation" semantic that fabric-x-orderer
encodes in its genesis blocks.

Signed-off-by: Stone <stoneshi@sign.global>

2955 of 39554 relevant lines covered (7.47%)

0.66 hits per line

Source Files on job utest-postgres - 24720167728.7