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

Add new orderer to channel #72

Merged
merged 10 commits into from
Dec 10, 2021
Merged

Conversation

sapthasurendran
Copy link
Collaborator

@sapthasurendran sapthasurendran commented Nov 29, 2021

closes #70

Signed-off-by: sapthasurendran <[email protected]>
Signed-off-by: sapthasurendran <[email protected]>
Signed-off-by: sapthasurendran <[email protected]>
Signed-off-by: sapthasurendran <[email protected]>
Signed-off-by: sapthasurendran <[email protected]>
# "Specs" - See PeerOrgs for complete description
# ---------------------------------------------------------------------------
Specs:
- Hostname: orderer
Copy link

Choose a reason for hiding this comment

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

if we want to generate the crypto only for orderer6, then we can remove this entry Hostname: orderer

# import utils
. scripts/envVar.sh

# fetchChannelConfig <org> <channel_id> <output_json>
Copy link

Choose a reason for hiding this comment

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

I think this comment needs to go down closer to function definition.

Signed-off-by: sapthasurendran <[email protected]>
Signed-off-by: sapthasurendran <[email protected]>
Signed-off-by: sapthasurendran <[email protected]>
Signed-off-by: sapthasurendran <[email protected]>
@sapthasurendran
Copy link
Collaborator Author

sapthasurendran commented Dec 10, 2021

@davidkel @Param-S I have updated the scripts. Works fine.

@Param-S Thank you for the help.

Copy link
Owner

@davidkel davidkel left a comment

Choose a reason for hiding this comment

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

LGTM

@davidkel davidkel merged commit 4c3e547 into davidkel:main Dec 10, 2021
@sapthasurendran sapthasurendran linked an issue Dec 10, 2021 that may be closed by this pull request
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.

Test adding a new orderer Test adding a new orderer
3 participants