Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update nano test network #675

Closed
wants to merge 1 commit into from
Closed

Conversation

jt-nti
Copy link
Member

@jt-nti jt-nti commented Mar 15, 2022

Fix minor bugs, add network.sh script to simplify standing up the network, and add an option to run the network in a single container

Note: the updated peerNadmin.sh scripts no longer create or join a channel and now only configure the environment for the relevant peer

Signed-off-by: James Taylor jamest@uk.ibm.com

@jt-nti
Copy link
Member Author

jt-nti commented Mar 15, 2022

Note: currently in draft until the ccaas builder is published hyperledger/fabric#3286

@jt-nti jt-nti force-pushed the nano-docker branch 5 times, most recently from d1ec79e to 5526a3e Compare March 15, 2022 17:10
@@ -120,7 +131,7 @@ export CHAINCODE_SERVER_ADDRESS=127.0.0.1:9999
And start the chaincode service:

```
npm run start:server-notls
npm run start:server-nontls

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is it a typo?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @timo-kang, the correct script is start:server-nontls so there is currently a typo in the test-network-nano-bash readme. I included the fix in this draft PR but it might be better to fix sooner since it's not really related to the other changes here.

@jkneubuh
Copy link
Contributor

@jt-nti can this be un-drafted and moved forward? Do we have the builders now as part of the release archives?

@jt-nti
Copy link
Member Author

jt-nti commented Apr 25, 2022

@jkneubuh not yet- there hasn't been a release with the ccaas builder binaries in the release tarballs. Hopefully soon.

Add an option to run the network in a single container

Signed-off-by: James Taylor <jamest@uk.ibm.com>

# Conflicts:
#	test-network-nano-bash/README.md
@denyeart
Copy link
Contributor

The intent of test network nano bash was to have a minimal sample that can be run on local machine without docker layers to complicate things. For docker options, there is the traditional test-network and microfab. I'll go ahead and close.

@denyeart denyeart closed this Apr 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants