feat(metrics): migrate sei-cosmos module packages to OpenTelemetry (PLT-414)#3512
feat(metrics): migrate sei-cosmos module packages to OpenTelemetry (PLT-414)#3512amir-deris wants to merge 3 commits into
Conversation
PR SummaryLow Risk Overview New per-package Slashing adds an OTel No consensus or business-logic behavior changes—observability only, with a planned follow-up to drop legacy telemetry. Reviewed by Cursor Bugbot for commit 97bc34d. 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.04% 58.22% -0.82%
==========================================
Files 2199 2138 -61
Lines 182096 174001 -8095
==========================================
- Hits 107513 101311 -6202
+ Misses 64933 63694 -1239
+ Partials 9650 8996 -654
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).