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.

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

[mainnet] Q1 Mainnet Pause Test #132

Merged
merged 3 commits into from
Mar 6, 2024
Merged

Conversation

nadir-akhtar-coinbase
Copy link
Contributor

Prepares a planned pause with the incident template

Copy link
Contributor

@anikaraghu anikaraghu left a comment

Choose a reason for hiding this comment

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

LGTM!

@@ -0,0 +1,5 @@
OP_COMMIT=10b06fb49861053999a89533d846ee5c2ccb33e1
Copy link
Contributor

Choose a reason for hiding this comment

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

this commit should be fine (looks like it's from November)? just curious how we decided on this commit?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Great question -- in short, the OptimismPortal contract has changed a lot since we deployed to mainnet. The Semver we're currently on is 1.7.0, and the interface changed when Optimism implemented their SuperchainConfig to store their pause instead. Hence, we need to use a much older commit.

Comment on lines +4 to +5
OPTIMISM_PORTAL_PROXY=0x49048044D57e1C92A77f79988d21Fa8fAF74E97e
GUARDIAN=0x14536667Cd30e52C0b458BaACcB9faDA7046E056
Copy link
Contributor

Choose a reason for hiding this comment

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

@nadir-akhtar-coinbase nadir-akhtar-coinbase changed the title Q1 Mainnet Pause Test [mainnet] Q1 Mainnet Pause Test Mar 6, 2024
@nadir-akhtar-coinbase nadir-akhtar-coinbase merged commit f81b887 into main Mar 6, 2024
4 checks passed
@nadir-akhtar-coinbase nadir-akhtar-coinbase deleted the nadir/q1-mainnet-pause branch March 6, 2024 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants