summaryrefslogtreecommitdiff
path: root/scripts
diff options
context:
space:
mode:
authorAchilleas Pipinellis <axil@gitlab.com>2017-11-15 13:05:53 +0100
committerAchilleas Pipinellis <axil@gitlab.com>2017-11-16 08:05:32 +0100
commite1fb8b35e7a298233fc635c5ec4be30debc1045c (patch)
tree50170dea6da4202ec6861ddc3879df5a6bbdb4af /scripts
parent331c807a40c2c61ea9b14423d65bed7e4fccca33 (diff)
downloadgitlab-ce-e1fb8b35e7a298233fc635c5ec4be30debc1045c.tar.gz
Change logic of repo name and slugs in docs review scriptsimplify-docs-review-script
We need to make room for Runner and Omnibus docs. This change makes sure the correct values are defined based on CI_PROJECT_NAME.
Diffstat (limited to 'scripts')
-rwxr-xr-xscripts/trigger-build-docs35
1 files changed, 24 insertions, 11 deletions
diff --git a/scripts/trigger-build-docs b/scripts/trigger-build-docs
index d3a9f5ff4ea..89ad6a99467 100755
--- a/scripts/trigger-build-docs
+++ b/scripts/trigger-build-docs
@@ -31,13 +31,6 @@ def docs_branch
end
#
-# Dummy way to find out in which repo we are, CE or EE
-#
-def ee?
- File.exist?('CHANGELOG-EE.md')
-end
-
-#
# Create a remote branch in gitlab-docs
#
def create_remote_branch
@@ -56,14 +49,34 @@ def remove_remote_branch
end
#
+# Define suffix in review app URL based on project
+#
+def slug
+ case ENV["CI_PROJECT_NAME"]
+ when 'gitlab-ce'
+ 'ce'
+ when 'gitlab-ee'
+ 'ee'
+ when 'gitlab-runner'
+ 'runner'
+ when 'omnibus-gitlab'
+ 'omnibus'
+ end
+end
+
+#
+# Overriding vars in https://gitlab.com/gitlab-com/gitlab-docs/blob/master/.gitlab-ci.yml
+#
+def param_name
+ "BRANCH_#{slug.upcase}"
+end
+
+#
# Trigger a pipeline in gitlab-docs
#
def trigger_pipeline
- # Overriding vars in https://gitlab.com/gitlab-com/gitlab-docs/blob/master/.gitlab-ci.yml
- param_name = ee? ? 'BRANCH_EE' : 'BRANCH_CE'
-
# The review app URL
- app_url = "http://#{docs_branch}.#{ENV["DOCS_REVIEW_APPS_DOMAIN"]}/#{ee? ? 'ee' : 'ce'}"
+ app_url = "http://#{docs_branch}.#{ENV["DOCS_REVIEW_APPS_DOMAIN"]}/#{slug}"
# Create the pipeline
puts "=> Triggering a pipeline..."