Skip to content

op-interop-mon#15902

Merged
axelKingsley merged 20 commits intodevelopfrom
new-op-interop-mon
Jun 11, 2025
Merged

op-interop-mon#15902
axelKingsley merged 20 commits intodevelopfrom
new-op-interop-mon

Conversation

@axelKingsley
Copy link
Copy Markdown
Contributor

@axelKingsley axelKingsley commented May 13, 2025

See README.md for implementation specific details.

New Service for monitoring Executing Messages during their lifecycle.

  • Finders walk for new Blocks and extract Executing Messages. They will also track Finalized blocks for expiry tracking
  • Maintainer handles routing, and single-point metric evaluation/submission
  • Updaters watch a set of Executing Messages pointed at their given chain. They update job statues regularly

#16225

Fast Follow Tasks:

  • More unit tests
  • Potentially updated Ring Buffer (use std)
  • Expiry based on block finality (ongoing)
  • Integrations with Kurtosis/Devstack/Infra to come

@codecov
Copy link
Copy Markdown

codecov bot commented May 13, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 0.00%. Comparing base (962c3f7) to head (f6f055d).
Report is 7 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff              @@
##           develop   #15902       +/-   ##
============================================
- Coverage    82.49%        0   -82.50%     
============================================
  Files          161        0      -161     
  Lines         8866        0     -8866     
============================================
- Hits          7314        0     -7314     
+ Misses        1412        0     -1412     
+ Partials       140        0      -140     
Flag Coverage Δ
cannon-go-tests-64 ?
contracts-bedrock-tests ?

Flags with carried forward coverage won't be shown. Click here to find out more.

see 161 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

geoknee added 6 commits June 11, 2025 09:25
* seenBlocks init moved to constructor (so we don't need to Start the component in unit tests)
* local processBlocks helper reintroduced for easier unit testing
* sentinel error for no contiguous blocks added, walkback handled by parent function
* next block set properly when walkback completes
* minor logging improvements
covering
* start / stop
* processBlock
* walkback with reorg example
geoknee added a commit to geoknee/optimism-package that referenced this pull request Jun 11, 2025
@axelKingsley axelKingsley marked this pull request as ready for review June 11, 2025 16:49
@axelKingsley axelKingsley requested review from a team as code owners June 11, 2025 16:49
@axelKingsley axelKingsley enabled auto-merge June 11, 2025 16:56
@axelKingsley axelKingsley added this pull request to the merge queue Jun 11, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jun 11, 2025
@geoknee geoknee added this pull request to the merge queue Jun 11, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jun 11, 2025
@axelKingsley axelKingsley added this pull request to the merge queue Jun 11, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jun 11, 2025
@axelKingsley axelKingsley added this pull request to the merge queue Jun 11, 2025
Merged via the queue into develop with commit 5a5e515 Jun 11, 2025
59 checks passed
@axelKingsley axelKingsley deleted the new-op-interop-mon branch June 11, 2025 20:22
sigma added a commit to ethpandaops/optimism-package that referenced this pull request Jun 12, 2025
See ethereum-optimism/optimism#15902

I have manually tested this with the downstream kurtosis-devnet from
optimism monorepo.

---------

Signed-off-by: Yann Hodique <yann@oplabs.co>
Co-authored-by: Yann Hodique <yann@oplabs.co>
iquidus pushed a commit to Layr-Labs/optimism that referenced this pull request Jul 24, 2025
* op-interop-mon

* renames

* Add Updaters

* Build fixes and remove dripper references

* Don't Stop on Start

* Use Enqueue pattern on Finder

* basic fixes

* Runtime improvements

* Fix broken ExecutingMessage extraction

* Track Jobs by pointer ; Job Thread Safety ; Metric Collection

* Metric Collection ; Terminal State Transition Tracking ; General Improvements

* Refactor Job Management ; Address Comments

* use sources.EthClient and manual Block-Walking

* refactor finder

* seenBlocks init moved to constructor (so we don't need to Start the component in unit tests)
* local processBlocks helper reintroduced for easier unit testing
* sentinel error for no contiguous blocks added, walkback handled by parent function
* next block set properly when walkback completes
* minor logging improvements

* add unit tests for finder

covering
* start / stop
* processBlock
* walkback with reorg example

* add arch diagram to readme

* remove unused field

* spell

* update block buffer

---------

Co-authored-by: geoknee <georgeknee@googlemail.com>
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.

2 participants