feat(metrics): migrate sei-cosmos module packages to OpenTelemetry (PLT-414)#3512
Conversation
PR SummaryLow Risk Overview New Slashing keeper adds an OTel Reviewed by Cursor Bugbot for commit 1e237ac. Bugbot is set up for automated code reviews on this repo. Configure here. |
|
The latest Buf updates on your PR. Results from workflow Buf / buf (pull_request).
|
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #3512 +/- ##
==========================================
- Coverage 59.05% 58.17% -0.89%
==========================================
Files 2205 2138 -67
Lines 182317 173502 -8815
==========================================
- Hits 107672 100939 -6733
+ Misses 64945 63587 -1358
+ Partials 9700 8976 -724
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
bdchatham
left a comment
There was a problem hiding this comment.
Two focused follow-ups from a cross-review pass. (Both non-blocking on their own — the larger naming-collision discussion on the per-package begin_blocker_duration / end_blocker_duration instruments is separate.)
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 97bc34d. Configure here.

Summary
Migrates the following
sei-cosmospackages from legacytelemetry/go-metricsto OpenTelemetry, using the dual-emit pattern (OTel instrument recorded first, old call retained withTODO(PLT-414)for removal once verified):types/module—module_total_mid_block_duration,module_mid_block_duration(per-module label)x/evidence—begin_blocker_durationx/capability—begin_blocker_durationx/crisis—end_blocker_duration,init_genesis_unmarshal_durationx/distribution—begin_blocker_durationx/gov—end_blocker_durationx/slashing—begin_blocker_duration; keeper:validator_slashed(withtype+validatorattributes)x/staking—begin_blocker_duration,end_blocker_durationEach package gets a new
metrics.godefining its OTel meter and instruments, following the pattern established inx/upgrade(PLT-353).