Sepolia OPCM v2.0.0: op, soneium, ink #709
Draft
+826
−3
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
blmalone
changed the title
blmalone
force-pushed
the
bm/opcm-upgrade-13
branch
from
91eb264 to
aa1962a
Compare
blmalone
force-pushed
the
03-04-sepolia_opcm_v2.0.0_op_soneium_ink
branch
5 times, most recently
from
00361c9 to
b621b8c
Compare
blmalone
force-pushed
the
bm/opcm-upgrade-13
branch
from
e78dbde to
ccb45b9
Compare
blmalone
force-pushed
the
03-04-sepolia_opcm_v2.0.0_op_soneium_ink
branch
2 times, most recently
from
5a7284e to
bedc930
Compare
blmalone
commented
Mar 5, 2025
| string memory expectedErrors_1946 = | ||
| "SYSCON-20,PDDG-50,PDDG-DWETH-30,PDDG-DWETH-40,PDDG-ANCHORP-40,PDDG-120,PLDG-10"; | ||
| string memory expectedErrors_763373 = | ||
| "SYSCON-20,PDDG-50,PDDG-DWETH-30,PDDG-DWETH-40,PDDG-ANCHORP-40,PLDG-50,PLDG-DWETH-30,PLDG-DWETH-40,PLDG-ANCHORP-40"; |
There was a problem hiding this comment.
@maurelian can you check if these are ok too? This is for ink sepolia.
blmalone
force-pushed
the
03-04-sepolia_opcm_v2.0.0_op_soneium_ink
branch
4 times, most recently
from
02cca5b to
1a16781
Compare
This was referenced Mar 5, 2025
blmalone
force-pushed
the
03-04-sepolia_opcm_v2.0.0_op_soneium_ink
branch
2 times, most recently
from
b597832 to
3b9bddb
Compare
maurelian
reviewed
Mar 6, 2025
| @@ -0,0 +1,706 @@ | |||
| ## Understanding Task Calldata | |||
|
|
|||
| Multicall3DelegateCall calldata: | |||
There was a problem hiding this comment.
How will this be validated? Can signers just copy it and ctrl+f for this string in the tenderly sim?
We should also include the target address for Multicall3DelegateCall as that must not change.
I'm out of time now, but in the morning I'll generate a quick script to compute the calldata directly from a config to facilitate this review.
maurelian
reviewed
Mar 6, 2025
Comment on lines
11
to
13
| {absolutePrestate = "0x035ac388b5cb22acf52a2063cfde108d09b1888655d21f02f595f9c3ea6cbdcd", chainId = 11155420}, | ||
| {absolutePrestate = "0x035ac388b5cb22acf52a2063cfde108d09b1888655d21f02f595f9c3ea6cbdcd", chainId = 1946}, | ||
| {absolutePrestate = "0x035ac388b5cb22acf52a2063cfde108d09b1888655d21f02f595f9c3ea6cbdcd", chainId = 763373} |
There was a problem hiding this comment.
these will unfortunately need to change.
blmalone
force-pushed
the
bm/opcm-upgrade-13
branch
from
1772884 to
c4e14d3
Compare
blmalone
force-pushed
the
03-04-sepolia_opcm_v2.0.0_op_soneium_ink
branch
3 times, most recently
from
6ffbec3 to
b499854
Compare
blmalone
force-pushed
the
03-04-sepolia_opcm_v2.0.0_op_soneium_ink
branch
from
c4fbc57 to
ea592b4
Compare
blmalone
force-pushed
the
bm/opcm-upgrade-13
branch
from
8177326 to
34c1129
Compare
blmalone
force-pushed
the
03-04-sepolia_opcm_v2.0.0_op_soneium_ink
branch
from
ea592b4 to
f0d71d6
Compare
blmalone
force-pushed
the
bm/opcm-upgrade-13
branch
from
34c1129 to
99d5472
Compare
blmalone
force-pushed
the
03-04-sepolia_opcm_v2.0.0_op_soneium_ink
branch
from
f0d71d6 to
6cb2305
Compare
blmalone
force-pushed
the
bm/opcm-upgrade-13
branch
from
99d5472 to
6f93442
Compare
blmalone
force-pushed
the
03-04-sepolia_opcm_v2.0.0_op_soneium_ink
branch
from
705248d to
b10a724
Compare
blmalone
force-pushed
the
bm/opcm-upgrade-13
branch
from
6f93442 to
3ce45e2
Compare
blmalone
force-pushed
the
03-04-sepolia_opcm_v2.0.0_op_soneium_ink
branch
from
b10a724 to
822f4d4
Compare
blmalone
force-pushed
the
bm/opcm-upgrade-13
branch
from
3ce45e2 to
9943b6f
Compare
blmalone
force-pushed
the
03-04-sepolia_opcm_v2.0.0_op_soneium_ink
branch
from
822f4d4 to
a8da7f0
Compare
blmalone
force-pushed
the
bm/opcm-upgrade-13
branch
from
9943b6f to
def6e04
Compare
blmalone
force-pushed
the
03-04-sepolia_opcm_v2.0.0_op_soneium_ink
branch
from
a8da7f0 to
9d0bf43
Compare
blmalone
force-pushed
the
03-04-sepolia_opcm_v2.0.0_op_soneium_ink
branch
3 times, most recently
from
3024c09 to
528fa58
Compare
blmalone
force-pushed
the
bm/opcm-upgrade-13
branch
from
b040464 to
91287ff
Compare
blmalone
force-pushed
the
03-04-sepolia_opcm_v2.0.0_op_soneium_ink
branch
3 times, most recently
from
ddd342c to
b99c96d
Compare
blmalone
force-pushed
the
03-04-sepolia_opcm_v2.0.0_op_soneium_ink
branch
from
b99c96d to
0115e40
Compare
blmalone
force-pushed
the
03-04-sepolia_opcm_v2.0.0_op_soneium_ink
branch
from
0115e40 to
beecdce
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds the Sepolia task for OP Sepolia, Soneium and Ink. All of which share the same L1 proxy admin owner key:
0x1Eb2fFc903729a0F03966B917003800b145F56E2.You'll find the VALIDATION.md file in this PR which was generated by taking the state diff printed by our new superchain-ops code then cross referencing with the tenderly state diff.