Skip to content
Projects
Groups
Snippets
Help
Loading...
Help
Support
Keyboard shortcuts
?
Submit feedback
Contribute to GitLab
Sign in / Register
Toggle navigation
G
gitlab-ce
Project overview
Project overview
Details
Activity
Releases
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Issues
0
Issues
0
List
Boards
Labels
Milestones
Merge Requests
1
Merge Requests
1
Analytics
Analytics
Repository
Value Stream
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Create a new issue
Commits
Issue Boards
Open sidebar
nexedi
gitlab-ce
Commits
8f501a74
Commit
8f501a74
authored
Jul 30, 2019
by
Rémy Coutable
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
Remove the wait_for_review_app_to_be_accessible step from review-deploy
Signed-off-by:
Rémy Coutable
<
remy@rymai.me
>
parent
ab509c78
Changes
2
Hide whitespace changes
Inline
Side-by-side
Showing
2 changed files
with
1 addition
and
27 deletions
+1
-27
.gitlab/ci/review.gitlab-ci.yml
.gitlab/ci/review.gitlab-ci.yml
+1
-2
scripts/review_apps/review-apps.sh
scripts/review_apps/review-apps.sh
+0
-25
No files found.
.gitlab/ci/review.gitlab-ci.yml
View file @
8f501a74
...
...
@@ -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
]
...
...
scripts/review_apps/review-apps.sh
View file @
8f501a74
...
...
@@ -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
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment