Skip to content

Commit

Permalink
action: Explicitly set go-version
Browse files Browse the repository at this point in the history
Temporarily set go-version explicitly to avoid hitting this cache
restore issue [^1]. We can revert this when Go 1.24 gets released.

Also modify kind.yaml to install Cilium CLI using the local action.

[^1]: actions/setup-go#506

Signed-off-by: Michi Mutsuzaki <[email protected]>
  • Loading branch information
michi-covalent committed Nov 28, 2024
1 parent 5293e9d commit 53266ab
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 15 deletions.
22 changes: 8 additions & 14 deletions .github/workflows/kind.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -43,13 +43,10 @@ jobs:
# renovate: datasource=golang-version depName=go
go-version: 1.23.3

- name: Set up Go for root
run: |
sudo ln -sf `which go` `sudo which go` || true
sudo go version
- name: Build and install cilium CLI binary
run: sudo make install
- name: Install Cilium CLI
uses: ./
with:
local-path: '.'

- name: Create kind cluster
uses: helm/kind-action@0025e74a8c7512023d06dc019c617aa3cf561fde # v1.10.0
Expand Down Expand Up @@ -234,13 +231,10 @@ jobs:
# renovate: datasource=golang-version depName=go
go-version: 1.23.3

- name: Set up Go for root
run: |
sudo ln -sf `which go` `sudo which go` || true
sudo go version
- name: Build and install cilium CLI binary
run: sudo make install
- name: Install Cilium CLI
uses: ./
with:
local-path: '.'

- name: Create kind cluster 1
uses: helm/kind-action@0025e74a8c7512023d06dc019c617aa3cf561fde # v1.10.0
Expand Down
2 changes: 1 addition & 1 deletion action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ runs:
uses: actions/setup-go@41dfa10bad2bb2ae585af6ee5bb4d7d973ad74ed # v5.1.0
if: ${{ steps.build-cli.outputs.path != '' }}
with:
go-version-file: '${{ steps.build-cli.outputs.go-mod-path }}'
go-version: '1.23.3'
cache: true
cache-dependency-path: '${{ steps.build-cli.outputs.go-sum-path }}'

Expand Down

0 comments on commit 53266ab

Please sign in to comment.