[GitHub] [airflow] potiuk commented on pull request #15939: Add CI test for helm upgrade with different Executors

[GitBox <gi...@apache.org>]

potiuk commented on pull request #15939:
URL: https://github.com/apache/airflow/pull/15939#issuecomment-846814123


   What happened there  the parallell command started and completed immediately - likely there is a typo somewhere that make the script failed and it did not have chance to show output. The output should be printed if the scripts fail, but in this case I think they failed with some typo/error before they had a change to set a trap that save status to status file 
   
   In this case just remove the redirection of parallell command (in kind::run_kubernetes_tests):
   ```
       parallel --ungroup --bg --semaphore --semaphorename "${SEMAPHORE_NAME}" \
           --jobs "${MAX_PARALLEL_K8S_JOBS}" \
               "$(dirname "${BASH_SOURCE[0]}")/${single_job_filename}" \
                   "${kubernetes_version}" "${python_version}" >"${JOB_LOG}" 2>&1
   ```
   
   change this to:
   ```
       parallel --ungroup --bg --semaphore --semaphorename "${SEMAPHORE_NAME}" \
           --jobs "${MAX_PARALLEL_K8S_JOBS}" \
               "$(dirname "${BASH_SOURCE[0]}")/${single_job_filename}" \
                   "${kubernetes_version}" "${python_version}"
   ```
   
   And you should see the error
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org