Skip to content

Commit 304db52

Browse files
committed
fix: update comments for dual NIC support and clarify ARP proxy function
1 parent 3ddc19e commit 304db52

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

cni/network/network_linux.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -125,7 +125,7 @@ func getNATInfo(_ *cni.NetworkConfig, _ interface{}, _ bool) (natInfo []policy.N
125125

126126
func platformInit(cniConfig *cni.NetworkConfig) {}
127127

128-
// isDualNicFeatureSupported returns if the dual nic feature is supported. Currently it's only supported for windows hnsv2 path
128+
// isDualNicFeatureSupported returns true as the dual nic feature is supported on Linux.
129129
func (plugin *NetPlugin) isDualNicFeatureSupported(netNs string) bool {
130130
return true
131131
}

network/transparent_vlan_endpointclient_linux.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -399,7 +399,7 @@ func (client *TransparentVlanEndpointClient) PopulateVnet(epInfo *EndpointInfo)
399399
return nil
400400
}
401401

402-
// Set ARP proxy on the vlan interface to respond to ARP requests for the gateway IP
402+
// Set ARP proxy on the specified interface to respond to ARP requests for the gateway IP
403403
func (client *TransparentVlanEndpointClient) setArpProxy(ifName string) error {
404404
cmd := fmt.Sprintf("echo 1 > /proc/sys/net/ipv4/conf/%v/proxy_arp", ifName)
405405
_, err := client.plClient.ExecuteRawCommand(cmd)

0 commit comments

Comments
 (0)