-
Notifications
You must be signed in to change notification settings - Fork 0
Omnibus 20.01.2026 (#198 - Lido V3 Phase 2, CSM Share → 7.5%, Grant role to CSM Performance Oracle report weekday offset contract) #580
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
base: feat/next-vote
Are you sure you want to change the base?
Conversation
iamnp
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks solid ❤️
please address the comments
| UPDATE_VAULTS_FEES_IN_OPERATOR_GRID_FACTORY = "0x5C3bDFa3E7f312d8cf72F56F2b797b026f6B471c" # TODO update address after deployment | ||
|
|
||
| # Test parameters | ||
| EXPECTED_VOTE_ID = None # Set to None to create a new vote each test run |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| EXPECTED_VOTE_ID = None # Set to None to create a new vote each test run | |
| EXPECTED_VOTE_ID = 198 |
TheDZhon
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👀
tests/test_2026_01_20.py
Outdated
| # ==================== After DG proposal executed checks ================== | ||
| # ========================================================================= | ||
|
|
||
| # TODO Step 1.1. Check old VaultsAdapter does not have REGISTRY_ROLE on OperatorGrid |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why todos here?
feat: Lido V3 CI tests
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This pull request implements the Lido V3 Phase 2 voting script and comprehensive test suite. The PR contains three main components:
Purpose: Activate Lido V3 Phase 2 (Full Launch), increase Community Staking Module share limit to 7.5%, and grant permissions for CSM Performance Oracle weekday offset functionality.
Changes:
- Submit a Dual Governance proposal with 15 sub-items including role migrations, proxy upgrades, parameter updates, and permission grants
- Replace 7 Easy Track factories for Lido V3 vault management (old Phase 1 → new Phase 2)
- Comprehensive test coverage with over 1,350 lines of test code including scenario tests and permission checks
Reviewed changes
Copilot reviewed 11 out of 11 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
utils/test/rpc_helpers.py |
New utility function for setting storage slots during testing, supporting multiple RPC methods (Hardhat, Anvil, Geth) |
tests/test_2026_01_20.py |
Comprehensive test suite with voting execution, DG proposal execution, Easy Track factory migrations, and scenario tests |
scripts/upgrade_2026_01_20_v3_phase_2.py |
Voting script implementing the proposal with 15 vote items |
tests/snapshot/test_lido_snapshot.py |
Updated snapshot tests to account for max external ratio change (3% → 30%) |
tests/regression/test_permissions.py |
Updated permission expectations for TwoPhaseFrameConfigUpdate contract |
tests/regression/test_gate_seal.py |
Removed temporary PDG unpause code (now handled by the vote) |
configs/config_mainnet.py |
Updated CSM parameters and VaultsAdapter address |
interfaces/TwoPhaseFrameConfigUpdate.json |
New interface definition for the TwoPhaseFrameConfigUpdate contract |
.github/workflows/*.yml |
CI workflow configuration updates (runner and branch changes) |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
test: fix snap test
Lido V3 phase 2 voting script.