Commit db81ae28 authored by Achilleas Pipinellis's avatar Achilleas Pipinellis

Use a secret variable to define the repo suffix

That's a hack since we cannot inject variables that are
defined in `script:` into `environments:url`, and we want
to avoid maintaining two different job versions in CE and EE.

Relevant discussion:
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/14236/diffs#note_40140693
parent 54a49a06
......@@ -191,8 +191,9 @@ review-docs-deploy:
stage: build
environment:
name: review-docs/$CI_COMMIT_REF_NAME
# The URL must end in /ce
url: http://$CI_COMMIT_REF_SLUG-built-from-ce-ee.$DOCS_REVIEW_APPS_DOMAIN/ce
# DOCS_REVIEW_APPS_DOMAIN and DOCS_GITLAB_REPO_SUFFIX are secret variables
# Discussion: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/14236/diffs#note_40140693
url: http://$CI_COMMIT_REF_SLUG-built-from-ce-ee.$DOCS_REVIEW_APPS_DOMAIN/$DOCS_GITLAB_REPO_SUFFIX
on_stop: review-docs-cleanup
script:
- gem install gitlab --no-doc
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment