upgrade custom-endorser to test committer v1.0.3#24
Open
Storm1289 wants to merge 1 commit into
Open
Conversation
Upgrade the test committer to v1.0.3 and bump all direct go.mod dependencies to their latest versions, following the self-contained crypto pattern from the fabric-x-sdk Makefile. - extract the committer's embedded crypto material instead of generating it with cryptogen (orgs are now peer-org-0/peer-org-1) - migrate to the committer's serve package (StartAndServe, Servers, DefaultHealthCheckService) and the ctx-aware NewSubmitter/fabrictest.Start - dial the orderer by name (its cert has a localhost SAN, no IP SAN) - wait after submitting so the async broadcast reaches the orderer - update Makefile, compose, sample configs and README accordingly Signed-off-by: Storm1289 <divakarsharm2934@gmail.com>
Author
|
Hi @arner, whenever you have some time, take a look at the PR. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #23.
Summary
Upgrades the
custom-endorsersample to the test committerv1.0.3and refreshes all directgo.moddependencies to their latest compatible versions, aligning the sample with the self-contained crypto pattern used in thefabric-x-sdkMakefile.This removes the legacy
cryptogen-generated network setup and migrates the sample to the new committer API surface.Changes
Dependency upgrades
Updated:
fabric-x-committerv0.1.9→v1.0.3fabric-x-common→v0.2.6fabric-x-sdk→ latestgrpcfabric-lib-gofabric-protos-go-apiv2This brings the sample in line with the current SDK / committer stack.
Crypto bootstrap changes
init-networknow extracts the embedded crypto material directly from the committer image instead of generating it viacryptogen.Changes:
peer-org-0peer-org-1crypto-config.yamlconfigtx.yamlThis matches the new self-contained committer flow.
Committer API migration
Migrated from the old
connectionpackage APIs:Before:
connection.ServerConfigconnection.StartServiceconnection.DefaultHealthCheckServiceAfter:
serve.Configserve.Serversserve.StartAndServeAlso updated:
NewSubmitter(ctx, ...)fabrictest.Start(ctx, ...)to the new context-aware signatures.
Docker Compose / namespace flow
The committer now runs self-contained without crypto mounts.
Namespace creation now follows the explicit multi-step endorse → submit flow under: