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

Handle new IPv6 ping format; handle echoing tapbr0 interface #12

Open
wants to merge 2 commits into
base: pr/utils
Choose a base branch
from

Conversation

danpetry
Copy link

@danpetry danpetry commented Apr 9, 2019

This is two PRs in one really, separated into two commit messages. Let me know if it needs splitting...

  1. Adapt the ping command format according to the change made here
  2. Handle the case where the tapbr0 interface is also sending an echo back, when doing multicast testing

danpetry added 2 commits April 9, 2019 15:53
Update ping command for format introduced in PR 9523
Depending on the user configuration, the tapbr0 interface can also
echo. This leads to the final message from the pinging native instance including
the word "duplicate", and also means the pinging native instance receives
two times as many replies as sends.
This handles this without failing.
@danpetry danpetry changed the base branch from master to pr/utils April 9, 2019 14:55
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.

1 participant