From cc679ef266c7da49e9339d69130f242d8285bc96 Mon Sep 17 00:00:00 2001 From: Ziwen Ning Date: Tue, 16 May 2023 13:59:41 -0700 Subject: [PATCH] ci: remove setup-go and explicitly set shell in benchmark (#401) Issue #, if available: *Description of changes:* Fix the workflow failure happened recently. https://github.com/runfinch/finch/actions/runs/4920343212/jobs/8789029862 *Testing done:* In feature branch. - [ X ] I've reviewed the guidance in CONTRIBUTING.md #### License Acceptance By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license. Signed-off-by: Ziwen Ning --- .github/workflows/benchmark.yaml | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/.github/workflows/benchmark.yaml b/.github/workflows/benchmark.yaml index 4537dcaa4..b03f4c7f8 100644 --- a/.github/workflows/benchmark.yaml +++ b/.github/workflows/benchmark.yaml @@ -34,7 +34,6 @@ jobs: # We need to get all the git tags to make version injection work. See VERSION in Makefile for more detail. fetch-depth: 0 submodules: true - - uses: actions/setup-go@v4 - name: Clean up previous files run: | sudo rm -rf /opt/finch @@ -49,12 +48,15 @@ jobs: - name: Install Rosetta 2 run: echo "A" | softwareupdate --install-rosetta || true - run: brew install go lz4 automake autoconf libtool + shell: zsh {0} - name: Build project run: | export PATH="/opt/homebrew/opt/libtool/libexec/gnubin:$PATH" make + shell: zsh {0} - name: Run benchmark run: make test-benchmark | tee benchmark.txt + shell: zsh {0} - name: Set OS info as env variable run: | echo "OS_VERSION=$(sw_vers -productVersion | cut -d '.' -f 1)" >> $GITHUB_ENV