Skip to content
This repository was archived by the owner on Jan 16, 2026. It is now read-only.

feat(supervisor): broadcast cross head update events to managed node#2289

Merged
dhyaniarun1993 merged 6 commits intomainfrom
sa/feat/broadcast-cross-head
Jun 26, 2025
Merged

feat(supervisor): broadcast cross head update events to managed node#2289
dhyaniarun1993 merged 6 commits intomainfrom
sa/feat/broadcast-cross-head

Conversation

@sadiq1971
Copy link
Collaborator

@sadiq1971 sadiq1971 commented Jun 25, 2025 •
edited
Loading

Closes #2214
Closes #2215
Closes #2255
Closes #2256

@sadiq1971 sadiq1971 self-assigned this Jun 25, 2025
@sadiq1971 sadiq1971 added the W-supervisor Workstream: supervisor label Jun 25, 2025
@codecov
Copy link

codecov bot commented Jun 25, 2025 •
edited
Loading

Codecov Report

Attention: Patch coverage is 83.81877% with 50 lines in your changes missing coverage. Please review.

Project coverage is 81.2%. Comparing base (2a957ae) to head (ad8c86b).
Report is 3 commits behind head on main.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
crates/supervisor/core/src/chain_processor/task.rs 89.8% 13 Missing ⚠️
crates/supervisor/core/src/syncnode/node.rs 0.0% 11 Missing ⚠️
crates/supervisor/core/src/supervisor.rs 0.0% 10 Missing ⚠️
...rates/supervisor/core/src/chain_processor/chain.rs 62.5% 6 Missing ⚠️
crates/supervisor/core/src/safety_checker/task.rs 81.4% 5 Missing ⚠️
crates/supervisor/storage/src/chaindb.rs 95.7% 5 Missing ⚠️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

dhyaniarun1993
dhyaniarun1993 reviewed Jun 26, 2025
View reviewed changes
emhane
emhane approved these changes Jun 26, 2025
View reviewed changes
Copy link
Contributor

@emhane emhane left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@emhane
Copy link
Contributor

emhane commented Jun 26, 2025

does this close #2214 and #2215 @sadiq1971 ?

@sadiq1971
Copy link
Collaborator Author

sadiq1971 commented Jun 26, 2025 •
edited
Loading

does this close #2214 and #2215 @sadiq1971 ?

Yes, and a couple of more. Added to PR description

@sadiq1971 sadiq1971 requested a review from dhyaniarun1993 June 26, 2025 09:42
@sadiq1971 sadiq1971 enabled auto-merge June 26, 2025 10:47
@dhyaniarun1993 dhyaniarun1993 disabled auto-merge June 26, 2025 11:06
@dhyaniarun1993
Copy link
Collaborator

dhyaniarun1993 commented Jun 26, 2025

Kurtosis e2e test failure is not relevant here. Merging the PR.

@dhyaniarun1993 dhyaniarun1993 merged commit a4259d8 into main Jun 26, 2025
22 of 28 checks passed
@dhyaniarun1993 dhyaniarun1993 deleted the sa/feat/broadcast-cross-head branch June 26, 2025 11:07
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Dec 10, 2025
@sadiq1971
feat(supervisor): broadcast cross head update events to managed node (o…
867e920 
…p-rs/kona#2289)

Closes op-rs/kona#2214 
Closes op-rs/kona#2215 
Closes op-rs/kona#2255 
Closes op-rs/kona#2256
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Jan 14, 2026
@sadiq1971
feat(supervisor): broadcast cross head update events to managed node (o…
57a6bc9 
…p-rs/kona#2289)

Closes #2214 
Closes #2215 
Closes #2255 
Closes #2256
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@itschaindev itschaindev Awaiting requested review from itschaindev

2 more reviewers

@dhyaniarun1993 dhyaniarun1993 dhyaniarun1993 approved these changes

@emhane emhane emhane approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@sadiq1971 sadiq1971

Labels

W-supervisor Workstream: supervisor

Projects

None yet

Milestone

No milestone

3 participants

@sadiq1971 @emhane @dhyaniarun1993