- 11 Dec, 2024 1 commit
-
-
Sebastian Stammler authored
-
- 10 Dec, 2024 10 commits
-
-
Matthew Slipper authored
Adds additional op-deployer scripts to support alternative fault proof implementations. This is a prerequisite to deleting the allocs files, since many tests need a fast dispute game and an alphabet VM.
-
blaine authored
* fix: clarity around how salts are made. * fix: removed named return value. * fix: semver lock updated. * fix: pr comments. * fix: adding natpsec to the new salt helper functions. fix: adding natpsec to the new salt helper functions. * fix: more details around why proxy salts include contract name. * Update packages/contracts-bedrock/src/L1/OPContractsManager.sol Co-authored-by:
Matt Solomon <matt@mattsolomon.dev> * Update packages/contracts-bedrock/src/L1/OPContractsManager.sol Co-authored-by:
Matt Solomon <matt@mattsolomon.dev> * fix: only having one compute salt function * fix: pre-pr ran * Update packages/contracts-bedrock/src/L1/OPContractsManager.sol Co-authored-by:
Matt Solomon <matt@mattsolomon.dev> * fix: fix pre-pr --------- Co-authored-by:
Matt Solomon <matt@mattsolomon.dev>
-
George Knee authored
* use pendingBlocks() for s.metr.RecordL2BlocksAdded * use pendingBlocks when channel is opened
-
zhiqiangxu authored
* enhance TestBatchReader * add more test * address comments * address comments * remove extra comment
-
zhiqiangxu authored
-
Dmitry authored
Co-authored-by:
Matthew Slipper <me@matthewslipper.com>
-
Matthew Slipper authored
When fuzz tests fail they write data to the test directory. This PR archives the fuzz data to make future debugging easier.
-
ControlCplusControlV authored
Co-authored-by:
Matthew Slipper <me@matthewslipper.com>
-
Matthew Slipper authored
The SSZ fuzz tests were not erroring when the `edOffset` was 508 and the `txOffset` was 514. This is because the transaction unmarshaler was reading the number of transactions in the payload as zero, but not checking to see if there was additional data left over. This should never be the case for a valid payload. To verify, I ran the fuzz suite for 5 minutes: ``` fuzz: elapsed: 5m1s, execs: 12910962 (42267/sec), new interesting: 9 (total: 41) ```
-
Matthew Slipper authored
* Revert "maint: bump forge version (#13301)" This reverts commit a2e7d852. * Reapply "maint: bump forge version (#13301)" This reverts commit 048f0742eaa338e54105ab23a919af8a22f13e0a. * Rebuild all on mise.toml * fix: verify mise dependencies if necessary --------- Co-authored-by:
Kelvin Fichter <kelvinfichter@gmail.com>
-
- 09 Dec, 2024 4 commits
-
-
zhiqiangxu authored
* add RegSP constant * address comments * fix natspec * update semver-lock
-
Axel Kingsley authored
* supervisor: L1 Processor * comments ; test fixes * Make L1 source separate from RPC Addr * fix test * Add atomic bool for singleton processor routine
-
George Knee authored
* add additional test case for computeSyncActions * fixup test * check for unsafe = safe edge case & replace oldestUnsafeBlock with nextSafeBlock * add test case for no progress and safe=unsafe * refine log * rename variable * harmonize log ordering and labels * tighten up test behaviour for expectedLogs * add test case: no progress + unsafe=safe + blocks in state and fix behaviour so we don't try to fetch unsafe blocks if there aren't any, even when there are blocks in state * typo
-
Francis Li authored
* Fix conductor test race condition * update * Address shell ci check * rerun CI for non-related issue
-
- 07 Dec, 2024 15 commits
-
-
I love OP authored
Co-authored-by:
Matthew Slipper <me@matthewslipper.com>
-
Inphi authored
-
Matthew Slipper authored
This reverts commit a2e7d852.
-
Matthew Slipper authored
* op-deployer: Support output files in bootstrap * lint
-
smartcontracts authored
Bumps versions being used for forge, cast, and anvil.
-
Matthew Slipper authored
* ci: Specify number of jobs in contract tests * reduce runs * decrease runs * don't rebuild on semver lock * remove -j
-
zhiqiangxu authored
* single source of truth for DerivationVersion * op-node/rollup: fix imports --------- Co-authored-by:
protolambda <proto@protolambda.com>
-
Maurelian authored
* feat: remove unused systemConfigs mapping from OPCM * fix semver lock
-
Dmitry authored
-
Matthew Slipper authored
-
dependabot[bot] authored
Bumps [github.com/holiman/uint256](https://github.com/holiman/uint256) from 1.3.1 to 1.3.2. - [Release notes](https://github.com/holiman/uint256/releases) - [Commits](https://github.com/holiman/uint256/compare/v1.3.1...v1.3.2) --- updated-dependencies: - dependency-name: github.com/holiman/uint256 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by:
dependabot[bot] <support@github.com> Co-authored-by:
dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
-
Matthew Slipper authored
* op-deployer: Bootstrap superchain command Fixes https://github.com/ethereum-optimism/optimism/issues/13265. * fix test
-
blaine authored
-
protolambda authored
* op-supervisor: readme draft * op-supervisor: fix review suggestions * op-supervisor: readme extension * op-supervisor: fix mermaid diagram syntax
-
Michael Amadi authored
* test utils * test utils * rm unncesessary vm.assume * rm unncesessary vm.assume * rm unncesessary vm.assume * fixes... * rm testutils
-
- 06 Dec, 2024 10 commits
-
-
smartcontracts authored
semver-lock justfile task didn't build by default which was confusing. Now it does.
-
Sebastian Stammler authored
With Holocene, batcher transaction ordering has to be strictly preserved, so trying to send a transaction candidate should just never timeout. Note that the txmgr will still bump fees to get a transaction for the same nonce submitted.
-
zhiqiangxu authored
* Don't repeat yourself * add a log for loadBlocksIntoState * op-batcher: fix log in batcher/driver.go * modify log --------- Co-authored-by:
protolambda <proto@protolambda.com> Co-authored-by:
Matthew Slipper <me@matthewslipper.com>
-
Dmitry authored
Fix formatting issue in "Production Releases" section
-
Matthew Slipper authored
-
George Knee authored
-
Matthew Slipper authored
-
Matthew Slipper authored
-
zhiqiangxu authored
-
George Knee authored
op-batcher: extract state pruning, block fetching and progress checking into a single pure function (#13060) * remove lastStoredBlock and lastL1Tip from BatchSubmitter state We can use the channelManager's state to infer lastStoredBlock. And lastL1Tip is actually unused. * change log line wording * fix typo * remove unecessary method * WIP first pass at computeSyncActions * computeSyncAction takes a ChannelStatuser interface also report fully inclusive range of blocks to load * add happy path test case * clearState is a pointer we can use nil value to signal no state clearing should be performed * add more test cases * add another test case * computeSyncActions only takes prevCurrentL1, not prevSyncStatus * add batcher restart case * safe chain reorg case * failed to make progress case * simplify log messages, print entire struct * add godoc * wire up computeSyncActions * cache prevCurrentL1 on BatchSubmitter * document stages * fix loadBlocksIntoState range interpretation * pass syncStatus, not pointer to syncStatus and add test case for no progress * check unsafe status before trying to get more blocks * do not panic on invalid block ranges return an error instead. This error is ultimated swallowed, matching existing behaviour. * test: add assetions and mock data about blockID passed to clearState * add readme section on max channel duration * add back unit tests for pruning methods * fix pruneBlocks behaviour when blockCursor pointed at block which is now pruned * rename waitForNodeSync to sequencerOutOfSync * Introduce SeqOutOfSyncError * move SyncActions code to a separate file * ChannelStatuser -> channelStatuser * SeqOutOfSyncError -> ErrSeqOutOfSync * move ctx to first position in fn signature * do not update cached prevCurrentL1 value if there is an ErrSeqOutOfSync * Always warn log when computeSyncActions returns an error * move sync actions test to separate file * computeSyncActions returns a bool, not an error There is only ever one kind of error returned * SyncActions -> syncActions * define local variables to aid readability * organise computeSyncActions and introduce startAfresh syncAction Add comments explaining logical flow: the checks get increasingly deep and we return early where possible. * undo changes to submodule * move test utils to sync_actions_test.go file * ensure pruneChannels clears currentChannel when appropriate * fix submodule" * don't try to get number of block if none exists * improve log * Update op-batcher/batcher/driver.go Co-authored-by:
Sebastian Stammler <seb@oplabs.co> * use struct for block range, not array * use startAfresh in one more place * add test case for multiple channels also set HeadL1 to more realistic values (generally ahead of currentL1 due to nonzero confirmation depth) * print value of *struct in Stringer * add test case when there are no blocks in state * Update op-batcher/batcher/sync_actions.go Co-authored-by:
Sebastian Stammler <seb@oplabs.co> * tighten up log messages and test descriptions --------- Co-authored-by:
Sebastian Stammler <seb@oplabs.co>
-