Merge pull request #2319 from conectado/update-benchmark-ci

Update benchmarks CI to use less steps to save time
Joystream · Apr 2, 2021 · d373a5c · d373a5c
2 parents 19d5e16 + 0ef5fa8
commit d373a5c
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/.github/workflows/joystream-node-benchmarks.yml b/.github/workflows/joystream-node-benchmarks.yml
@@ -41,7 +41,7 @@ jobs:
 
       - name: Generate Weights
         run: |
-          ./scripts/generate-weights.sh
+          ./scripts/generate-weights.sh 5 2
           # Show any changes in computed weights
           git diff
-        if: env.GIT_DIFF
+        if: env.GIT_DIFF
diff --git a/scripts/generate-weights.sh b/scripts/generate-weights.sh
@@ -4,15 +4,18 @@
 
 SCRIPT_DIR=$(dirname "${BASH_SOURCE[0]}")
 
+STEPS=${1:-50}
+REPEAT=${2:-20}
+
 benchmark() {
   echo "Generating weights for $1..."
   start=`date +%s`
   ERROR=$($SCRIPT_DIR/../target/release/joystream-node benchmark \
       --pallet=$1 \
       --extrinsic=* \
       --chain=dev \
-      --steps=50 \
-      --repeat=20 \
+      --steps=$STEPS \
+      --repeat=$REPEAT \
       --execution=wasm \
       --output=. 2>&1 > /dev/null)