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

Comments

chore (supervisor): build info in metrics#2774

Merged
itschaindev merged 6 commits intomainfrom
jk/app-metrics
Aug 27, 2025
Merged

chore (supervisor): build info in metrics#2774
itschaindev merged 6 commits intomainfrom
jk/app-metrics

Conversation

@itschaindev
Copy link
Collaborator

Closes #2760

The Overview now looks like:

Screenshot 2025-08-26 at 19 30 52

@itschaindev itschaindev self-assigned this Aug 26, 2025
@itschaindev itschaindev added K-chore Kind: chore M-metrics Meta: metric related W-supervisor Workstream: supervisor labels Aug 26, 2025
@codecov
Copy link

codecov bot commented Aug 26, 2025

Codecov Report

❌ Patch coverage is 0% with 24 lines in your changes missing coverage. Please review.
✅ Project coverage is 85.8%. Comparing base (1540103) to head (1a6513e).
⚠️ Report is 2 commits behind head on main.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
bin/supervisor/src/metrics/version.rs 0.0% 23 Missing ⚠️
bin/supervisor/src/cli.rs 0.0% 1 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.

@itschaindev itschaindev removed the K-chore Kind: chore label Aug 26, 2025
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.

Nice.

@itschaindev itschaindev enabled auto-merge August 27, 2025 11:28
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.

i love the number of threads metric, good idea!

@itschaindev itschaindev added this pull request to the merge queue Aug 27, 2025
Merged via the queue into main with commit 451ebcd Aug 27, 2025
34 of 36 checks passed
@itschaindev itschaindev deleted the jk/app-metrics branch August 27, 2025 12:23
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Dec 10, 2025
Closes op-rs/kona#2760 

The Overview now looks like:

<img width="1705" height="427" alt="Screenshot 2025-08-26 at 19 30 52"
src="https://github.com/user-attachments/assets/aad99ab1-2f4d-48b8-99e3-31e1e4b17522"
/>
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Jan 14, 2026
Closes #2760 

The Overview now looks like:

<img width="1705" height="427" alt="Screenshot 2025-08-26 at 19 30 52"
src="https://github.com/user-attachments/assets/aad99ab1-2f4d-48b8-99e3-31e1e4b17522"
/>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

M-metrics Meta: metric related W-supervisor Workstream: supervisor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

chore (supervisor): add app build metrics

3 participants