[sepolia]: Incident Multisig Update #492
Conversation
🟡 Heimdall Review Status
|
…the EXISTING_OWNERS_LENGTH constant definition
leopoldjoy
changed the title
| OP_COMMIT=bfc4cce53463c0d79224f77c6aba65d7c3f0518b | ||
| BASE_CONTRACTS_COMMIT=dcd8c98aa881e0ae4ebf872e0d91692a7bf94000 | ||
|
|
||
| OWNER_SAFE=0x5dfEB066334B67355A15dc9b67317fD2a2e1f77f |
There was a problem hiding this comment.
Confirmed this is correct
| @@ -0,0 +1,10 @@ | |||
| OP_COMMIT=bfc4cce53463c0d79224f77c6aba65d7c3f0518b | |||
There was a problem hiding this comment.
Where is this commit coming from? Current version is 594bc933a38425f745b46399a3619bcdeb74965d
| @@ -0,0 +1,10 @@ | |||
| OP_COMMIT=bfc4cce53463c0d79224f77c6aba65d7c3f0518b | |||
| BASE_CONTRACTS_COMMIT=dcd8c98aa881e0ae4ebf872e0d91692a7bf94000 | |||
There was a problem hiding this comment.
Confirmed this is the latest on base/contracts
|
|
||
| OWNER_SAFE=0x5dfEB066334B67355A15dc9b67317fD2a2e1f77f | ||
|
|
||
| SENDER=0x0CF2F86C3338993ce10F74d6f4B095712c7efe26 |
There was a problem hiding this comment.
Confirmed this is a valid sender
|
|
||
| RECORD_STATE_DIFF=true | ||
|
|
||
| EXISTING_OWNERS_LENGTH=14 |
There was a problem hiding this comment.
I've done it this way before as well and BlockSec actually preferred to have 14 hardcoded in the script because it's easier to review
There was a problem hiding this comment.
Confirmed all addresses here are correct
| @@ -0,0 +1,50 @@ | |||
| # Update Sepolia Incident Multisig Signers | |||
|
|
|||
| Status: TODO[READY TO SIGN|DONE] | |||
There was a problem hiding this comment.
status should be ready to sign
| node --version | ||
| ``` | ||
|
|
||
| If you see a version output from the above command, you can move on. Otherwise, install node |
There was a problem hiding this comment.
Should specify that min version is v18.18
|
|
||
| Be sure to select the correct task from the list of available tasks to sign. | ||
|
|
||
| ### 4. Send signature to facilitator |
There was a problem hiding this comment.
Could also include a note about killing the signer tool with ctrl + c
There was a problem hiding this comment.
This looks good to me - however there is a new version of the signer tool that produces an updated format. I'd recommend re-generating this file!
3 participants
This PR adds the task for updating the Sepolia incident multi-sig to be in accordance with the latest members of the Base Chain Eng team.