-
Notifications
You must be signed in to change notification settings - Fork 75
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
Verify logs severity #3636
Comments
Similar situation with log: |
Thank you for the report! We will take a closer look at the log messages you mentioned. For the |
Another log that probably can be decreased to warning:
or with slightly different message
|
Version:
keep-client version v2.0.0-m3 (revision cdddee4b0)
Keep-client sometimes logs several error logs, which does not seem to have much impact.
Could you verify if they have proper severity? Many of them occur just once, which looks to me like there is a retry mechanism. If so, then maybe it'd be worth logging an error message only when all retries fail. This is just my guess.
A few examples
ERROR keep-contract-WalletCoordinator contract/WalletCoordinator.go:3499 subscription failed to pull events: [error retrieving past HeartbeatRequestSubmitted events: [Internal server error]]
context
ERROR keep-contract-WalletRegistry contract/WalletRegistry.go:7401 subscription failed to pull events: [error retrieving past DkgResultSubmitted events: [Internal server error]]
context
ERROR keep-contract-WalletCoordinator contract/WalletCoordinator.go:3550 subscription to event HeartbeatRequestSubmitted failed with error: [read tcp IP_ADDRESS:38014->IP_ADDRESS:443: i/o timeout]; resubscription attempt will be performed
and
ERROR keep-contract-WalletRegistry contract/WalletRegistry.go:7455 subscription to event DkgResultSubmitted failed with error: [read tcp IP_ADDRESS:33310->IP_ADDRESS:443: i/o timeout]; resubscription attempt will be performed
context
These are all error logs since 2023-05-15
These can be skipped, because they are related to the poor blockstream server issue:
I replaced IP addresses and all 0xaaaa strings from the logs. If you need them, let me know.
The text was updated successfully, but these errors were encountered: