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

Aegir betanet upgrade task #581

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

Aegir betanet upgrade task #581

wants to merge 8 commits into from

Conversation

maurelian
Copy link
Contributor

@maurelian maurelian commented Feb 11, 2025

Adds a task for upgrading the Aegir betanet.

This build on #570 in order to minimize the diff in the submodule update.

@maurelian maurelian requested review from a team as code owners February 11, 2025 17:18
@maurelian maurelian requested a review from refcell February 11, 2025 17:18
@maurelian maurelian force-pushed the aegir-opcm-upgrade branch 2 times, most recently from 6d1550d to 17ec49e Compare February 11, 2025 19:35
@maurelian maurelian changed the base branch from main to update-base-contracts February 11, 2025 19:35
Base automatically changed from update-base-contracts to main February 11, 2025 19:54
Comment on lines +49 to +52
This upgrade
* Changes dispute game implementation of the `CANNON` and `PERMISSIONED_CANNON` game types to contain a `op-program` release for the Holocene hardfork, which contains
the Holocene fork implementation as well as a `ChainConfig` and `RollupConfig` for the L2 chain being upgraded.
* Upgrades `MIPS.sol` to support the `F_GETFD` syscall, required by the golang 1.22+ runtime.
Copy link
Contributor

Choose a reason for hiding this comment

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

this is out of date

// For the purpose of the Aegir betanet upgrade, we will skip this check, as it will
// require allowing a very large number of state accesses, across multiple chains.
// checkStateDiff(accesses);
console.log("All assertions passed!");
Copy link
Contributor

Choose a reason for hiding this comment

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

We should run the standard validator checks here ethereum-optimism/optimism#14325

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