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.

Sign up for GitHub

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

Jump to bottom

SR + CSM + Negative Rebase #280

Merged
merged 274 commits into from
Nov 18, 2024
Merged

SR + CSM + Negative Rebase #280

merged 274 commits into from
Nov 18, 2024

Conversation

mkurayan
Copy link
Contributor

@mkurayan mkurayan commented Oct 16, 2024 •
edited by TheDZhon
Loading

Changes for:

  • Staking Router 2.0
  • CSM
  • Negative Rebase
  • Instadapp Oracle address rotation

mkurayan and others added 30 commits June 25, 2024 17:52
@mkurayan
test: fix test_permissions
09ac11e
@mkurayan
Merge commit '2106f303141490ddc12799a30bda6452ac696b6f' into fix/stak…
7f64e9f 
…ing-router-v2-test
@Amuhar
fix: submit correct part of voting script
facea16
@Amuhar
fix: removed duplicated script
9bc7dda
@krogla
fix: config for new forks
72c5e5e
@krogla
Merge branch 'fix/staking-router-v2' of github.com:lidofinance/script…
e388562 
…s into fix/staking-router-v2
@krogla
fix: refactor vote scripts
632de1c
@krogla
fix: add holesky vote test
43e575f
@krogla
fix: vote script paths
1122e60
@mkurayan
Merge commit '7f64e9fb7747d7c11dfe034bc65641ec8ce0f161' into fix/stak…
939d5dc 
…ing-router-v2

# Conflicts:
#	tests/regression/test_permissions.py
@mkurayan
feat: update interfaces
f9cd31d
@mkurayan
fest: set accint balance with Anvil
e968086
@mkurayan
feat: make sure that LDO_HOLDER_ADDRESS_FOR_TESTS has some eth before…
0404406 
… start vote
@avsetsin
fix: parse events from receipts
be02029
@Amuhar
Merge pull request #233 from lidofinance/fix/parse-events-from-receipt
89a022b 
fix: parse events from receipts
@avsetsin
fix: group logs from receipt
d3804f5
@Amuhar
fix: add easy track for csm
62178cb
@Amuhar
fix: group of events
3e3c1f6
@avsetsin
fix: events grouping
7b3c339
@Amuhar
fix: part of tests on holesky
c11e66a
@Amuhar
fix: remaining part of tests
b36b31e
@Amuhar
fix: remaining part of tests
fae472f
@Amuhar
fix: more tests
fc9c901
@mkurayan
test: update NOR version
6f89637
@mkurayan
test: fix dsm
02a2a9b
@mkurayan
test: update constants
fbbd772
@mkurayan
test: replace isTargetLimitActive to targetLimitMode
1cc02b3
@eddort
fix: deploy params
819c2ce
@Amuhar
Merge pull request #234 from lidofinance/fix/staking-router-v2-deploy
c7c719b 
fix: deploy params
@mkurayan
test: update limits
653b511
TheDZhon
TheDZhon reviewed Oct 22, 2024
View reviewed changes
Copy link
Contributor

@TheDZhon TheDZhon left a comment

Choose a reason for hiding this comment

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

👀

scripts/upgrade_sr_v2.py Outdated Show resolved Hide resolved
scripts/upgrade_sr_v2.py Outdated Show resolved Hide resolved
scripts/upgrade_sr_v2.py Outdated Show resolved Hide resolved
scripts/upgrade_sr_v2.py Outdated Show resolved Hide resolved
@bulbozaur
Copy link
Contributor

There are several skipped tests in test suite, pls check it
SKIPPED [1] tests/regression/test_pause_resume.py: Skipped
SKIPPED [1] tests/regression/test_sanity_checks.py: Skipped
SKIPPED [1] tests/regression/test_staking_module_happy_path.py: Skipped

CI has been failed.

@bulbozaur
Copy link
Contributor

bulbozaur commented Oct 29, 2024 •
edited
Loading

Screenshot 2024-10-29 at 13 51 40
Also has some failed after launch state tests

skhomuti and others added 17 commits October 29, 2024 20:07
@skhomuti
upload csm keys with batches
df20586 
fix rewards calculation with csm alive
@skhomuti
fix happy path test, add expected csm transfer event
b8b8470
@mkurayan
Merge pull request #312 from lidofinance/fix-csm-post-vote
5a3e0d0 
upload csm keys with batches
@skhomuti
fix unstable shares check in csm claim
8bb06ae
@iamnp
Merge pull request #315 from lidofinance/fix-csm-test
6d72479 
fix unstable shares check in csm claim
fix: not enough eth to make transaction
d587fc7
fix: revert gas price
63cc9fe
fix: eth check
b895c8d
@iamnp
Merge pull request #316 from lidofinance/fix/ensure-balance
b344d00 
Fix/ensure balance
fix: ensure balance (ii)
de875a7
@TheDZhon
fix: remove balance_check middleware side effects
678e83c
@TheDZhon
fix: add finally block
b9dec0b
@TheDZhon
chore: print out balance when can't set it up
50383fc
@TheDZhon
fix: sync the oracle simulation code + override
89b19f8
@TheDZhon
fix: update the negative rebase test
7c2b44c
@TheDZhon
fix: set initial balance for stranger
1d111db
@iamnp
Merge pull request #318 from lidofinance/fix/balance-check-fixture
ae58f35 
fix: tune-up tests after the vote #180
@iamnp iamnp merged commit ff929d4 into master Nov 18, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TheDZhon TheDZhon TheDZhon left review comments

@avsetsin avsetsin avsetsin left review comments

@kadmil kadmil kadmil left review comments

@Amuhar Amuhar Amuhar left review comments

@skhomuti skhomuti skhomuti left review comments

@iamnp iamnp iamnp approved these changes

@bulbozaur bulbozaur Awaiting requested review from bulbozaur

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.