Add IPv6 support for AddDefaultRoute#802
Conversation
Signed-off-by: Y7n05h <[email protected]>
|
Is there an issue with a description of something that isn't working? AFAICT this function isn't used. A few other things...
|
Usually the default route for IPv6 is automatically configured by Router Advertisements. But sometimes it is necessary to configure it manually.
|
|
|
Closed due to lack of coffee. Apologies
The (current) cloud world we live in sadly. The PR makes sense as a result. Is this for your plugin? IIRC this isn't used by any of the CNI reference plugins.
This I'm having a harder time parsing... "expected" device? If the network tells you what to use it's best to use that and not what you expected. I get 8 default routes from my ISP (cheapest plan), some come and go as ISP needs require. It's safe for me not to expect any. |
|
Note: This function isn't used by the plugins, I can't guarantee it continues to exist. |
AddDefaultRoute assumes that the user is adding a default route for IPv4, not IPv6. This pr will fix it.