Skip to content

fix(devnet): compatibility with upcoming mpc node version 3.0.0 #715

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

Merged
merged 6 commits into from
Jul 25, 2025

Conversation

kevindeforth
Copy link
Contributor

@kevindeforth kevindeforth commented Jul 24, 2025

resolves #521
requires https://github.com/Near-One/infra-ops/pull/503
related follow-ups:

changes:

  • deploy-contract command is now split into two commands deploy-contract (deploy the code) and init-contract (initialize the contract).
  • add-keys method, adding the public keys generated by the nodes to the mpc and responder accounts (signer and responder keys).
  • unused --max-requests-to-remove argument has been removed from the contract commands
  • devnet no longer panics upon calling describe in case the contract is not initialized:
thread 'main' panicked at devnet/src/mpc.rs:812:10:
Expected rpc node to respond: ServerError(HandlerError(ContractExecutionError { vm_error: "wasm execution failed with error: HostError(GuestPanic { panic_msg: \"Calling default not allowed.\" })", block_height: 206548722, block_hash: 2LPrNt4BEv2MU6GVRRCZLYXvD1dCUYFRymr5mXtdYWZC }))
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

@kevindeforth kevindeforth changed the title fix(devnet): fix(devnet): compatibility with mpc node version 2.2.0 Jul 24, 2025
@netrome netrome changed the title fix(devnet): compatibility with mpc node version 2.2.0 fix(devnet): compatibility with mpc node version 3.0.0 Jul 24, 2025
@netrome netrome changed the title fix(devnet): compatibility with mpc node version 3.0.0 fix(devnet): compatibility with upcoming mpc node version 3.0.0 Jul 24, 2025
Copy link
Collaborator

@netrome netrome left a comment

Choose a reason for hiding this comment

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

Started reviewing. One question mark so far. Will continue tomorrow morning.

Copy link
Collaborator

@netrome netrome left a comment

Choose a reason for hiding this comment

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

Thanks for fixing devnet 🙏

@kevindeforth kevindeforth added this pull request to the merge queue Jul 25, 2025
Merged via the queue into main with commit 4546c1c Jul 25, 2025
13 checks passed
@kevindeforth kevindeforth deleted the kd/fix-devnet branch July 25, 2025 09:17
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.

[Task] Update devnet to work with new key setup
2 participants