Skip to content

Moving to using a home grown ping engine.#739

Merged
i3149 merged 5 commits into
mainfrom
try-new-ping
Aug 23, 2024
Merged

Moving to using a home grown ping engine.#739
i3149 merged 5 commits into
mainfrom
try-new-ping

Conversation

@i3149
Copy link
Copy Markdown
Contributor

@i3149 i3149 commented Aug 23, 2024

Switching up to a new ping system because the other one was too buggy. This is mostly a drop in replacement but is a lot simpler.

Also bumping go version to 1.22.
Closes #738

@i3149 i3149 requested a review from jryburn August 23, 2024 05:06
Copy link
Copy Markdown

@jryburn jryburn left a comment

Choose a reason for hiding this comment

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

LGTM

@jryburn
Copy link
Copy Markdown

jryburn commented Aug 23, 2024

This is attempting to address issue #738

@i3149 i3149 merged commit 83a2bb7 into main Aug 23, 2024
@i3149 i3149 deleted the try-new-ping branch August 23, 2024 20:40
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.

Reopening packet loss on ping_only tests from issues 732 and 736

2 participants