Addresing the comments

Azure · Sep 21, 2024 · c6520ed · c6520ed
1 parent 7e79d33
commit c6520ed
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 6 deletions.
diff --git a/.pipelines/pipeline.yaml b/.pipelines/pipeline.yaml
@@ -338,7 +338,6 @@ stages:
         clusterType: overlay-byocni-up
         clusterName: "statelesswin"
         vmSize: Standard_B2ms
-        k8sVersion: "1.30"
         dependsOn: "containerize"
 
     # AKS Swift E2E tests
@@ -460,6 +459,10 @@ stages:
                 name: azure_overlay_e2e
                 clusterName: "azovere2e"
                 region: $(REGION_AKS_CLUSTER_TEST)
+              azure_overlay_stateless_e2e:
+                name: azure_overlay_stateless_e2e
+                clusterName: "statelesswin"
+                region: $(REGION_AKS_CLUSTER_TEST)
               aks_swift_e2e:
                 name: aks_swift_e2e
                 clusterName: "swifte2e"

diff --git a/...ngletenancy/azure-cni-overlay-stateless/azure-cni-overlay-stateless-e2e-job-template.yaml b/...ngletenancy/azure-cni-overlay-stateless/azure-cni-overlay-stateless-e2e-job-template.yaml
@@ -69,7 +69,7 @@ stages:
       name: $(BUILD_POOL_NAME_DEFAULT)
     jobs:
       - job: ${{ parameters.name }}_windows
-        displayName: Azure CNI Overlay Test Suite | Windows - (${{ parameters.name }})
+        displayName: Azure Stateless CNI Overlay Test Suite | Windows - (${{ parameters.name }})
         pool:
           name: $(BUILD_POOL_NAME_DEFAULT)
           demands:

diff --git a/...gletenancy/azure-cni-overlay-stateless/azure-cni-overlay-stateless-e2e-step-template.yaml b/...gletenancy/azure-cni-overlay-stateless/azure-cni-overlay-stateless-e2e-step-template.yaml
@@ -19,7 +19,7 @@ steps:
     inputs:
       kubectlVersion: latest
 
-  - task: AzureCLI@1
+  - task: AzureCLI@2
     inputs:
       azureSubscription: $(BUILD_VALIDATIONS_SERVICE_CONNECTION)
       scriptLocation: "inlineScript"
@@ -45,7 +45,7 @@ steps:
       displayName: "Windows v4Overlay ControlPlane Scale Tests"
       retryCountOnTaskFailure: 2
 
-    - task: AzureCLI@1
+    - task: AzureCLI@2
       inputs:
         azureSubscription: $(BUILD_VALIDATIONS_SERVICE_CONNECTION)
         scriptLocation: "inlineScript"
@@ -61,7 +61,7 @@ steps:
           done
       displayName: "Restart Nodes"
 
-    - task: AzureCLI@1
+    - task: AzureCLI@2
       inputs:
         azureSubscription: $(BUILD_VALIDATIONS_SERVICE_CONNECTION)
         scriptLocation: "inlineScript"

diff --git a/cni/network/network.go b/cni/network/network.go
@@ -1163,7 +1163,6 @@ func (plugin *NetPlugin) Delete(args *cniSkel.CmdArgs) error {
 			logger.Error("Failed to query endpoint",
 				zap.String("endpoint", endpointID),
 				zap.Error(err))
-
 			logger.Error("Release ip by ContainerID (endpoint not found)",
 				zap.String("containerID", args.ContainerID))
 			sendEvent(plugin, fmt.Sprintf("Release ip by ContainerID (endpoint not found):%v", args.ContainerID))