[CI] Tolerate errors when collecting k8s logs in CI (#217)

- The log collection failed after a command failed.
- tolerate errors

.ci/helm.sh

@@ -91,12 +91,12 @@ function ci::collect_k8s_logs() {
     echo "Collecting k8s logs to ${K8S_LOGS_DIR}"
     for k8sobject in $(${KUBECTL} get pods,jobs -n ${NAMESPACE} -l app=pulsar -o=name); do
       filebase="${k8sobject//\//_}"
-      ${KUBECTL} logs -n ${NAMESPACE} "$k8sobject" --all-containers=true --ignore-errors=true --prefix=true > "${filebase}.$$.log.txt"
-      ${KUBECTL} logs -n ${NAMESPACE} "$k8sobject" --all-containers=true --ignore-errors=true --prefix=true --previous=true > "${filebase}.previous.$$.log.txt"
+      ${KUBECTL} logs -n ${NAMESPACE} "$k8sobject" --all-containers=true --ignore-errors=true --prefix=true > "${filebase}.$$.log.txt" || true
+      ${KUBECTL} logs -n ${NAMESPACE} "$k8sobject" --all-containers=true --ignore-errors=true --prefix=true --previous=true > "${filebase}.previous.$$.log.txt" || true
     done;
-    ${KUBECTL} get events --sort-by=.lastTimestamp -A > events.$$.log.txt
-    ${KUBECTL} get events --sort-by=.lastTimestamp -A -o yaml > events.$$.log.yaml
-    ${KUBECTL} get -n ${NAMESPACE} all -o yaml > k8s_resources.$$.yaml
+    ${KUBECTL} get events --sort-by=.lastTimestamp -A > events.$$.log.txt || true
+    ${KUBECTL} get events --sort-by=.lastTimestamp -A -o yaml > events.$$.log.yaml || true
+    ${KUBECTL} get -n ${NAMESPACE} all -o yaml > k8s_resources.$$.yaml || true
 }
 
 function ci::install_pulsar_chart() {

.github/workflows/pulsar.yml

@@ -56,6 +56,7 @@ jobs:
         shell: bash
         run: |
           source .ci/helm.sh
+          set +e
           ci::collect_k8s_logs
 
       - name: Upload k8s logs on failure

.github/workflows/pulsar_bk_tls.yml

@@ -55,6 +55,7 @@ jobs:
         shell: bash
         run: |
           source .ci/helm.sh
+          set +e
           ci::collect_k8s_logs
 
       - name: Upload k8s logs on failure

.github/workflows/pulsar_broker_tls.yml

@@ -56,6 +56,7 @@ jobs:
         shell: bash
         run: |
           source .ci/helm.sh
+          set +e
           ci::collect_k8s_logs
 
       - name: Upload k8s logs on failure

.github/workflows/pulsar_function.yml

@@ -58,6 +58,7 @@ jobs:
         shell: bash
         run: |
           source .ci/helm.sh
+          set +e
           ci::collect_k8s_logs
 
       - name: Upload k8s logs on failure

.github/workflows/pulsar_image.yml

@@ -56,6 +56,7 @@ jobs:
         shell: bash
         run: |
           source .ci/helm.sh
+          set +e
           ci::collect_k8s_logs
 
       - name: Upload k8s logs on failure

.github/workflows/pulsar_jwt_asymmetric.yml

@@ -58,6 +58,7 @@ jobs:
         shell: bash
         run: |
           source .ci/helm.sh
+          set +e
           ci::collect_k8s_logs
 
       - name: Upload k8s logs on failure

.github/workflows/pulsar_jwt_symmetric.yml

@@ -58,6 +58,7 @@ jobs:
         shell: bash
         run: |
           source .ci/helm.sh
+          set +e
           ci::collect_k8s_logs
 
       - name: Upload k8s logs on failure

.github/workflows/pulsar_tls.yml

@@ -56,6 +56,7 @@ jobs:
         shell: bash
         run: |
           source .ci/helm.sh
+          set +e
           ci::collect_k8s_logs
 
       - name: Upload k8s logs on failure

.github/workflows/pulsar_zk_tls.yml

@@ -56,6 +56,7 @@ jobs:
         shell: bash
         run: |
           source .ci/helm.sh
+          set +e
           ci::collect_k8s_logs
 
       - name: Upload k8s logs on failure

.github/workflows/pulsar_zkbk_tls.yml

@@ -56,6 +56,7 @@ jobs:
         shell: bash
         run: |
           source .ci/helm.sh
+          set +e
           ci::collect_k8s_logs
 
       - name: Upload k8s logs on failure