diff options
| author | Rémy Coutable <remy@rymai.me> | 2019-07-30 17:33:00 +0200 |
|---|---|---|
| committer | Rémy Coutable <remy@rymai.me> | 2019-07-30 19:13:43 +0200 |
| commit | 8f501a74bde199723e1f01121a9a56e01aacaf1c (patch) | |
| tree | 2abca7d63488438a409eeb6ec3103b481a5fe602 | |
| parent | ab509c78929d9b5f4e02d013e8911ac9d0a07aad (diff) | |
| download | gitlab-ce-8f501a74bde199723e1f01121a9a56e01aacaf1c.tar.gz | |
Remove the wait_for_review_app_to_be_accessible step from review-deploy65003-remove-wait_for_review_app_to_be_accessible-step
Signed-off-by: Rémy Coutable <remy@rymai.me>
| -rw-r--r-- | .gitlab/ci/review.gitlab-ci.yml | 3 | ||||
| -rwxr-xr-x | scripts/review_apps/review-apps.sh | 25 |
2 files changed, 1 insertions, 27 deletions
diff --git a/.gitlab/ci/review.gitlab-ci.yml b/.gitlab/ci/review.gitlab-ci.yml index 41d52c4e095..aa722f6ef33 100644 --- a/.gitlab/ci/review.gitlab-ci.yml +++ b/.gitlab/ci/review.gitlab-ci.yml @@ -101,8 +101,7 @@ schedule:review-build-cng: - install_tiller - install_external_dns - download_chart - - deploy || display_deployment_debug - - wait_for_review_app_to_be_accessible + - deploy || (display_deployment_debug && exit 1) - add_license artifacts: paths: [review_app_url.txt] diff --git a/scripts/review_apps/review-apps.sh b/scripts/review_apps/review-apps.sh index 408a0595f9e..4935c1342a3 100755 --- a/scripts/review_apps/review-apps.sh +++ b/scripts/review_apps/review-apps.sh @@ -340,31 +340,6 @@ function display_deployment_debug() { fi } -function wait_for_review_app_to_be_accessible() { - echoinfo "Waiting for the Review App at ${CI_ENVIRONMENT_URL} to be accessible..." true - - local interval=5 - local elapsed_seconds=0 - local max_seconds=$((2 * 60)) - while true; do - local review_app_http_code - review_app_http_code=$(curl --silent --output /dev/null --max-time 5 --write-out "%{http_code}" "${CI_ENVIRONMENT_URL}/users/sign_in") - if [[ "${review_app_http_code}" -eq "200" ]] || [[ "${elapsed_seconds}" -gt "${max_seconds}" ]]; then - break - fi - - let "elapsed_seconds+=interval" - sleep ${interval} - done - - if [[ "${review_app_http_code}" -eq "200" ]]; then - echoinfo "The Review App at ${CI_ENVIRONMENT_URL} is ready after ${elapsed_seconds} seconds!" - else - echoerr "The Review App at ${CI_ENVIRONMENT_URL} isn't ready after ${max_seconds} seconds of polling..." - exit 1 - fi -} - function add_license() { if [ -z "${REVIEW_APPS_EE_LICENSE}" ]; then echo "License not found" && return; fi |