-
Notifications
You must be signed in to change notification settings - Fork 167
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
rename census broadcaster metrics #3055
Conversation
This commit renames the metrics related to Broadcaster to Gateway, following a team decision. More details can be found in the discussion here: [Team Discussion Link](.com/channels/423160867534929930/1051963444598943784/1210356864643109004).
fdc4362
to
06d1053
Compare
f67ab3a
to
7c50d41
Compare
@@ -306,8 +306,8 @@ func InitCensus(nodeType NodeType, version string) { | |||
census.mTicketValueSent = stats.Float64("ticket_value_sent", "TicketValueSent", "gwei") | |||
census.mTicketsSent = stats.Int64("tickets_sent", "TicketsSent", "tot") | |||
census.mPaymentCreateError = stats.Int64("payment_create_errors", "PaymentCreateError", "tot") | |||
census.mDeposit = stats.Float64("broadcaster_deposit", "Current remaining deposit for the broadcaster node", "gwei") | |||
census.mReserve = stats.Float64("broadcaster_reserve", "Current remaining reserve for the broadcaster node", "gwei") | |||
census.mDeposit = stats.Float64("gateway_deposit", "Current remaining deposit for the gateway node", "gwei") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@thomshutt @pwilczynskiclearcode @hjpotter92 Any thoughts if we're using these metrics anywhere (grafana, alerts, etc.) and they'd need to be updated?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep, we are - we'll want to make sure that Grafana is find/replaced in lockstep with this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@leszko, @thomshutt, I believed I had implemented it in a non-breaking way 🤔. What changes would you like me to make?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ahh, I see. So both metrics will be produced. That's fine.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3055 +/- ##
===================================================
+ Coverage 57.36267% 57.39517% +0.03250%
===================================================
Files 92 92
Lines 15728 15740 +12
===================================================
+ Hits 9022 9034 +12
Misses 6104 6104
Partials 602 602
Continue to review full report in Codecov by Sentry.
|
* refactor(census): rename Broadcaster metrics to Gateway This commit renames the metrics related to Broadcaster to Gateway, following a team decision. More details can be found in the discussion here: [Team Discussion Link](.com/channels/423160867534929930/1051963444598943784/1210356864643109004). * chore: update pending changelog
What does this pull request do? Explain your changes. (required)
This pull request renames the metrics related to Broadcaster to Gateway, following a team decision. More details can be found in the discussion here: Team Discussion Link.
Specific updates (required)
gateway_deposit
andgateway_reserve
metrics to census while keeping the old metrics for back-compatibility.How did you test each of these updates (required)
-gateway
flag and checked the metrics show up correctly:Does this pull request close any open issues?
AI-LIV-357
Checklist:
make
runs successfully./test.sh
passTestSubmitSegment_HttpPostError
server test does fails on my system but I don't think it is related to my changesTest Logs