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

chore: staking deployment scripts #177

Merged
merged 22 commits into from
Jul 23, 2024
Merged

chore: staking deployment scripts #177

merged 22 commits into from
Jul 23, 2024

Conversation

kupermind
Copy link
Collaborator

  • Staking deployment scripts.

@kupermind kupermind requested review from DavidMinarsch and 77ph July 19, 2024 19:09
77ph
77ph previously approved these changes Jul 19, 2024
7. EOA to set up correct L1->L2 links for all the bridging contracts calling `setL2TargetDispenser(L2 corresponding contract)`;
3. EOA to deploy Dispenser with VoteWeighting contract being deployed before that in `autonolas-governance`;
4. EOA to change Dispenser address in VoteWeighting calling `changeDispenser(Dispenser)`;
5. DAO to unpause staking incentives in Dispenser calling `setPauseState(0)`;
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
5. DAO to unpause staking incentives in Dispenser calling `setPauseState(0)`;
5. EOA to unpause staking incentives in Dispenser calling `setPauseState(0)`;

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

@DavidMinarsch
Copy link
Contributor

@DavidMinarsch
Copy link
Contributor

@DavidMinarsch
Copy link
Contributor

@DavidMinarsch
Copy link
Contributor

@DavidMinarsch
Copy link
Contributor

@DavidMinarsch
Copy link
Contributor

DavidMinarsch
DavidMinarsch previously approved these changes Jul 20, 2024
10. DAO to change staking parameters in Tokenomics calling `changeStakingParams()`;
11. DAO to change Treasury managers calling `changeManagers(ZeroAddress, ZeroAddress, DispenserTwo)`;
11. DAO to change Treasury managers calling `changeManagers(ZeroAddress, ZeroAddress, Dispenser)`;
12. DAO to enable bridge deposit processors in Dispenser calling `setDepositProcessorChainIds()`;
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
12. DAO to enable bridge deposit processors in Dispenser calling `setDepositProcessorChainIds()`;
12. EOA to enable bridge deposit processors in Dispenser calling `setDepositProcessorChainIds()`;

Copy link
Contributor

Choose a reason for hiding this comment

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

This should be moved above step 8!

4. EOA to change Dispenser address in VoteWeighting calling `changeDispenser(Dispenser)`;
5. EOA to unpause staking incentives in Dispenser calling `setPauseState(0)`;
6. EOA to add a retainer address as a nominee in VoteWeighting;
7. EOA to deploy staking bridging contracts on L1 and L2 with StakingFactory contract being deployed before that in `autonolas-registries`;
8. EOA to transfer ownership rights of Dispenser to Timelock calling `changeOwner(Timelock)`;
Copy link
Contributor

Choose a reason for hiding this comment

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

@kupermind we still need to do this change of the Dispenser ownership!

Copy link
Contributor

Choose a reason for hiding this comment

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

For this - please confirm Dispenser is in the correct state to transfer ownership.

Comment on lines 15 to 16
13. EOA to set up correct L1->L2 links for all the bridging contracts calling `setL2TargetDispenser(L2 corresponding contract)`;
14. EOA to transfer ownership rights of all the L2 bridging contracts to Timelock representation calling `changeOwner(Timelock)`.
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's move this above 9 as technically that's the correct order

Copy link
Contributor

Choose a reason for hiding this comment

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

14 was now done - please confirm all is good there

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

All is good, confirmed by scripts

@DavidMinarsch DavidMinarsch merged commit 9cc1126 into main Jul 23, 2024
2 checks passed
@DavidMinarsch DavidMinarsch deleted the staking_deployment branch July 23, 2024 13:41
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