ci: Add exit code capture for PR pipeline (#3125)

* ci: Add exit code capture for PR pipeline

* ci: ignore errors from cilium status

Author: jpayne3506

.pipelines/singletenancy/aks-swift/aks-swift-e2e.steps.yaml

@@ -68,6 +68,7 @@ steps:
       workingDirectory: $(ACN_DIR)
       addSpnToEnvironment: true
       inlineScript: |
+        set -e
         cd test/integration/load
 
         # Scale Cluster Up/Down to confirm functioning CNS

.pipelines/singletenancy/aks-swift/e2e-step-template.yaml

@@ -71,6 +71,7 @@ steps:
       scriptType: "bash"
       addSpnToEnvironment: true
       inlineScript: |
+        set -e
         cd test/integration/load
 
         # Scale Cluster Up/Down to confirm functioning CNS

.pipelines/singletenancy/aks/aks-e2e.steps.yaml

@@ -70,6 +70,7 @@ steps:
     displayName: "Restart Nodes"
 
   - script: |
+      set -e
       kubectl get pods -A -o wide
 
       echo "Deploying test pods"

.pipelines/singletenancy/aks/e2e-step-template.yaml

@@ -64,6 +64,7 @@ steps:
         done
     displayName: "Restart Nodes"
   - script: |
+      set -e
       kubectl get pods -A -o wide
       echo "Deploying test pods"
       cd test/integration/load

.pipelines/singletenancy/azure-cni-overlay/azure-cni-overlay-e2e-step-template.yaml

@@ -64,6 +64,7 @@ steps:
         scriptType: "bash"
         addSpnToEnvironment: true
         inlineScript: |
+          set -e
           cd test/integration/load
 
           # Scale Cluster Up/Down to confirm functioning CNS
@@ -129,6 +130,7 @@ steps:
         scriptType: "bash"
         addSpnToEnvironment: true
         inlineScript: |
+          set -e
           cd test/integration/load
 
           # Scale Cluster Up/Down to confirm functioning CNS

.pipelines/singletenancy/azure-cni-overlay/azure-cni-overlay-e2e.steps.yaml

@@ -68,6 +68,7 @@ steps:
         workingDirectory: $(ACN_DIR)
         addSpnToEnvironment: true
         inlineScript: |
+          set -e
           cd test/integration/load
 
           # Scale Cluster Up/Down to confirm functioning CNS
@@ -138,6 +139,7 @@ steps:
         workingDirectory: $(ACN_DIR)
         addSpnToEnvironment: true
         inlineScript: |
+          set -e
           cd test/integration/load
 
           # Scale Cluster Up/Down to confirm functioning CNS

.pipelines/singletenancy/cilium-dualstack-overlay/cilium-dualstackoverlay-e2e-step-template.yaml

@@ -92,6 +92,7 @@ steps:
       scriptType: "bash"
       addSpnToEnvironment: true
       inlineScript: |
+        set -e
         cd test/integration/load
 
         # Scale Cluster Up/Down to confirm functioning CNS
@@ -106,6 +107,7 @@ steps:
     retryCountOnTaskFailure: 3
 
   - script: |
+      set -e
       echo "Run Cilium Connectivity Tests"
       cilium status
       cilium connectivity test --connect-timeout 4s --request-timeout 30s --test '!pod-to-pod-encryption,!node-to-node-encryption,!no-unexpected-packet-drops' --force-deploy

.pipelines/singletenancy/cilium-dualstack-overlay/cilium-dualstackoverlay-e2e.steps.yaml

@@ -96,6 +96,7 @@ steps:
       workingDirectory: $(ACN_DIR)
       addSpnToEnvironment: true
       inlineScript: |
+        set -e
         cd test/integration/load
 
         # Scale Cluster Up/Down to confirm functioning CNS
@@ -110,6 +111,7 @@ steps:
     retryCountOnTaskFailure: 3
 
   - script: |
+      set -e
       echo "Run Cilium Connectivity Tests"
       cilium status
       cilium connectivity test --connect-timeout 4s --request-timeout 30s --test '!pod-to-pod-encryption,!node-to-node-encryption,!no-unexpected-packet-drops' --force-deploy

.pipelines/singletenancy/cilium-nodesubnet/cilium-nodesubnet-e2e-step-template.yaml

@@ -53,7 +53,7 @@ steps:
       inlineScript: |
         set -e
         make -C ./hack/aks set-kubeconf AZCLI=az CLUSTER=${{ parameters.clusterName }}
-        ls -lah      
+        ls -lah
         pwd
         kubectl cluster-info
         kubectl get po -owide -A
@@ -81,5 +81,8 @@ steps:
     retryCountOnTaskFailure: 3
     name: "nodeSubnetE2ETests"
     displayName: "Run NodeSubnet E2E"
-  
+
   - template: ../../templates/cilium-tests.yaml
+    parameters:
+      clusterName: ${{ parameters.clusterName }}
+      scaleup: ${{ parameters.scaleup }}

.pipelines/singletenancy/cilium-overlay-withhubble/cilium-overlay-e2e-step-template.yaml

@@ -96,6 +96,7 @@ steps:
       scriptType: "bash"
       addSpnToEnvironment: true
       inlineScript: |
+        set -e
         cd test/integration/load
 
         # Scale Cluster Up/Down to confirm functioning CNS
@@ -110,6 +111,7 @@ steps:
     retryCountOnTaskFailure: 3
 
   - script: |
+      set -e
       echo "Run Cilium Connectivity Tests"
       cilium status
       cilium connectivity test --connect-timeout 4s --request-timeout 30s --test '!pod-to-pod-encryption,!node-to-node-encryption' --force-deploy
@@ -135,19 +137,17 @@ steps:
         displayName: "Run Hubble Connectivity Tests"
 
   - script: |
+      set -e
       echo "validate pod IP assignment and check systemd-networkd restart"
       kubectl get pod -owide -A
-      # Deleting echo-external-node deployment until cilium version matches TODO. https://github.com/cilium/cilium-cli/issues/67 is addressing the change.
-      # Saves 17 minutes
 
-      kubectl delete deploy -n $(ciliumNamespace) echo-external-node
       if [ "$CILIUM_VERSION_TAG" = "cilium-nightly-pipeline" ]; then
         echo "Check cilium identities in $(ciliumNamespace) namepsace during nightly run"
         echo "expect the identities to be deleted when the namespace is deleted"
         kubectl get ciliumidentity | grep cilium-test
       fi
       make test-validate-state
-      echo "delete cilium connectivity test resources and re-validate state"
+      echo "delete cilium connectivity test resources and re-validate state" # TODO Delete this and the next 4 lines if connectivity no longer has bug
       kubectl delete ns $(ciliumNamespace)
       kubectl get pod -owide -A
       make test-validate-state