From 7d65bb2c419d280b699f18117e06d3548e6c6017 Mon Sep 17 00:00:00 2001 From: Monthon Klongklaew Date: Wed, 25 Sep 2024 12:13:18 +0000 Subject: [PATCH] Fix resource utilization check in the CI Our CI workflows work by checking out the code from branch `gh-pages` to retrieve benchmark results from previous commits and compare them to values in the current run. However, the resource utilization check was done after the benchmark result check which already has pulled in the branch `gh-pages` resulting in errors because the branch already exists. This change fixes that. Signed-off-by: Monthon Klongklaew --- .github/workflows/bench.yml | 2 ++ .github/workflows/bench_s3express.yml | 1 + 2 files changed, 3 insertions(+) diff --git a/.github/workflows/bench.yml b/.github/workflows/bench.yml index c9d205851..ecbdfb57c 100644 --- a/.github/workflows/bench.yml +++ b/.github/workflows/bench.yml @@ -94,6 +94,7 @@ jobs: auto-push: ${{ inputs.publish }} comment-on-alert: true max-items-in-chart: 20 + skip-fetch-gh-pages: true latency-bench: name: Benchmark (Latency) @@ -218,3 +219,4 @@ jobs: auto-push: ${{ inputs.publish }} comment-on-alert: true max-items-in-chart: 20 + skip-fetch-gh-pages: true diff --git a/.github/workflows/bench_s3express.yml b/.github/workflows/bench_s3express.yml index dd83d6e6f..17c39de4d 100644 --- a/.github/workflows/bench_s3express.yml +++ b/.github/workflows/bench_s3express.yml @@ -93,6 +93,7 @@ jobs: auto-push: ${{ inputs.publish }} comment-on-alert: true max-items-in-chart: 20 + skip-fetch-gh-pages: true latency-bench: name: Benchmark (Latency)