1. 26 Nov, 2024 10 commits
  2. 25 Nov, 2024 3 commits
    • Mael Regnery's avatar
      fix: use eth_getBlockReceipts for Infura RPC kind (#12969) · 816b75a8
      Mael Regnery authored
      * fix: use eth_getBlockReceipts for Infura RPC kind
      
      * fix infura test
      816b75a8
    • Matthew Slipper's avatar
      ci: Convert all Go tests into a single job (#13046) · 418f0cca
      Matthew Slipper authored
      * ci: Convert all Go tests into a single job
      
      Takes all the go-e2e and go-test jobs and combines them into one single super-job. This has the following benefits:
      
      1. We can centralize all config for Go tests in one place and simplify our CI pipeline.
      2. It leaves parallelism up to the Go runtime, which is more efficient.
      4. It makes our test reporting more accurate.
      
      As part of this PR, I also:
      
      1. Created a new resource class just for Go tests. It runs on the large Latitude runner, but limits the number of parallel go test jobs to 3 to prevent resource starvation.
      2. Enabled test retries on `gotestsum`. This should reduce the number of flakes we see from the Go tests by retrying them up to 2 times in the event that they fail. Failures are still captured by the test reporting, so we can address them.
      
      * code review updates
      418f0cca
    • Michael Amadi's avatar
      fixes (#13052) · 7719c853
      Michael Amadi authored
      7719c853
  3. 23 Nov, 2024 8 commits
  4. 22 Nov, 2024 11 commits
    • smartcontracts's avatar
      fix(ci): contracts test names check failures (#13034) · d139cc98
      smartcontracts authored
      lint-forge-tests-check-no-build was causing issues in CI because
      it was running as part of contracts-bedrock-checks which depends
      on contracts-bedrock-build. contracts-bedrock-build builds the
      contract files EXCEPT for tests whenever contracts-bedrock has
      changed. This means that contracts-bedrock-build would not build
      any test files and therefore lint-forge-tests-check-no-build would
      not find any test files to check and the job would do nothing.
      However, when contracts-bedrock is NOT changed, the build job gets
      the full set of artifacts INCLUDING test artifacts from the cache.
      This would trigger the full check to actually be run and it would
      properly fail.
      
      This PR moves that particular check into contracts-bedrock-tests
      so that it's guaranteed to have all of the compiled contracts.
      d139cc98
    • zhiqiangxu's avatar
      f9a38de2
    • Matthew Slipper's avatar
      op-service: Remove txmgr queue timing test (#13032) · ae24f01d
      Matthew Slipper authored
      The tests in the txmgr queue were asserting that calls complete within a certain time window. This was causing flakes, particularly when the CI runner was under load. Generally we should avoid tests that use fixed durations as a way of asserting that code functions correctly.
      ae24f01d
    • Sam Stokes's avatar
      op-node: log mgasps across block building/inserting lifecycle (#12907) · 1c36df3e
      Sam Stokes authored
      * op-node: log mgasps across block processing lifecycle
      
      * op-node: add 'import_time' field to block processing log
      
      * op-node: make log message more descriptive
      
      * op-node: log legacy codepath for InsertUnsafePayload
      
      * op-node: include final ForkchoiceUpdate in block insertion time
      
      * op-node: remove unused BuildTime field from events
      
      * op-node: encapsulate log logic in TryUpdateEngineEvent methods
      
      * op-node: linter fix
      
      * op-node: add comment and adjust sync log wording
      
      * op-node: fix BlockHash, BlockNumber in log
      
      * op-node: seq add BuildStarted to PayloadProcessEvent
      
      * op-node: refactor getBlockProcessingMetrics, protect against divide-by-zero
      1c36df3e
    • Matthew Slipper's avatar
      Revert "improve Legacy Contract tests (#13022)" (#13031) · 99810e0b
      Matthew Slipper authored
      This reverts commit 8f5c20ee.
      99810e0b
    • zhiqiangxu's avatar
      op-batcher: fix a deadlock (#13014) · 5aa14b37
      zhiqiangxu authored
      * fix a deadlock
      
      * add comment
      5aa14b37
    • clabby's avatar
      e52030db
    • Michael Amadi's avatar
      improve Legacy Contract tests (#13022) · 8f5c20ee
      Michael Amadi authored
      * improve DeployerWhitelist test
      
      * fix L1BlockNumber fallback test
      
      * fixes
      8f5c20ee
    • AgusDuha's avatar
      ccb544d0
    • AgusDuha's avatar
      34046982
    • mbaxter's avatar
      cannon: Add more sync tests (#12949) · cc2715c3
      mbaxter authored
      * cannon: Port go map tests
      
      * cannon: Port pool_test.go
      
      * cannon: Port a few more mutex tests
      
      * cannon: Port waitgroup_test.go
      
      * cannon: Port oncefunc_test.go (in progress)
      
      * cannon: Port atomic_test.go (in progress)
      
      * cannon: Port value_test.go (in progress)
      
      * cannon: Fix atomic tests by using a test mock
      
      * cannon: Fix test fail behavior
      
      * cannon: Move test util to a shared module
      
      * cannon: Use common testutil throughout
      
      * cannon: Fix failing tests
      
      * cannon: Add sanity check test program for test runner utils
      
      * cannon: Add more util tests, fix step counts
      
      * cannon: Rename test util method
      
      * cannon: Fix panic, error handling in testutil
      
      * cannon: Dedupe test running code
      
      * cannon: Simplify testutil interface
      
      * cannon: Mark mt tests as slow
      
      * cannon: Cut debugging code from test
      
      * cannon: Validate gc complete output
      
      * cannon: Synchronize access to bool values
      cc2715c3
  5. 21 Nov, 2024 7 commits
  6. 20 Nov, 2024 1 commit
    • Matthew Slipper's avatar
      op-deployer: Safety and validation improvements (#12993) · b05ec5af
      Matthew Slipper authored
      - Enables support for deploying tagged versions against new chains, but behind a huge warning that requires user input to bypass. Since our tagged release versions do not contain all implementations, using op-deployer in this way will deploy contracts that haven't been governance approved. Since this workflow is useful for development but bad for prod, I've added support for it but users have to bypass a large warning that describes the risks.
      - Validates the hashes of tagged version artifacts after downloading them. This prevents users from downloading tampered versions of the artifacts from GCS.
      b05ec5af