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

Ignore network read timeout errors #115

Merged
merged 2 commits into from
Jul 12, 2023
Merged

Ignore network read timeout errors #115

merged 2 commits into from
Jul 12, 2023

Conversation

SuperQ
Copy link
Owner

@SuperQ SuperQ commented Jul 12, 2023

Network timeout errors are handled by the pinger with an exponential backoff. We can ignore these errors.

Fixes: #114

Network timeout errors are handled by the pinger with an exponential
backoff. We can ignore these errors.

Fixes: #114

Signed-off-by: SuperQ <superq@gmail.com>
@SuperQ
Copy link
Owner Author

SuperQ commented Jul 12, 2023

👓 @hhromic @floatingstatic

@SuperQ
Copy link
Owner Author

SuperQ commented Jul 12, 2023

Tested this locally, mutes the error noise.

* [BUGFIX] Ignore network read timeout errors #115

Signed-off-by: SuperQ <superq@gmail.com>
Copy link

@hhromic hhromic left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for fixing this 👍 🚀

@SuperQ SuperQ merged commit c0848dc into master Jul 12, 2023
4 checks passed
@SuperQ SuperQ deleted the superq/read_timeout branch July 12, 2023 14:21
@@ -5,6 +5,10 @@
* [ENHANCEMENT]
* [BUGFIX]

## 0.7.1 / 2023-06-12
Copy link

Choose a reason for hiding this comment

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

Ah, one comment I just noticed.
The date of the changelog entry is wrong, should be 2023-07-12.

And thanks for releasing a fix so quickly!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New packet receive errors metric is spamming
2 participants