Skip to content
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

Incorrect Metric Mapping for load_state_for_block_production #5102

Closed
skisel-bt opened this issue Jan 21, 2024 · 0 comments
Closed

Incorrect Metric Mapping for load_state_for_block_production #5102

skisel-bt opened this issue Jan 21, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@skisel-bt
Copy link
Contributor

skisel-bt commented Jan 21, 2024

Description

load_state_for_block_production blocking method is incorrectly mapped to "produce_partial_beacon_block" metric bucket.

Version

"unstable"

Present Behaviour

Currently, when the load_state_for_block_production method is invoked, it is being recorded under the "produce_partial_beacon_block" metrics. This misalignment in mapping leads to inaccurate metrics reporting for produce_partial_beacon_block method.

Expected Behaviour

The correct behavior would be for the load_state_for_block_production method to be mapped to its own dedicated metric bucket.

Steps to resolve

During a single block production a metric "produce_partial_beacon_block" is invoked twice.

skisel-bt added a commit to skisel-bt/lighthouse that referenced this issue Jan 21, 2024
@chong-he chong-he added the bug Something isn't working label Jan 25, 2024
mergify bot pushed a commit that referenced this issue Jan 30, 2024
* #5102 Fix load_state_for_block_production metric mapping
danielramirezch pushed a commit to danielramirezch/lighthouse that referenced this issue Feb 14, 2024
* sigp#5102 Fix load_state_for_block_production metric mapping
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants