change apipa endpoint gateway back to .1 (#3653)

ZetaoZhuang · web-flow · commit 28331b40adb9 · 2025-05-14T23:02:20.000Z
* change apipa endpoint gateway back to .1

* lint fix

* add comment
diff --git a/cns/hnsclient/hnsclient_windows.go b/cns/hnsclient/hnsclient_windows.go
@@ -507,7 +507,6 @@ func configureHostNCApipaEndpoint(
 	}
 
 	networkContainerApipaIP := localIPConfiguration.IPSubnet.IPAddress
-	hostApipaIP := localIPConfiguration.GatewayIPAddress
 	protocolList := []string{protocolICMPv4, protocolTCP, protocolUDP}
 
 	endpointPolicies, err := configureAclSettingHostNCApipaEndpoint(
@@ -526,8 +525,9 @@ func configureHostNCApipaEndpoint(
 		endpoint.Policies = append(endpoint.Policies, endpointPolicy)
 	}
 
+	// keep Apipa Endpoint gw as 169.254.128.1 to make sure NC to host connectivity work for both Linux and Windows containers
 	hcnRoute := hcn.Route{
-		NextHop:           hostApipaIP,
+		NextHop:           hnsLoopbackAdapterIPAddress,
 		DestinationPrefix: "0.0.0.0/0",
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -507,7 +507,6 @@ func configureHostNCApipaEndpoint(`
`507`	`507`	`}`
`508`	`508`
`509`	`509`	`networkContainerApipaIP := localIPConfiguration.IPSubnet.IPAddress`
`510`		`- hostApipaIP := localIPConfiguration.GatewayIPAddress`
`511`	`510`	`protocolList := []string{protocolICMPv4, protocolTCP, protocolUDP}`
`512`	`511`
`513`	`512`	`endpointPolicies, err := configureAclSettingHostNCApipaEndpoint(`
`@@ -526,8 +525,9 @@ func configureHostNCApipaEndpoint(`
`526`	`525`	`endpoint.Policies = append(endpoint.Policies, endpointPolicy)`
`527`	`526`	`}`
`528`	`527`
	`528`	`+ // keep Apipa Endpoint gw as 169.254.128.1 to make sure NC to host connectivity work for both Linux and Windows containers`
`529`	`529`	`hcnRoute := hcn.Route{`
`530`		`- NextHop: hostApipaIP,`
	`530`	`+ NextHop: hnsLoopbackAdapterIPAddress,`
`531`	`531`	`DestinationPrefix: "0.0.0.0/0",`
`532`	`532`	`}`
`533`	`533`