[percy] only execute percy setup when necessary (elastic#39043)

(cherry picked from commit 9c1bdb9)
thompsongl · Jun 16, 2019 · 4cfdda4 · 4cfdda4
1 parent e1d73f4
commit 4cfdda4
Show file tree

Hide file tree

Showing 4 changed files with 21 additions and 14 deletions.
diff --git a/src/dev/ci_setup/setup.sh b/src/dev/ci_setup/setup.sh
@@ -149,13 +149,6 @@ else
   echo "~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"
 fi
 
-###
-### skip chomium download, use the system chrome install
-###
-export PUPPETEER_SKIP_CHROMIUM_DOWNLOAD=true
-PUPPETEER_EXECUTABLE_PATH="$(command -v google-chrome-stable)"
-export PUPPETEER_EXECUTABLE_PATH
-
 ###
 ### install dependencies
 ###
@@ -172,13 +165,6 @@ if [ "$GIT_CHANGES" ]; then
   exit 1
 fi
 
-###
-### Set Percy parallel build support environment vars
-###
-eval "$(node ./src/dev/ci_setup/get_percy_env)"
-echo " -- PERCY_PARALLEL_NONCE='$PERCY_PARALLEL_NONCE'"
-echo " -- PERCY_PARALLEL_TOTAL='$PERCY_PARALLEL_TOTAL'"
-
 ###
 ### rebuild kbn-pm distributable to ensure it's not out of date
 ###

diff --git a/src/dev/ci_setup/setup_percy.sh b/src/dev/ci_setup/setup_percy.sh
@@ -0,0 +1,17 @@
+#!/usr/bin/env bash
+
+set -e
+
+###
+### skip chomium download, use the system chrome install
+###
+export PUPPETEER_SKIP_CHROMIUM_DOWNLOAD=true
+PUPPETEER_EXECUTABLE_PATH="$(command -v google-chrome-stable)"
+export PUPPETEER_EXECUTABLE_PATH
+
+###
+### Set Percy parallel build support environment vars
+###
+eval "$(node ./src/dev/ci_setup/get_percy_env)"
+echo " -- PERCY_PARALLEL_NONCE='$PERCY_PARALLEL_NONCE'"
+echo " -- PERCY_PARALLEL_TOTAL='$PERCY_PARALLEL_TOTAL'"
diff --git a/test/scripts/jenkins_visual_regression.sh b/test/scripts/jenkins_visual_regression.sh
@@ -3,6 +3,8 @@
 set -e
 trap 'node "$KIBANA_DIR/src/dev/failed_tests/cli"' EXIT
 
+source "$KIBANA_DIR/src/dev/ci_setup/setup_percy.sh"
+
 node scripts/build --debug --oss;
 linuxBuild="$(find "$KIBANA_DIR/target" -name 'kibana-oss-*-linux-x86_64.tar.gz')"
 installDir="$PARENT_DIR/install/kibana"

diff --git a/test/scripts/jenkins_xpack_visual_regression.sh b/test/scripts/jenkins_xpack_visual_regression.sh
@@ -3,6 +3,8 @@
 set -e
 trap 'node "$KIBANA_DIR/src/dev/failed_tests/cli"' EXIT
 
+source "$KIBANA_DIR/src/dev/ci_setup/setup_percy.sh"
+
 node scripts/build --debug --no-oss;
 linuxBuild="$(find "$KIBANA_DIR/target" -name 'kibana-*-linux-x86_64.tar.gz')"
 installDir="$PARENT_DIR/install/kibana"