Fix scripts and issues

base-org · Dec 4, 2024 · b9b6752 · b9b6752
1 parent 60ec57f
commit b9b6752
Show file tree

Hide file tree

Showing 4 changed files with 17 additions and 11 deletions.
diff --git a/mainnet/2024-11-18-increase-max-gas-limit/.env b/mainnet/2024-11-18-increase-max-gas-limit/.env
@@ -2,9 +2,8 @@ OPTIMISM_REPO=https://github.com/ethereum-optimism/optimism.git
 OPTIMISM_VERSION=op-contracts/v1.5.0
 OPTIMISM_CONTRACT_PATCH=patch/max-gas-limit.patch
 
-OP_MULTISIG=0x2501c477D0A35545a387Aa4A3EEe4292A9a8B3F0
-CB_MULTISIG=0x6e1DFd5C1E22A4677663A81D24C6BA03561ef0f6
-
+OP_MULTISIG=0x9BA6e03D8B90dE867373Db8cF1A58d2F7F006b3A
+CB_MULTISIG=0x9855054731540A48b28990B63DcF4f33d8AE46A1
 SAFE_ADDRESS=0x7bb41c3008b3f03fe483b28b8db90e19cf07595c
 PROXY_ADMIN_ADDRESS=0x0475cBCAebd9CE8AfA5025828d5b98DFb67E059E
 SYSTEM_CONFIG_ADDRESS=0x73a79Fab69143498Ed3712e519A88a918e1f4072

diff --git a/mainnet/2024-11-18-increase-max-gas-limit/Makefile b/mainnet/2024-11-18-increase-max-gas-limit/Makefile
@@ -21,8 +21,7 @@ checkout-op-commit:
 
 .PHONY: deploy
 deploy:
-	$(GOPATH)/bin/eip712sign --ledger --hd-paths "m/44'/60'/$(LEDGER_ACCOUNT)'/0/0" -- \
-	forge script --rpc-url $(L1_RPC_URL) DeploySystemConfig --sig "sign(address)"
+	forge script --rpc-url $(L1_RPC_URL) DeploySystemConfig --ledger --verify --mnemonic-derivation-paths "m/44'/60'/$(LEDGER_ACCOUNT)'/0/0"
 
 .PHONY: sign-cb
 sign-cb:

diff --git a/mainnet/2024-11-18-increase-max-gas-limit/README.md b/mainnet/2024-11-18-increase-max-gas-limit/README.md
@@ -1,4 +1,4 @@
-## [READY TO SIGN] Upgrade to new system config with 400m max gas limit
+## [DRAFT] Upgrade to new system config with 400m max gas limit
 
 Base is continuing to scale and we need higher block gas limit to support the increased demand.
 
@@ -15,14 +15,21 @@ make deps
 make deploy
 ```
 
-4. sign (both base & op team):
+4. sign (op team):
 ```
 cd mainnet/2024-11-18-increase-max-gas-limit
 make deps # skip if already installed
-make sign
+make sign-op
 ```
 
-5. upgrade
+5. sign (cb team):
+```
+cd mainnet/2024-11-18-increase-max-gas-limit
+make deps # skip if already installed
+make sign-cb
+```
+
+6. upgrade
 ```
 cd mainnet/2024-11-18-increase-max-gas-limit
 make run-upgrade

diff --git a/mainnet/2024-11-18-increase-max-gas-limit/script/UpgradeSystemConfig.s.sol b/mainnet/2024-11-18-increase-max-gas-limit/script/UpgradeSystemConfig.s.sol
@@ -5,7 +5,8 @@ import {Vm} from "forge-std/Vm.sol";
 import {IMulticall3} from "forge-std/interfaces/IMulticall3.sol";
 
 import {SystemConfig} from "@eth-optimism-bedrock/src/L1/SystemConfig.sol";
-import {MultisigBuilder, Simulation} from "@base-contracts/script/universal/MultisigBuilder.sol";
+import {Simulation} from "@base-contracts/script/universal/MultisigBuilder.sol";
+import "@base-contracts/script/universal/NestedMultisigBuilder.sol";
 
 interface IProxyAdmin {
     function upgrade(address _proxy, address _implementation) external;
@@ -15,7 +16,7 @@ interface IProxy {
     function implementation() external view returns (address);
 }
 
-contract UpgradeSystemConfig is MultisigBuilder {
+contract UpgradeSystemConfig is NestedMultisigBuilder {
     address internal SAFE_ADDRESS = vm.envAddress("SAFE_ADDRESS");
     address internal PROXY_ADMIN_ADDRESS = vm.envAddress("PROXY_ADMIN_ADDRESS");
     address internal SYSTEM_CONFIG_ADDRESS = vm.envAddress("SYSTEM_CONFIG_ADDRESS");