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

Fix vmotion ping #31

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fix vmotion ping #31

wants to merge 2 commits into from

Conversation

jgc234
Copy link

@jgc234 jgc234 commented Sep 1, 2020

This is a simple proposal to fix the vmotion ping failure problem. The first few vmotion pings often fail on a cluster node that has been idle. If you run the HXTool.py script again, it works fine. Assuming it takes a moment to populate the ARP cache, the aggressive nature of the timeout and the short count on the ping may cause a packet or two to be dropped.. The parser that checks the ping response expects 0% packet loss, hence it fails the test. Rather than re-write the pingstatus() parser, it's easier to just run the test twice - once with a normal timeout and normal MTU to populate the ARP cache, then run the existing test. That being said, I haven't conclusively proven it is an ARP problem - esxcli network ip neighbor list doesn't show the vmotion interfaces or addresses for me (even in the default stack). The other potential solution is to just remove the excessively low timeout, but I assume there was a reason for this (maybe to prune out long WAN links?)

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