chore: update logs step SD-1405

Author: sowecthal

charts/tekton-pipelines/Chart.yaml

@@ -6,7 +6,7 @@ type: application
 # This is the chart version. This version number should be incremented each time you make changes
 # to the chart and its templates, including the app version.
 # Versions are expected to follow Semantic Versioning (https://semver.org/)
-version: 2.2.14-dev.13
+version: 2.2.14-dev.14
 
 maintainers:
   - url: https://www.saritasa.com/

charts/tekton-pipelines/README.md

@@ -31,7 +31,7 @@ saritasa-tekton-pipelines
 
 ## `chart.version`
 
-![Version: 2.2.14-dev.13](https://img.shields.io/badge/Version-2.2.14-dev.13-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square)
+![Version: 2.2.14-dev.14](https://img.shields.io/badge/Version-2.2.14-dev.14-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square)
 
 ## Maintainers
 

charts/tekton-pipelines/templates/common/tasks/slack-notification.yaml

@@ -71,9 +71,9 @@ spec:
           failed_task=$(cat $(results.failed_info.path) | jq '.task' -r)
           failed_step=$(cat $(results.failed_info.path) | jq '.step' -r)   
 
+          echo $failed_pod_name > $(results.failed_pod_name.path)
           echo $failed_task > $(results.failed_task.path)
           echo $failed_step > $(results.failed_step.path)
-          echo $failed_pod_name > $(results.failed_pod_name.path)
         fi
 
     - name: get-failed-logs
@@ -82,14 +82,12 @@ spec:
       script: |
         #!/bin/sh
 
-        failed_pod_name=$(cat $(results.failed_pod_name.path))
-        failed_step=$(cat $(results.failed_step.path))
-
-        if [ -n "$failed_pod_name" ] && [ -n "$failed_step" ]; then
+        if [ -s "$(results.failed_pod_name.path)" ] && [ -s "$(results.failed_step.path)" ]; then
           # Fetch logs and save info about the failed task
           echo "Fetching logs for failed step: ${failed_step}"
           until [ -s "$(results.logs.path)" ]
           do
+          # It takes time for logs to appear, until then curl will return an empty string 
           sleep 5
           curl -sk "https://tekton-logs-server.ci.svc.cluster.local/logs/ci/${failed_pod_name}/step-${failed_step}" | tail -c 1982 > $(results.logs.path)
           done