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

[CDK-46] Add DAP CLI on Validium node #69

Merged
merged 3 commits into from
Feb 8, 2024

Conversation

goran-ethernal
Copy link
Collaborator

What does this PR do?

This PR adds a new CLI command to cdk-validium-node for setting up a new data availability protocol.

The command is called set-data-availability-protocol (alias is set-dap), and it has following flags:

  • data-availability-address (da-addr) - address of the new data availability protocol.
  • key-store-path (ksp) - the path of the key store file containing the private key of the account going to set new data availability protocol.
  • password (pw) - the password do decrypt the key store file.
  • cfg (c) - path tho the configuration file.
  • network (net) - indicates which network configuration file should be loaded. For now only custom is supported.
  • custom-network-file (net-file) - path to the custom network configuration file.

The command loads the sender private key, creates an etherman instances and sends a transaction to the https://github.com/0xPolygonHermez/zkevm-contracts/blob/develop/contracts/v2/consensus/validium/PolygonValidiumEtrog.sol#L254 function.

This PR also adds an e2e test to check if the new CLI commands works as expected.

Copy link
Collaborator

@arnaubennassar arnaubennassar left a comment

Choose a reason for hiding this comment

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

💯

@goran-ethernal goran-ethernal force-pushed the CDK-46-add-dap-cli-on-validium-node branch from 668b6f9 to 9158242 Compare February 8, 2024 15:16
@goran-ethernal goran-ethernal merged commit c3a18cd into develop Feb 8, 2024
17 checks passed
@goran-ethernal goran-ethernal deleted the CDK-46-add-dap-cli-on-validium-node branch February 8, 2024 15:36
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.

3 participants