Skip to content

Downgrade light client errors #7300

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

Merged
merged 1 commit into from
Apr 10, 2025

Conversation

eserilev
Copy link
Member

@eserilev eserilev commented Apr 9, 2025

Issue Addressed

Downgrade light client errors to debug

Error messages are alarming and usually indicate somethings wrong with the beacon node. The Light Client service is supposed to minimally impact users, and most will not care if the light client server is erroring. Furthermore, the only errors we've seen in the wild are during hard forks, for the first few epochs before the fork finalizes.

@eserilev eserilev added ready-for-review The code is ready for review UX-and-logs light-client v7.0.0 New release c. Q1 2025 labels Apr 9, 2025
@eserilev eserilev changed the title downgrade errors Downgrade light client errors Apr 9, 2025
@eserilev eserilev mentioned this pull request Apr 9, 2025
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.

Agree this is a good temporary fix before v7.0.0 until we can repro the fork failures.

@michaelsproul michaelsproul added ready-for-merge This PR is ready to merge. and removed ready-for-review The code is ready for review labels Apr 10, 2025
mergify bot added a commit that referenced this pull request Apr 10, 2025
@mergify mergify bot merged commit aed562a into sigp:release-v7.0.0 Apr 10, 2025
31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
light-client ready-for-merge This PR is ready to merge. UX-and-logs v7.0.0 New release c. Q1 2025
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants