Skip to content

Conversation

@lgaroche
Copy link

Add Nethermind's Starknet Staking v2 package, with signer configuration and metrics.

Remarks:

  • I'm not sure of all the package naming, numbering and DNS resolution in the case of the multi-variant setup
  • Is the root level docker compose still needed?
  • Staking configuration could be added to the wizard

Copy link

@pablomendezroyo pablomendezroyo left a comment

Choose a reason for hiding this comment

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

Could you resolve the conflicts? we have been implementing changes recently

thanks for the contribution!

{
"name": "juno.dnp.dappnode.eth",
"version": "0.1.0",
"version": "2.0.3",

Choose a reason for hiding this comment

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

This version is not the upstream (starknet-staking) one, its the dappnode package version. So current dappnode "starknetstaking" package is 0.1.0, next should be 0.1.1

Copy link
Author

Choose a reason for hiding this comment

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

The version was following the numbering on the original repository. I rolled it back to 0.1.2

@lgaroche
Copy link
Author

Rebased on main to include latest updates. Also bumped the staking upstream package version to 0.3.0.
I tested it on my dappnode (only the mainnet version) and it is successfully attesting.

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