potiuk edited a comment on pull request #15939:
URL: https://github.com/apache/airflow/pull/15939#issuecomment-848936369


   Just change (in `_parallel.sh`):  
   
   ```
   status=$(cat "${PARALLEL_MONITORED_DIR}/${SEMAPHORE_NAME}/${job}/status")
   ```
   
   into 
   
   ```
   status=$(cat "${PARALLEL_MONITORED_DIR}/${SEMAPHORE_NAME}/${job}/status"  || 
true)
   ```
   
   Lack of "true" there, masks the root cause. Simply the actual error is not 
printed to the output because this line fails. When we add `|| true`, the cat 
command will not fail and few lines below content of the output will be 
written, and this should reveal the actual problem that is causing the command 
to fail.
   
   
   ```
           status=$(cat 
"${PARALLEL_MONITORED_DIR}/${SEMAPHORE_NAME}/${job}/status")
           if [[ ${status} == "0" ]]; then
               parallel::output_log_for_successful_job "${job}"
           else
               parallel::output_log_for_failed_job "${job}"    <- THIS LINE 
SHOULD PRINT THE ROOT CAUSE OF THE PROBLEM
               # SKIPPED_FAILED_JOB failure does not trigger whole test failure
               if [[ ${SKIPPED_FAILED_JOB=} != "${job}" ]]; then
                   return_code=$((return_code + 1))
               fi
           fi
   ```


-- 
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:
[email protected]


Reply via email to