-
Notifications
You must be signed in to change notification settings - Fork 19
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
Multiple interfaces for gateway host #223
Comments
Err. So there is an undocumented network_interface param that can be used to select the interface. Do you think it needs to be further automated? I can certainly put in code to make a guess as to the active interface, but here's why my use-case makes it difficult:
Let me know which of these 3 options you would like me to add. Option 2 is the easiest, and should work with fixed IP addresses. |
Yeah, it's fine for now. Just a thought for the future beyond the demo. There is so much configuration for multi-robot environments, I like to automate absolutely everything humanly possible, so long as I can guarantee it introduces no errors in esoteric use cases. |
I've finally decided on the fix for this problem. If we create a wrapper around
All we have to do is parse this output, figure out within which line the hub lies and pick out the interface. We'll need some special machinery to make sure that the loopback is not being used, as |
👍 sounds worth trying. |
I'm actually happy with the parameter configuration now. I usually set a roslaunch arg to trigger off an environment variable. We can leave it at that to save having to insert and maintain tricky code. Leave this as a todo? |
ok. changed milestone. |
With two or more interfaces and no parameterised definition, we get:
I wonder if we can intelligently guess which interface by looking up ROS_HOSTNAME, ROS_IP and lastly ROS_MASTER_URI to choose the right one without having to set a parameter (still have that there as a last resort though).
Another way might be to work out an existing route through to a known hub to determine what interface it uses.
Note that if this is not set we get this issue later:
which is expected I guess.
The text was updated successfully, but these errors were encountered: