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

feat(supervisor/core): implemented task retrier#2386

Merged
sadiq1971 merged 10 commits intomainfrom
sa/feat/retry-on-subscription
Jul 10, 2025
Merged

feat(supervisor/core): implemented task retrier#2386
sadiq1971 merged 10 commits intomainfrom
sa/feat/retry-on-subscription

Conversation

@sadiq1971
Copy link
Collaborator

Closes #2353

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

codecov bot commented Jul 9, 2025

Codecov Report

Attention: Patch coverage is 25.27473% with 136 lines in your changes missing coverage. Please review.

Project coverage is 82.2%. Comparing base (0a1aaa6) to head (17129e2).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
crates/supervisor/core/src/syncnode/utils.rs 0.0% 41 Missing ⚠️
crates/supervisor/core/src/syncnode/node.rs 0.0% 37 Missing ⚠️
crates/supervisor/core/src/syncnode/task.rs 47.0% 27 Missing ⚠️
crates/supervisor/core/src/syncnode/client.rs 0.0% 20 Missing ⚠️
crates/supervisor/core/src/syncnode/error.rs 0.0% 7 Missing ⚠️
crates/supervisor/core/src/supervisor.rs 0.0% 4 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.

Copy link
Collaborator

@dhyaniarun1993 dhyaniarun1993 left a comment

Choose a reason for hiding this comment

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

Please fix the ci pipeline. Other than that look good to me.

@sadiq1971 sadiq1971 added this pull request to the merge queue Jul 10, 2025
Merged via the queue into main with commit 10b8d3b Jul 10, 2025
26 of 28 checks passed
@sadiq1971 sadiq1971 deleted the sa/feat/retry-on-subscription branch July 10, 2025 11:50
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Dec 10, 2025
Closes op-rs/kona#2353

---------

Co-authored-by: Arun Dhyani <dhyaniarun7@gmail.com>
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Jan 14, 2026
Closes #2353

---------

Co-authored-by: Arun Dhyani <dhyaniarun7@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

W-supervisor Workstream: supervisor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(supervisor/core): Handle managed node websocket connection error

2 participants