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

eth/029: upgrade and re-initialize Base's SystemConfig to v2.3.0 #484

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

Conversation

xenoliss
Copy link
Contributor

@xenoliss xenoliss commented Jan 25, 2025

Description

Upgrade Base's SystemConfig on Mainnet to v2.3.0.
Duplicate of #476 with slight modifications for mainnet nested multisig.

Additional context

Base specifics:

  1. Added checkSystemConfigUpgradeWithPreviousGasLimitOverride to script/verification/HoloceneSystemConfigUpgrade.s.sol to allow updating the gasLimit.
  2. Manually added a SystemConfigProxy.setEIP1559Params(250,2) call to the end of the tasks/eth/029-holocene-system-config-upgrade-and-init-base/input.json via just add-transaction tasks/eth/029-holocene-system-config-upgrade-and-init-base/input.json 0x73a79Fab69143498Ed3712e519A88a918e1f4072 'setEIP1559Params(uint32,uint32)' 250 2
  3. Manually added a SystemConfigProxy.transferOwnership(SystemConfigOwner call to the end of the tasks/eth/029-holocene-system-config-upgrade-and-init-base/input.json via just add-transaction tasks/eth/029-holocene-system-config-upgrade-and-init-base/input.json 0x73a79Fab69143498Ed3712e519A88a918e1f4072 'transferOwnership(address)' 0x14536667Cd30e52C0b458BaACcB9faDA7046E056

@xenoliss xenoliss marked this pull request as ready for review January 27, 2025 14:17
@xenoliss xenoliss requested review from a team as code owners January 27, 2025 14:17
@xenoliss xenoliss requested a review from refcell January 27, 2025 14:17
Copy link
Contributor

@blmalone blmalone left a comment

Choose a reason for hiding this comment

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

First pass, going to continue reviewing soon.

@blmalone blmalone requested a review from JosepBove January 27, 2025 20:51
@JosepBove
Copy link

I have reviewed the changes and it is ready to merge

JosepBove
JosepBove previously approved these changes Jan 28, 2025
blmalone
blmalone previously approved these changes Jan 29, 2025
Copy link
Contributor

@blmalone blmalone left a comment

Choose a reason for hiding this comment

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

Approving. I left one comment which seems to be a typo.

Lets also wait for a review from @JosepBove before merging.

@blmalone blmalone requested a review from JosepBove January 29, 2025 01:05
@JosepBove
Copy link

After Blaine's change it looks good.

JosepBove
JosepBove previously approved these changes Jan 29, 2025
@blmalone blmalone enabled auto-merge January 29, 2025 19:10
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.

3 participants