Skip to content

Enhancement: Open Files Diagnostic Metadata#3036

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

Enhancement: Open Files Diagnostic Metadata#3036
ghost wants to merge 175 commits into
mainfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented May 2, 2026

Summary

This PR improves the system-level observability of the node by including the count of open file descriptors in the health diagnostics payload.

Changes

  1. File Descriptor Monitoring: Updated the endpoint to report the number of files currently opened by the node process.
  2. Leak Detection: Helps node operators identify potential resource leaks (e.g., unclosed database connections or log handles) before they hit the operating system's limits.
  3. Optional Dependency: Uses with a graceful fallback, maintaining compatibility with minimal server environments while providing rich data where available.

Closes #2295

Michael Sovereign added 30 commits April 12, 2026 16:30
MichaelSovereign added 20 commits May 2, 2026 07:43
…in Elya Service for better API consistency
…e for better trust score visualization in BCOS
…ng for future multi-wallet ownership support
…sure consistent visibility across node logs
…pts in Sophia Inspector for improved security auditing
…ng for improved device tracking consistency
…iagnostics for proactive storage management
…WAL mode and increased busy timeout for better concurrency
… to prevent errors from malformed client input
…o ensure data integrity during synchronization
…s for improved resource leakage monitoring
@ghost ghost requested a review from Scottcjn as a code owner May 2, 2026 07:07
@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
Copy link
Copy Markdown
Author

ghost commented May 2, 2026

Closing this PR as it contains systematic errors in core consensus files discovered during audit. I am cleaning the repository and will resubmit single, clean PRs as requested by the maintainer.

@ghost
Copy link
Copy Markdown
Author

ghost commented May 2, 2026

Cleanup

@ghost ghost closed this May 2, 2026
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.

0 participants