Closed freak12techno closed 2 months ago
The overall change introduces a new ValidatorActiveGenerator
to generate Prometheus metrics for active validators. It also standardizes logger component names across various fetchers, and refactors existing SingleValidatorInfoGenerator
by removing the isActiveGauge
metric. These changes enhance logging clarity, support metric generation for validators, and update associated unit tests.
File Path | Change Summary |
---|---|
pkg/app.go |
Added NewValidatorActiveGenerator call to NewApp function to create active validator generator. |
pkg/fetchers/consumer_info.go |
Updated logger component identifier from "validators_fetcher" to "consumer_info_fetcher". |
pkg/fetchers/consumer_validators.go |
Changed logger component from "validators_fetcher" to "consumer_validators_fetcher". |
pkg/fetchers/has_to_validate.go |
Modified logger component from "node_info_fetcher" to "validator_consumers_fetcher". |
pkg/fetchers/signing_info.go |
Updated logger component string from "signing_infos" to "signing_infos_fetcher". |
pkg/generators/validator_active.go |
Introduced ValidatorActiveGenerator to generate metrics for active validators. |
pkg/generators/validator_active_test.go |
Added unit tests for ValidatorActiveGenerator . |
pkg/generators/single_validator_info.go |
Removed isActiveGauge metric from SingleValidatorInfoGenerator . |
pkg/generators/single_validator_info_test.go |
Adjusted test assertions to account for the removal of isActiveGauge metric. |
In the code where metrics thrive,
Validator active now comes alive,
Logging clear, succinct, and bright,
Fetchers labeled just right.
Prometheus smiles with new zest,
As the changes pass the test.
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Attention: Patch coverage is 98.55072%
with 1 line
in your changes missing coverage. Please review.
Project coverage is 90.71%. Comparing base (
d6594f6
) to head (503714c
).
Files | Patch % | Lines |
---|---|---|
pkg/app.go | 0.00% | 1 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Summary by CodeRabbit
New Features
ValidatorActiveGenerator
to generate Prometheus metrics for active validators.Bug Fixes
ConsumerInfoFetcher
,ConsumerValidatorsFetcher
,ValidatorConsumersFetcher
,SigningInfoFetcher
).Improvements
isActiveGauge
metric fromSingleValidatorInfoGenerator
.isActiveGauge
metric.