[Windows] Remove HNSEndpoint only if infra container fails to create (#…

…2976)

For non-infra containers, there's no need to remove HNSEndpoint at
failure since it will be done when the Pod is removed.

Signed-off-by: Zhecheng Li <[email protected]>

pkg/agent/cniserver/interface_configuration_windows.go

@@ -140,7 +140,9 @@ func (ic *ifConfigurator) configureContainerLink(
 	containerIface, err := attachContainerLink(endpoint, containerID, containerNetNS, containerIFDev)
 	if err != nil {
 		klog.V(2).Infof("Failed to attach HNS Endpoint to the container, remove it.")
-		ic.removeHNSEndpoint(endpoint, containerID)
+		if isInfraContainer(containerID) {
+			ic.removeHNSEndpoint(endpoint, containerID)
+		}
 		return fmt.Errorf("failed to configure container IP: %v", err)
 	}
 
@@ -299,9 +301,7 @@ func attachContainerLink(ep *hcsshim.HNSEndpoint, containerID, sandbox, containe
 		if hcnEp == nil {
 			// Docker runtime
 			if err := hcsshim.HotAttachEndpoint(containerID, ep.Id); err != nil {
-				if isInfraContainer(sandbox) || hcsshim.ErrComputeSystemDoesNotExist != err {
-					return nil, err
-				}
+				return nil, err
 			}
 		} else {
 			// Containerd runtime