-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
ibc: metrics #7441
ibc: metrics #7441
Conversation
Codecov Report
@@ Coverage Diff @@
## master #7441 +/- ##
==========================================
- Coverage 58.02% 57.71% -0.31%
==========================================
Files 306 439 +133
Lines 19484 27547 +8063
==========================================
+ Hits 11305 15900 +4595
- Misses 6971 10091 +3120
- Partials 1208 1556 +348 |
[]string{"ibc", types.ModuleName, "receive"}, | ||
1, | ||
[]metrics.Label{ | ||
telemetry.NewLabel("chain_id", ctx.ChainID()), |
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.
why the chain ID? this would never change
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.
👍 although it would be useful to retrieve the chain_id
from the counterparty chain? Would that be better?
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.
yea that makes a lot more sense
Co-authored-by: colin axnér <[email protected]>
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.
🚀
…into fedekunze/ibc-metrics
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.
🚀
Description
Adds metric for the IBC module. For a full list see here.
It also updates some logs that had wrong format
Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.
docs/
) or specification (x/<module>/spec/
)godoc
comments.Unreleased
section inCHANGELOG.md
Files changed
in the Github PR explorerCodecov Report
in the comment section below once CI passes