summaryrefslogtreecommitdiff
path: root/scripts
diff options
context:
space:
mode:
authorMark Lapierre <mlapierre@gitlab.com>2018-12-17 13:22:09 -0500
committerMark Lapierre <mlapierre@gitlab.com>2018-12-17 15:30:52 -0500
commit0f751f6f17d1f457276f15632b31203f658ec4c9 (patch)
tree29e0b9eb4f41d07ecc8f5d09116a9be900605d58 /scripts
parent4a10c813e726d09216c534bb0ad0ae50a0400259 (diff)
downloadgitlab-ce-0f751f6f17d1f457276f15632b31203f658ec4c9.tar.gz
Run gitlab:assets:compile on qa jobs
Optionally compiling assets when needed for qa jobs proved unworkable so instead we fall back on compiling them for all qa jobs.
Diffstat (limited to 'scripts')
-rwxr-xr-xscripts/review_apps/review-apps.sh4
-rwxr-xr-xscripts/trigger-build8
2 files changed, 3 insertions, 9 deletions
diff --git a/scripts/review_apps/review-apps.sh b/scripts/review_apps/review-apps.sh
index b50bf2161cb..8f9414cc1ed 100755
--- a/scripts/review_apps/review-apps.sh
+++ b/scripts/review_apps/review-apps.sh
@@ -292,7 +292,7 @@ function get_job_id() {
local job_id=$(curl --silent --show-error --header "PRIVATE-TOKEN: ${API_TOKEN}" "${url}" | jq "map(select(.name == \"${job_name}\")) | map(.id) | last")
[[ "${job_id}" == "null" && "${page}" -lt "$max_page" ]] || break
- ((page++))
+ let "page++"
done
if [[ "${job_id}" == "" ]]; then
@@ -334,7 +334,7 @@ function wait_for_job_to_be_done() {
[[ "${job_status}" == "pending" || "${job_status}" == "running" ]] || break
printf "."
- ((elapsed_seconds+=$interval))
+ let "elapsed_seconds+=interval"
sleep ${interval}
done
diff --git a/scripts/trigger-build b/scripts/trigger-build
index 14af3281106..4032ba853e6 100755
--- a/scripts/trigger-build
+++ b/scripts/trigger-build
@@ -68,7 +68,7 @@ module Trigger
def base_variables
{
- 'GITLAB_REF_SLUG' => ref_slug,
+ 'GITLAB_REF_SLUG' => ENV['CI_COMMIT_REF_SLUG'],
'TRIGGERED_USER' => ENV['TRIGGERED_USER'] || ENV['GITLAB_USER_NAME'],
'TRIGGER_SOURCE' => ENV['CI_JOB_URL'],
'TOP_UPSTREAM_SOURCE_PROJECT' => ENV['CI_PROJECT_PATH'],
@@ -77,12 +77,6 @@ module Trigger
}
end
- def ref_slug
- return 'master' if ENV['CI_COMMIT_REF_SLUG'] =~ %r{(\Aqa[/-]|-qa\z)}
-
- ENV['CI_COMMIT_REF_SLUG']
- end
-
# Read version files from all components
def version_file_variables
Dir.glob("*_VERSION").each_with_object({}) do |version_file, params|