ashb edited a comment on pull request #9247:
URL: https://github.com/apache/airflow/pull/9247#issuecomment-643261552


   I've got it.
   
   
   ```diff
   diff --git scripts/ci/ci_run_airflow_testing.sh 
scripts/ci/ci_run_airflow_testing.sh
   index cb4fb4aad..73c1e95f9 100755
   --- scripts/ci/ci_run_airflow_testing.sh
   +++ scripts/ci/ci_run_airflow_testing.sh
   @@ -30,7 +30,9 @@ function run_airflow_testing_in_docker() {
               /opt/airflow/scripts/ci/in_container/entrypoint_ci.sh "${@}"
             # Note the command is there twice (!) because it is passed via 
bash -c
             # and bash -c starts passing parameters from $0. TODO: fixme
   +    local status=$?
        set -u
   +    return $status
    }
    
    # shellcheck source=scripts/ci/_script_init.sh
   
   ```
   
   Removing the loop works too. The issue was the `set +e` in the loop meant 
the exit code of `docker-compose` was ignored.
   
   I'll do both for belt-and-braces approach.


----------------------------------------------------------------
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