Skip to content

Conversation

@paulhauner
Copy link
Member

Issue Addressed

NA

Proposed Changes

Adds some metrics so we can track payload status responses from the EE. I think this will be useful for troubleshooting and alerting.

I also bumped the BecaonChain::per_slot_task to debug since it doesn't seem too noisy and would have helped us with some things we were debugging in the past.

Additional Info

NA

@paulhauner paulhauner added ready-for-review The code is ready for review v3.0.0 🐼 Required for the v3.0.0 release labels Aug 19, 2022
Copy link
Member

@michaelsproul michaelsproul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@michaelsproul michaelsproul added ready-for-merge This PR is ready to merge. and removed ready-for-review The code is ready for review labels Aug 19, 2022
@paulhauner
Copy link
Member Author

bors r+

bors bot pushed a commit that referenced this pull request Aug 19, 2022
## Issue Addressed

NA

## Proposed Changes

Adds some metrics so we can track payload status responses from the EE. I think this will be useful for troubleshooting and alerting.

I also bumped the `BecaonChain::per_slot_task` to `debug` since it doesn't seem too noisy and would have helped us with some things we were debugging in the past.

## Additional Info

NA
@bors bors bot changed the title Add metrics for EE PayloadStatus returns [Merged by Bors] - Add metrics for EE PayloadStatus returns Aug 19, 2022
@bors bors bot closed this Aug 19, 2022
Woodpile37 pushed a commit to Woodpile37/lighthouse that referenced this pull request Jan 6, 2024
## Issue Addressed

NA

## Proposed Changes

Adds some metrics so we can track payload status responses from the EE. I think this will be useful for troubleshooting and alerting.

I also bumped the `BecaonChain::per_slot_task` to `debug` since it doesn't seem too noisy and would have helped us with some things we were debugging in the past.

## Additional Info

NA
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-merge This PR is ready to merge. v3.0.0 🐼 Required for the v3.0.0 release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants