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

Use any peer to evaluate system chaincode transactions #3447

Conversation

bestbeforetoday
Copy link
Member

@bestbeforetoday bestbeforetoday commented May 25, 2022

Cherry-pick of d0e2093 from main branch.

System chaincodes are not included in the installed chaincodes returned by service discovery. The Gateway service was relying on discovery results to find peers on which to evaluate transactions and so failed to evaluate system chaincode transaction functions. Now the Gateway service uses any network peer to evaluate transactions invoked on known built-in system chaincodes.

Also added a system test to confirm that the Gateway service can endorse system chaincode transaction functions.

System chaincodes are not included in the installed chaincodes returned by service discovery. The Gateway service was relying on discovery results to find peers on which to evaluate transactions and so failed to evaluate system chaincode transaction functions. Now the Gateway service uses any network peer to evaluate transactions invoked on known built-in system chaincodes.

Also added a system test to confirm that the Gateway service can endorse system chaincode transaction functions.

Signed-off-by: Mark S. Lewis <mark_lewis@uk.ibm.com>
@bestbeforetoday bestbeforetoday marked this pull request as ready for review May 25, 2022 12:10
@bestbeforetoday bestbeforetoday requested a review from a team as a code owner May 25, 2022 12:10
@andrew-coleman andrew-coleman merged commit a6947fa into hyperledger:release-2.4 May 25, 2022
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.

2 participants