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

Added a check to make sure that chaincode is actually running. #4932

Merged
merged 1 commit into from
Jul 29, 2024

Conversation

pfi79
Copy link
Contributor

@pfi79 pfi79 commented Jul 24, 2024

Found that sometimes chaincode won't start. It waits for "Ready" message, but another message comes to it, for example "Init".
This happens when another transaction arrives during chaincode start. And it falls between the two events:

  • Handler for chaincode is already there - the start process has started
  • "Ready" state has not yet occurred.

I corrected the situation by checking the Handler state as well.

@pfi79 pfi79 requested a review from a team as a code owner July 24, 2024 22:02
@pfi79 pfi79 force-pushed the launch-chaincode-error branch 3 times, most recently from 2c76651 to 602453b Compare July 26, 2024 20:22
Signed-off-by: Fedor Partanskiy <fredprtnsk@gmail.com>
@C0rWin C0rWin merged commit 908b965 into hyperledger:main Jul 29, 2024
14 checks passed
@pfi79 pfi79 deleted the launch-chaincode-error branch July 29, 2024 12:38
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