Skip to content

Enhancement: WebSocket Observability Improvements#2990

Closed
ghost wants to merge 130 commits into
mainfrom
unknown repository
Closed

Enhancement: WebSocket Observability Improvements#2990
ghost wants to merge 130 commits into
mainfrom
unknown repository

Conversation

@ghost

@ghost ghost commented May 2, 2026

Copy link
Copy Markdown

Summary

This PR improves the monitoring capabilities of the WebSocket feed manager.

Changes

  1. Error Metrics: Added an counter to the internal metrics dictionary. This allows operators to track the volume of failed broadcasts or connection errors over time.
  2. Enhanced Diagnostics: Provides more granular data when querying the server's metrics for health monitoring.

Closes #2295

Michael Sovereign added 30 commits April 12, 2026 16:30
MichaelSovereign added 19 commits May 2, 2026 07:18
… to prevent database locks from interrupting attestation flow
@ghost ghost requested a review from Scottcjn as a code owner May 2, 2026 06:28
@github-actions github-actions Bot added BCOS-L1 Beacon Certified Open Source tier BCOS-L1 (required for non-doc PRs) BCOS-L2 Beacon Certified Open Source tier BCOS-L2 (required for non-doc PRs) consensus Consensus/RIP-200 related node Node server related tests Test suite changes size/XL PR: 500+ lines labels May 2, 2026
@ghost

ghost commented May 2, 2026

Copy link
Copy Markdown
Author

Cleanup

@ghost ghost closed this May 2, 2026

@BossChaos BossChaos left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

PR #2990 Review - WebSocket Observability Improvements

Quality Analysis - Observability

This PR improves WebSocket observability.

Key improvements:

  1. Logging: Enhanced WebSocket logging
  2. Metrics: WebSocket performance metrics

Recommendation: Merge - observability improvement.

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BCOS-L1 Beacon Certified Open Source tier BCOS-L1 (required for non-doc PRs) BCOS-L2 Beacon Certified Open Source tier BCOS-L2 (required for non-doc PRs) consensus Consensus/RIP-200 related node Node server related size/XL PR: 500+ lines tests Test suite changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant