Skip to content

Add metrics about face value and max float#3593

Merged
leszko merged 2 commits intomasterfrom
rafal/add-log-about-max-float-to-check-reserve
Jun 4, 2025
Merged

Add metrics about face value and max float#3593
leszko merged 2 commits intomasterfrom
rafal/add-log-about-max-float-to-check-reserve

Conversation

@leszko
Copy link
Copy Markdown
Contributor

@leszko leszko commented May 27, 2025

No description provided.

@github-actions github-actions bot added the go Pull requests that update Go code label May 27, 2025
@codecov
Copy link
Copy Markdown

codecov bot commented May 27, 2025

Codecov Report

Attention: Patch coverage is 53.33333% with 14 lines in your changes missing coverage. Please review.

Project coverage is 30.79303%. Comparing base (6a8b5e0) to head (7007b72).
Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
monitor/census.go 61.53846% 10 Missing ⚠️
pm/recipient.go 0.00000% 3 Missing and 1 partial ⚠️
Additional details and impacted files

Impacted file tree graph

@@                 Coverage Diff                 @@
##              master       #3593         +/-   ##
===================================================
+ Coverage   30.77611%   30.79303%   +0.01692%     
===================================================
  Files            153         153                 
  Lines          45870       45900         +30     
===================================================
+ Hits           14117       14134         +17     
- Misses         30930       30942         +12     
- Partials         823         824          +1     
Files with missing lines Coverage Δ
pm/recipient.go 88.41202% <0.00000%> (-1.54431%) ⬇️
monitor/census.go 62.23044% <61.53846%> (-0.01126%) ⬇️

... and 1 file with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6a8b5e0...7007b72. Read the comment docs.

Files with missing lines Coverage Δ
pm/recipient.go 88.41202% <0.00000%> (-1.54431%) ⬇️
monitor/census.go 62.23044% <61.53846%> (-0.01126%) ⬇️

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@leszko leszko assigned mjh1 and unassigned mjh1 Jun 3, 2025
@leszko leszko requested a review from mjh1 June 3, 2025 13:04
census.mDeposit = stats.Float64("gateway_deposit", "Current remaining deposit for the gateway node", "gwei")
census.mReserve = stats.Float64("gateway_reserve", "Current remaining reserve for the gateway node", "gwei")
census.mMaxFloat = stats.Float64("gateway_max_float", "Last maximum float for the gateway node", "gwei")
census.mTicketFaceValue = stats.Float64("ticket_face_value", "Last ticket face value for the gateway node", "gwei")
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.

I don't know much about maxFloat and ticketFace but I hope this is up to date - https://github.com/livepeer/go-livepeer/blob/master/doc/redeemer.md

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I think that is up to date but its for running go-livepeer as a -redeemer. The downside to this is that the redeemer then becomes the central failure point because the Orchestrator then depends on it to get maxFloat.

I have a PR removing the dependency on my fork of go-livepeer so the redeemer only accepts tickets sent from the Orchestrator if interested.

Copy link
Copy Markdown
Collaborator

@ad-astra-video ad-astra-video left a comment

Choose a reason for hiding this comment

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

I did not run locally but changes look OK to me.

Curious, why the wei2gwei conversion? If converting why not convert to ETH?

@leszko
Copy link
Copy Markdown
Contributor Author

leszko commented Jun 4, 2025

I did not run locally but changes look OK to me.

Curious, why the wei2gwei conversion? If converting why not convert to ETH?

It's just for the sake of consistency. We have other metrics (deposit, reserve) in gwei as well.

@leszko leszko merged commit 4f47484 into master Jun 4, 2025
13 checks passed
@leszko leszko deleted the rafal/add-log-about-max-float-to-check-reserve branch June 4, 2025 07:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

go Pull requests that update Go code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants