Conversation
|
I'm not sure the CI failure is related to my changes. The three most recent pull requests all failed on the same test, and one of them is only a documentation change. |
CtrlZvi
force-pushed
the
1159-ndp-proxy
branch
2 times, most recently
from
b3dc86f to
b026e38
Compare
drivers/bridge/bridge_store.go
Outdated
| ncfg.BridgeName = nMap["BridgeName"].(string) | ||
| ncfg.EnableIPv6 = nMap["EnableIPv6"].(bool) | ||
| ncfg.EnableIPMasquerade = nMap["EnableIPMasquerade"].(bool) | ||
| ncfg.NDPProxyInterface = nMap["NDPProxyInterface"].(string) |
There was a problem hiding this comment.
Given this new field is not there for existing bridge network in store, you need to first check if the entry exists in the map before converting to string, or it will panic.
There was a problem hiding this comment.
Good catch. I will take care of that tonight and update the PR.
- Proxying is enabled via the network label com.docker.network.bridge.ndp_proxy_interface=<interface> fixes moby#1159 Signed-off-by: Zvi "CtrlZvi" Effron <viz+GitHub@flippedperspective.com>
| if config.NDPProxyInterface != "" && config.EnableIPv6 { | ||
| link, err := d.nlh.LinkByName(config.NDPProxyInterface) | ||
| if err != nil { | ||
| return err |
There was a problem hiding this comment.
Please format and return a more expressive error, otherwise caller will receive a simple link not found, which does not help much.
There was a problem hiding this comment.
I'm not familiar with go;
Is this what you need:
return fmt.Errorf("Link not found: %v", err)
There was a problem hiding this comment.
Simply return fmt.Errorf("could not find link for ndp proxy interface %q: %v", config.NDPProxyInterface, err)
|
ping @CtrlZvi |
1 similar comment
|
ping @CtrlZvi |
5 participants
fixes #1159
Signed-off-by: Zvi "CtrlZvi" Effron viz+GitHub@flippedperspective.com