Return nil when HNS endpoint ID is empty during deletion

[Copilot]

Pods stuck in delete loop when HNS endpoint never created during failed ADD calls in Stateless CNI Windows SwiftV1.

Change

Modified deleteEndpointImpl in network/endpoint_windows.go to return nil instead of error when ep.HnsId is empty:

if ep.HnsId == "" {
    logger.Info("No HNS id found. Skip endpoint deletion", zap.Any("nicType", ep.NICType), zap.String("containerId", ep.ContainerID))
    return nil  // Previously returned error
}

This matches existing behavior in both deleteEndpointImplHnsV1 and deleteEndpointImplHnsV2 where missing endpoints return nil rather than blocking deletion. Log level changed from Error to Info since empty HNS ID is now treated as expected state.

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Return non-nil in case there is no HNS id found for an endpoint.</issue_title>
<issue_description>What happened:
POd will stuck in a delete loop when hns ifor endpoint never created in a failed ADD call.

The issue is related to this line:

azure-container-networking/network/endpoint_windows.go

Line 534 in 6c232c8

return fmt.Errorf("No HNS id found. Skip endpoint deletion for nicType %v, containerID %s", ep.NICType, ep.ContainerID) //nolint

This should return nil instead of error.
What you expected to happen:
The endpoint should be able to be deleted if HNS id never created for the pod

How to reproduce it:
HAve add call that failed to created HNS endpoint in Stateless CNI Windows SwiftV1

Orchestrator and Version (e.g. Kubernetes, Docker):

Operating System (Linux/Windows):

Kernel (e.g. uanme -a for Linux or $(Get-ItemProperty -Path "C:\windows\system32\hal.dll").VersionInfo.FileVersion for Windows):

Anything else we need to know?:
[Miscellaneous information that will assist in solving the issue.]
</issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes Return non-nil in case there is no HNS id found for an endpoint. #4104

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[Copilot]

Pull Request Overview

This PR resolves an issue where pods get stuck in a delete loop when HNS endpoints are never created during failed ADD calls in Stateless CNI Windows SwiftV1. The fix changes the error handling to treat missing HNS endpoint IDs as a valid state rather than a blocking error.

• Changes deleteEndpointImpl to return nil instead of an error when HnsId is empty
• Downgrades log level from Error to Info since missing HNS ID is now expected
• Aligns behavior with existing deleteEndpointImplHnsV1 and deleteEndpointImplHnsV2 implementations

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[behzad-mir]

Remove the changes form go.sum.
Only need to make changes on endpoint_windows.go

[paulyufan2]

/azp run Azure Container Networking PR

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).