Skip to content

Commit

Permalink
ci: remove setup-go and explicitly set shell in benchmark (#401)
Browse files Browse the repository at this point in the history
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 <[email protected]>
  • Loading branch information
ningziwen authored May 16, 2023
1 parent 0dfeaa9 commit cc679ef
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion .github/workflows/benchmark.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand All @@ -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
Expand Down

0 comments on commit cc679ef

Please sign in to comment.