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

op-sepolia: Add task to switch back to singlethreaded cannon. #544

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

ajsutton
Copy link
Contributor

@ajsutton ajsutton commented Feb 5, 2025

Description

Add task to switch back to singlethreaded cannon.

Updates the set-game-implementation task to handle recent breaking changes in superchain-registry and the monorepo.

Updates the set-game-implementation task to handle recent breaking changes in superchain-registry and the monorepo.
@ajsutton ajsutton requested review from a team as code owners February 5, 2025 04:02
@ajsutton ajsutton requested a review from mbaxter February 5, 2025 04:02
import {PermissionedDisputeGame} from "@eth-optimism-bedrock/src/dispute/PermissionedDisputeGame.sol";
import {SystemConfig} from "@eth-optimism-bedrock/src/L1/SystemConfig.sol";

contract NestedSignFromJson is OriginalNestedSignFromJson {
Copy link
Member

Choose a reason for hiding this comment

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

Could we use the verification contracts instead, like for many Holocene upgrades?

Copy link
Contributor

Choose a reason for hiding this comment

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

Agreed with @sebastianst.
This is easier for us to review and also increase the consistency for the signers + reviewers.
I would suggest to use this as example:
https://github.com/ethereum-optimism/superchain-ops/tree/main/tasks/sep/027-holocene-system-config-upgrade-and-init-multi-chain

Copy link
Contributor

Choose a reason for hiding this comment

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

This one uses the new DisputeGameUpgrade verification contract.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Discussed more in discord - this was generated via the standard template for setting dispute games. The verification framework assumes there is permissionless and permissioned game types which isn't always the case so doesn't fit well with the template which can handle all chains. But for this case I've updated the script to use the verification framework.

@ajsutton ajsutton requested a review from Ethnical February 6, 2025 00:53
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.

4 participants