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

SmartContracts deployments are failing over the member node in besu network #999

Closed
lakshyakumar opened this issue Sep 8, 2020 · 2 comments · Fixed by #1048
Closed

SmartContracts deployments are failing over the member node in besu network #999

lakshyakumar opened this issue Sep 8, 2020 · 2 comments · Fixed by #1048
Assignees
Labels
besu Related to the Hyperledger Besu DLT Platform bug Something isn't working

Comments

@lakshyakumar
Copy link
Contributor

lakshyakumar commented Sep 8, 2020

Describe the bug
Smart contracts are only deployed on besu node with the address that are added to the RLP encoded string in extraData in genesis.json, i.e., the validator nodes. smartcontract deployments are failing on member nodes in besu.

To Reproduce
Steps to reproduce the behavior:

  1. Edit the besu network.yaml
  2. Edit the crypto generation of IBFT consensus
  3. Modify the role Get node data from main.yaml role of blockchain-automation-framework/platforms/hyperledger-besu/configuration/roles/create/crypto/ibft
  4. Try to deploy the smart contract over one of the member node.
    `
    Expected behavior
    SmartContract should be deployed over the member node.
@lakshyakumar
Copy link
Contributor Author

lakshyakumar commented Sep 9, 2020

@azoumi
Copy link
Contributor

azoumi commented Sep 22, 2020

Fixed with PR #1048 It seems like that the minimum gasprice was the issue. As we are not using gas for contract deployment it can be set to 0 so we don't need to give member nodes wei to deploy contracts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
besu Related to the Hyperledger Besu DLT Platform bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants