Skip to content

v3.1: Add WFSM stake metric (backport of #9803)#10035

Merged
alexpyattaev merged 1 commit intov3.1from
mergify/bp/v3.1/pr-9803
Jan 14, 2026
Merged

v3.1: Add WFSM stake metric (backport of #9803)#10035
alexpyattaev merged 1 commit intov3.1from
mergify/bp/v3.1/pr-9803

Conversation

@mergify
Copy link

@mergify mergify bot commented Jan 14, 2026

Problem

The 2025-12-03 testnet restart failed because some nodes saw 80% of stake online and some did not.

Summary of Changes

Add a metric to make it easier to monitor WFSM status in future restarts.


This is an automatic backport of pull request #9803 done by Mergify.

* Add wfsm metric. Add trace logging for peers.

* Remove trace logging, since peers are already logged by gossip

* Remove wrong_shred_stake from wfsm_gossip metric. This will always be 0 and the associated code will be cleaned up in a future PR

(cherry picked from commit 14a5055)
@mergify mergify bot requested a review from a team as a code owner January 14, 2026 19:45
@gregcusack
Copy link

this metric would be nice to have for restarts. we had to get this through logs before. Will used this metric to help us debug testnet restart false start

@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.2%. Comparing base (433a685) to head (690bdf9).

Additional details and impacted files
@@            Coverage Diff            @@
##             v3.1   #10035     +/-   ##
=========================================
- Coverage    83.2%    83.2%   -0.1%     
=========================================
  Files         865      865             
  Lines      376152   376156      +4     
=========================================
- Hits       313162   313138     -24     
- Misses      62990    63018     +28     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@gregcusack gregcusack self-requested a review January 14, 2026 20:40
Copy link

@gregcusack gregcusack left a comment

Choose a reason for hiding this comment

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

lgtm!

@alexpyattaev alexpyattaev merged commit a8c20c0 into v3.1 Jan 14, 2026
44 checks passed
@alexpyattaev alexpyattaev deleted the mergify/bp/v3.1/pr-9803 branch January 14, 2026 21:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants