feat: update L2 genesis specs#20
feat: update L2 genesis specs#20agusduha wants to merge 1 commit intosc-feat/standard-l2-genesisfrom
Conversation
| | `L1_ERC_721_BRIDGE_ADDRESS` | `uint8(5)` | Sets the `L1ERC721Bridge` address | | ||
| | `L1_STANDARD_BRIDGE_ADDRESS` | `uint8(6)` | Sets the `L1StandardBridge` address | | ||
| | `REMOTE_CHAIN_ID` | `uint8(7)` | Sets the chain id of the base chain | | ||
| | Name | Value | Description | |
There was a problem hiding this comment.
Missing ADD_DEPENDENCY and REMOVE_DEPENDENCY
There was a problem hiding this comment.
These weren't part of this feature and also they have been removed so no problem
| | `GAS_LIMIT` | `uint8(2)` | `abi.encode(uint64 _gasLimit)` | Modifies the L2 gas limit | | ||
| | `UNSAFE_BLOCK_SIGNER` | `uint8(3)` | `abi.encode(address)` | Modifies the account that is authorized to progress the unsafe chain | | ||
| | `EIP_1559_PARAMS` | `uint8(4)` | `uint256(uint64(uint32(_denominator))) << 32 \| uint64(uint32(_elasticity))` | Modifies the EIP-1559 denominator and elasticity | | ||
| | Name | Value | Definition | Usage | |
There was a problem hiding this comment.
Missing FEE_VAULT_ADMIN which is added here defi-wonderland/optimism#274
There was a problem hiding this comment.
There will be a need for addition on the Initialization paragraph aswell probably
There was a problem hiding this comment.
Regarding the OptimismPortal2 on the upgrade the function signature is missing 1 param, check defi-wonderland/optimism#273
There was a problem hiding this comment.
Also on the same note, it mentions
to is Predeploys.ProxyAdmin
Should be changed to Predeploys.L2_PROXY_ADMIN
| | `L1_ERC_721_BRIDGE_ADDRESS` | `bytes32(uint256(keccak256("opstack.l1erc721bridgeaddress")) - 1)` | `abi.encode(address(L1ERC721BridgeProxy))` | | ||
| | `L1_STANDARD_BRIDGE_ADDRESS` | `bytes32(uint256(keccak256("opstack.l1standardbridgeaddress")) - 1)` | `abi.encode(address(L1StandardBridgeProxy))` | | ||
| | `REMOTE_CHAIN_ID` | `bytes32(uint256(keccak256("opstack.remotechainid")) - 1)` | Chain ID of the remote chain | | ||
| | Name | Value | Definition | |
There was a problem hiding this comment.
I dont quite follow where these constants come from?
| | `L1_STANDARD_BRIDGE_ADDRESS` | `bytes32(uint256(keccak256("opstack.l1standardbridgeaddress")) - 1)` | `abi.encode(address(L1StandardBridgeProxy))` | | ||
| | `REMOTE_CHAIN_ID` | `bytes32(uint256(keccak256("opstack.remotechainid")) - 1)` | Chain ID of the remote chain | | ||
| | Name | Value | Definition | | ||
| | ----------------------------------- | -------------------------------------------------------------------------- | ------------------------------------------------------------------------------------------------------------------ | |
There was a problem hiding this comment.
Looks like the constant values for the enum members in Types are the ones for their _SLOT counterpart in L1Block, maybe it's worth to make this distinction.
|
Outdated, follow on #23 |
3 participants
Closes OPT-647