fabric icon indicating copy to clipboard operation
fabric copied to clipboard

DOC / Update create_channel_test_net.md

Open jannesklee opened this issue 2 years ago • 5 comments

The commit exhanges TwoOrgsApplicationGenesis with ChannelUsingRaft because of commit eb16ca in the fabric-sample repository.

jannesklee avatar Nov 05 '23 11:11 jannesklee

Good catch, thank you!

I also see the same reference in docs/source/test_network.md and docs/source/create_channel/create_channel_config.md , do you want to update those in the same PR?

denyeart avatar Nov 06 '23 16:11 denyeart

I changed the according entries. Please be aware that within the create_channel_test_net-article the rest of the description of the configtx.yaml file does not seem to be up to date anymore.

The Capabilities and Applications are not mentioned in the description. The entry Etcdraft is now two times included, because it moved from Orderer to the Profiles section and I am not certain if the description is still correct (even if the block is removed).

jannesklee avatar Nov 06 '23 17:11 jannesklee

I changed the according entries. Please be aware that within the create_channel_test_net-article the rest of the description of the configtx.yaml file does not seem to be up to date anymore.

The Capabilities and Applications are not mentioned in the description. The entry Etcdraft is now two times included, because it moved from Orderer to the Profiles section and I am not certain if the description is still correct (even if the block is removed).

Ok, thanks, I'll take a look.

denyeart avatar Nov 07 '23 21:11 denyeart

One more thing, you'll need to sign the second commit as well to get the DCO check to pass, see https://github.com/hyperledger/fabric/pull/4528/checks?check_run_id=18410805423.

Alternatively, you can squash the two commits into a single signed commit. See https://hyperledger-fabric.readthedocs.io/en/latest/github/github.html#updating-a-pull-request.

denyeart avatar Nov 07 '23 21:11 denyeart

Hello. I found that there are also sections in comments in fabric-samples that need to be revised. So, I've created the following patch and submited it. https://github.com/hyperledger/fabric-samples/pull/1146

satota2 avatar Dec 11 '23 04:12 satota2

Ok, the doc changes have been made in https://github.com/hyperledger/fabric/pull/4781 and https://github.com/hyperledger/fabric/pull/4788, closing.

denyeart avatar Apr 03 '24 14:04 denyeart