Add Supechain Pause Execution (SPE) Tool #401
Conversation
Ethnical
force-pushed
the
ethni/add-pause-automation
branch
from
2a5dfc0 to
024467e
Compare
…mism/superchain-ops into ethni/add-pause-automation
pause-automation/README.md
Outdated
|
|
||
|
|
||
| https://github.com/user-attachments/assets/e09f0a5c-5d2a-4df3-b7cd-3249d3a8b1e4 | ||
| ## Automation Pause (AP) |
There was a problem hiding this comment.
What do you think about calling this Superchain Pause or Superchain Pause Execution? This way it's clear this is different from the prior automated pause project that would pause in response to some monitoring condition
There was a problem hiding this comment.
Agreed, I have chosen Superchain Pause Execution (SPE) introduce in this commit: ea22725
pause-automation/just.just
Outdated
| set dotenv-load | ||
| export rpcUrl := env_var('ETH_RPC_URL') | ||
|
|
||
| # By default we choose sepolia network (if there is a mistake we not pausing mainnet). |
There was a problem hiding this comment.
We also don't want to pause sepolia by mistake, can we do no default?
There was a problem hiding this comment.
Yes actually the comment is not accurate and the file just.just need to be removed, nice catch!
pause-automation/just.just
Outdated
| echo "op is not installed, please install op (one-password cli) using homebrew. Using brew install --cask 1password-cli" | ||
| exit |
There was a problem hiding this comment.
Perhaps in this repo, or in the runbook, we should have full instructions for setting this up and authenticating with op (this can come later, not a blocker for this PR)
There was a problem hiding this comment.
agreed have created a subtask for runbook improvements
There was a problem hiding this comment.
what's the difference between this file and pause-automation/just.just? perhaps we should just have one pause.sh script that we call from the justfile? might be easier than writing bash in the justfile
There was a problem hiding this comment.
Correct, this is undesired file pushed, I have remove this one in the commit 6b11
There was a problem hiding this comment.
we probably could sign with cast directly (cast wallet sign) in the bash script, which is nice because it removes the need setting up a new forge project with new deps in this folder
There was a problem hiding this comment.
Clearly, this was part of my first choice however, since the DPM is not yet deployed on Sepolia/Mainnet this harder to operate for the testing.
The script allows us to mock and tweak the behavior using the cheatcodes for now.
The idea would be to wait deployment of DPM to get rid of the �forge script imo.
Co-authored-by: Matt Solomon <[email protected]>
Co-authored-by: Matt Solomon <[email protected]>
…mism/superchain-ops into ethni/add-pause-automation
Ethnical
changed the title
Description
This PR add a new tool called Superchain Pause Execution (SPE) for pausing the Superchain Config.
This make the process way simpler and fast than before in case of an emergency.
I created a folder called
superchain-pause-executionthat contain:README.md( a quick readme to explain how to use and what issuperchain-pause-execution)superchain-pause-execution.just( a just file that contain the main logic for pausing the superchain).SignPauseMessage.s.sol( a solidity script that use forge to broadcast and sign the transaction to pause the superchain).Video Demo
Pause.Automation.-.Demo.mp4
More infos for reviewers:
I will demo this during the Huddle Thursday.
The code is not finished yet so there is still TODO in the code.
As the
DeputyPauseModuleis not deployed on mainnet/sepolia