From e1e21aa539975756e048396ed3d243035b383e1d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C5=81ukasz=20Mierzwa?= <l.mierzwa@gmail.com>
Date: Sun, 24 May 2020 16:06:41 +0100
Subject: [PATCH] fix(ci): run Go benchmarks only on Go code changes

---
 .travis.yml                     | 18 ++----------------
 scripts/ci-diff-benchmark-go.sh | 12 ++++++++++++
 scripts/ci-diff-webpack.sh      | 14 ++++++++++++++
 scripts/have-backend-changes.sh | 27 +++++++++++++++++++++++++++
 4 files changed, 55 insertions(+), 16 deletions(-)
 create mode 100755 scripts/ci-diff-benchmark-go.sh
 create mode 100755 scripts/ci-diff-webpack.sh
 create mode 100755 scripts/have-backend-changes.sh

diff --git a/.travis.yml b/.travis.yml
index b30d353b3..5f7ad906c 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -49,15 +49,7 @@ jobs:
         # GITHUB_TOKEN for CI Bundle size diff comments and gist uploads
         - secure: "Or5fiXZfgIsXvzoOdEprRwJ0uwUjMvxGHE3LG+h3OsIBO6WA8vgOqVOjqVHV2dgC9cSjm5A7MX52S7cDqfkQDkgnVHpVxRwDC9n9O4vnaFdCZ4nC+d18z8dikbiwgdeWQ+Wi6RhZENye1Lu5sBaAJ09wYgx9lNdEVpRaTUvUw6grSlESJZSXoxfxWWpmTyx+yPH4sxuWjZ7gCspDX9s9k4fjpY4LkhQQwLlk8wPc2hfDg48e+K1OR6sYB8uRS33Xc4fQtzElzazmaZ0fn77h5ysDgC1g/ko+E2j8HHMbZvFpzYpm1bCpIv1G/0A2ItH7gT3HsuwkDvfH/it56JTCbBWJJ+hTDeswCQNu0h797QM6jv0o5wgKpHR1t+AeM9vDe4Ds0pAXouJz0LJewNOdNvi5O1BZA9OooKc34hwTJs/zj5NwiZuOyPSMhDBGa++Vhsr9K3rPD9+97M2hac6NO6TBVWZjvqJilmkjJs+bKrl//ClBvdhDGkJNDbB+2emdD1/wzpPVJPp3IRhzeEF89IVE58qE+OQnIwtbEZ2W1ct6Ep7ZJdrXWc/VBdJB1ELfUtNmkvWFZD5IJfnb/Z5MS6iespXlV5alPQ7eZ2jNl3tn7uDaCStuQN1tO2wthNnsSU/OkfFRch/Ks3gYC5+v7n8aJMkTYFmHr4Y/xlXBsrA="
       script:
-        - git fetch origin master
-        - git reset --hard FETCH_HEAD
-        - make -C ui build/stats.json
-        - mv ui/build/stats.json master.json
-        - make clean
-        - git checkout -f ${TRAVIS_COMMIT}
-        - make -C ui build/stats.json
-        - ./scripts/cra-bundle-stats-diff.py master.json ui/build/stats.json | tee diff.html
-        - ./scripts/pr-comment.py "Webpack bundle size diff" diff.html html
+        - ./scripts/ci-diff-webpack.sh
 
     - stage: Stats
       name: Benchmark Go code compare
@@ -69,13 +61,7 @@ jobs:
       before_script:
         - travis_retry make mock-assets
       script:
-        - git fetch origin master
-        - git reset --hard FETCH_HEAD
-        - make benchmark-go | tee master.txt
-        - git checkout -f ${TRAVIS_COMMIT}
-        - make benchmark-go | tee new.txt
-        - make benchmark-compare-go | tee benchstat.txt
-        - ./scripts/pr-comment.py "Go benchmark diff" benchstat.txt noformat
+        - ./scripts/have-backend-changes.sh || ./scripts/ci-diff-benchmark-go.sh
 
     - stage: Test
       name: Test Go code
diff --git a/scripts/ci-diff-benchmark-go.sh b/scripts/ci-diff-benchmark-go.sh
new file mode 100755
index 000000000..d8f1411e6
--- /dev/null
+++ b/scripts/ci-diff-benchmark-go.sh
@@ -0,0 +1,12 @@
+#!/usr/bin/env bash
+
+set -o errexit
+set -o pipefail
+
+git fetch origin master
+git reset --hard FETCH_HEAD
+make benchmark-go | tee master.txt
+git checkout -f ${TRAVIS_COMMIT}
+make benchmark-go | tee new.txt
+make benchmark-compare-go | tee benchstat.txt
+./scripts/pr-comment.py "Go benchmark diff" benchstat.txt noformat
diff --git a/scripts/ci-diff-webpack.sh b/scripts/ci-diff-webpack.sh
new file mode 100755
index 000000000..becb72cd2
--- /dev/null
+++ b/scripts/ci-diff-webpack.sh
@@ -0,0 +1,14 @@
+#!/usr/bin/env bash
+
+set -o errexit
+set -o pipefail
+
+git fetch origin master
+git reset --hard FETCH_HEAD
+make -C ui build/stats.json
+mv ui/build/stats.json master.json
+make clean
+git checkout -f ${TRAVIS_COMMIT}
+make -C ui build/stats.json
+./scripts/cra-bundle-stats-diff.py master.json ui/build/stats.json | tee diff.html
+./scripts/pr-comment.py "Webpack bundle size diff" diff.html html
diff --git a/scripts/have-backend-changes.sh b/scripts/have-backend-changes.sh
new file mode 100755
index 000000000..dbdad45cf
--- /dev/null
+++ b/scripts/have-backend-changes.sh
@@ -0,0 +1,27 @@
+#!/usr/bin/env bash
+
+set -o errexit
+set -o pipefail
+
+
+if [ "${TRAVIS_BRANCH}" == "master" ] && [ -n ${TRAVIS_COMMIT_RANGE} ]; then
+  RANGE="${TRAVIS_COMMIT_RANGE}"
+else
+  git fetch origin master
+  RANGE="FETCH_HEAD...${TRAVIS_COMMIT}"
+fi
+
+
+git log --no-merges --name-only --pretty=format: ${RANGE} | grep -Ev '^$' | sort | uniq | while read FILE ; do
+  if [[ "${FILE}" =~ ^cmd/.+ ]]; then
+    echo "[G] ${FILE}"
+    exit 1
+  elif [[ "${FILE}" =~ ^internal/.+ ]]; then
+    echo "[G] ${FILE}"
+    exit 1
+  else
+    echo "[ ] ${FILE}"
+  fi
+done
+
+exit 0