Skip to content

Commit

Permalink
ci: disable Go cache (#1156)
Browse files Browse the repository at this point in the history
This change disables Go cache which is taking 5-7 minutes to create per
workflow.

Signed-off-by: Austin Vazquez <[email protected]>
  • Loading branch information
austinvazquez authored Oct 31, 2024
1 parent 6ef662f commit 9c72ad1
Show file tree
Hide file tree
Showing 5 changed files with 10 additions and 10 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/build-pkg.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ jobs:
- uses: actions/setup-go@41dfa10bad2bb2ae585af6ee5bb4d7d973ad74ed # v5.1.0
with:
go-version-file: go.mod
cache: true
cache: false
- name: Build for macOS ${{ inputs.version }} (${{ inputs.output_arch }})
run: |
brew install lz4 automake autoconf libtool yq llvm
Expand Down
10 changes: 5 additions & 5 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ jobs:
- uses: actions/setup-go@41dfa10bad2bb2ae585af6ee5bb4d7d973ad74ed # v5.1.0
with:
go-version-file: go.mod
cache: true
cache: false
- run: make gen-code
- run: git diff --exit-code
unit-tests:
Expand All @@ -94,7 +94,7 @@ jobs:
# Since this repository is not meant to be used as a library,
# we don't need to test the latest 2 major releases like Go does: https://go.dev/doc/devel/release#policy.
go-version-file: go.mod
cache: true
cache: false
- run: make test-unit
# It's recommended to run golangci-lint in a job separate from other jobs (go test, etc) because different jobs run in parallel.
go-linter:
Expand All @@ -105,7 +105,7 @@ jobs:
- uses: actions/setup-go@41dfa10bad2bb2ae585af6ee5bb4d7d973ad74ed # v5.1.0
with:
go-version-file: go.mod
cache: false # caching can result in tar errors that files already exist
cache: false
- name: set GOOS env to windows
run: |
echo "GOOS=windows" >> $GITHUB_ENV
Expand Down Expand Up @@ -145,7 +145,7 @@ jobs:
- uses: actions/setup-go@41dfa10bad2bb2ae585af6ee5bb4d7d973ad74ed # v5.1.0
with:
go-version-file: go.mod
cache: true
cache: false
# TODO: Use `go mod tidy --check` after https://github.com/golang/go/issues/27005 is fixed.
- run: go mod tidy
- run: git diff --exit-code
Expand All @@ -156,7 +156,7 @@ jobs:
- uses: actions/setup-go@41dfa10bad2bb2ae585af6ee5bb4d7d973ad74ed # v5.1.0
with:
go-version-file: go.mod
cache: true
cache: false
- run: make check-licenses
macos-e2e-tests:
strategy:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/e2e-macos.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ jobs:
- uses: actions/setup-go@41dfa10bad2bb2ae585af6ee5bb4d7d973ad74ed # v5.1.0
with:
go-version-file: go.mod
cache: true
cache: false
- name: Set output variables
id: vars
run: |
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/test-pkg.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ jobs:
- uses: actions/setup-go@41dfa10bad2bb2ae585af6ee5bb4d7d973ad74ed # v5.1.0
with:
go-version-file: go.mod
cache: true
cache: false
- name: Clean up previous files
run: |
sudo rm -rf /opt/finch
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/upload-build-to-S3.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ jobs:
- uses: actions/setup-go@41dfa10bad2bb2ae585af6ee5bb4d7d973ad74ed # v5.1.0
with:
go-version-file: go.mod
cache: true
cache: false
- name: Make macos aarch64 build
run: |
brew install lz4 automake autoconf libtool yq
Expand Down Expand Up @@ -53,7 +53,7 @@ jobs:
- uses: actions/setup-go@41dfa10bad2bb2ae585af6ee5bb4d7d973ad74ed # v5.1.0
with:
go-version-file: go.mod
cache: true
cache: false
- name: Make macos x86_64 build
run: |
brew install lz4 automake autoconf libtool yq
Expand Down

0 comments on commit 9c72ad1

Please sign in to comment.