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/08/04 06:37:00 UTC
[GitHub] [airflow] feluelle commented on a change in pull request #10146: Retry max 3 times if failing to initialize integration
feluelle commented on a change in pull request #10146:
URL: https://github.com/apache/airflow/pull/10146#discussion_r464830382
##########
File path: scripts/ci/testing/ci_run_airflow_testing.sh
##########
@@ -31,14 +31,22 @@ fi
function run_airflow_testing_in_docker() {
set +u
set +e
- # shellcheck disable=SC2016
- docker-compose --log-level INFO \
- -f "${SCRIPTS_CI_DIR}/docker-compose/base.yml" \
- -f "${SCRIPTS_CI_DIR}/docker-compose/backend-${BACKEND}.yml" \
- "${INTEGRATIONS[@]}" \
- "${DOCKER_COMPOSE_LOCAL[@]}" \
- run airflow "${@}"
- EXIT_CODE=$?
+ for TRY_NUM in {1..3}
+ do
+ docker-compose --log-level INFO \
+ -f "${SCRIPTS_CI_DIR}/docker-compose/base.yml" \
+ -f "${SCRIPTS_CI_DIR}/docker-compose/backend-${BACKEND}.yml" \
+ "${INTEGRATIONS[@]}" \
+ "${DOCKER_COMPOSE_LOCAL[@]}" \
+ run airflow "${@}"
+ EXIT_CODE=$?
+ if [[ ${EXIT_CODE} == 254 ]]; then
+ echo "Failed on starting integration on ${TRY_NUM} try."
Review comment:
Should we wait some time before retrying?
----------------------------------------------------------------
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