search

Azure / azure-container-networking

Azure Container Networking Solutions for Linux and Windows Containers
MIT License
376 stars 240 forks source link

fix: [WIN-NPM] HNS Remote Endpoint Fix #3033

Closed rejain456 closed 4 days ago

rejain456 commented 1 week ago

Reason for Change:

As we were booting up npm agent on windows, we saw ~4K endpoints on a node with 16 pods, when there should be a 1:1 endpoint to pod ratio.

Issue Fixed:

Changed logic: When booting up the data plane, instead of getting all remote endpoints, we only get local endpoints which will be the endpoints for the pods in the node. Testing: Took a node with 11 non-system pods on it. Ran windows NPM with new image and saw total of 12 endpoints getting reset vs ~4K that were getting reset originally.

Requirements:

Notes:

huntergregory commented 4 days ago

no need to run azp NPM pipelines for this PR since it modifies a windows-only file, and we don't have windows tests currently

rejain456 commented 4 days ago

/azp run Azure Container Networking PR

azure-pipelines[bot] commented 4 days ago
Azure Pipelines successfully started running 1 pipeline(s).