You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2020/06/12 13:09:41 UTC

[GitHub] [airflow] ashb edited a comment on pull request #9247: CI: Propogate Exit Code Correctly

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


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