Skip to content

Commit

Permalink
Reduce cache thrashing on E2E vagrant boxes. Don't cache for windows …
Browse files Browse the repository at this point in the history
…unit tests. (#11769)

Signed-off-by: Derek Nola <[email protected]>
  • Loading branch information
dereknola authored Feb 11, 2025
1 parent 45dbc43 commit 303350d
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 6 deletions.
6 changes: 3 additions & 3 deletions .github/actions/setup-go/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,18 +6,18 @@ runs:
- uses: actions/setup-go@v5
with:
go-version-file: 'go.mod' # Just use whatever version is in the go.mod file
cache: ${{ github.ref == 'refs/heads/master' || github.ref == 'refs/heads/release-1.29' }}
cache: ${{ github.ref == 'refs/heads/master' || github.ref == 'refs/heads/release-1.32' }}

- name: Prepare for go cache
if: ${{ github.ref != 'refs/heads/master' && github.ref != 'refs/heads/release-1.29' }}
if: github.ref != 'refs/heads/master' && github.ref != 'refs/heads/release-1.32'
shell: bash
run: |
echo "GO_CACHE=$(go env GOCACHE)" | tee -a "$GITHUB_ENV"
echo "GO_MODCACHE=$(go env GOMODCACHE)" | tee -a "$GITHUB_ENV"
echo "GO_VERSION=$(go env GOVERSION | tr -d 'go')" | tee -a "$GITHUB_ENV"
- name: Setup read-only cache
if: ${{ github.ref != 'refs/heads/master' && github.ref != 'refs/heads/release-1.29' }}
if: github.ref != 'refs/heads/master' && github.ref != 'refs/heads/release-1.32'
uses: actions/cache/restore@v4
with:
path: |
Expand Down
13 changes: 10 additions & 3 deletions .github/workflows/e2e.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -54,15 +54,22 @@ jobs:

- name: Set up vagrant and libvirt
uses: ./.github/actions/vagrant-setup
- name: "Vagrant Cache"
- name: Vagrant R/W Cache
if: matrix.etest != 'btrfs' && github.ref == 'refs/heads/master'
uses: actions/cache@v4
with:
path: |
~/.vagrant.d/boxes
key: vagrant-box-ubuntu-2404
key: ${{ matrix.etest != 'btrfs' && 'vagrant-box-ubuntu-2404' || 'vagrant-box-leap' }}
- name: Vagrant Read Cache
if: matrix.etest != 'btrfs' && github.ref != 'refs/heads/master'
uses: actions/cache/restore@v4
with:
path: |
~/.vagrant.d/boxes
key: ${{ matrix.etest != 'btrfs' && 'vagrant-box-ubuntu-2404' || 'vagrant-box-leap' }}
- name: "Vagrant Plugin(s)"
run: vagrant plugin install vagrant-k3s vagrant-reload vagrant-scp

- name: Install Go
uses: actions/setup-go@v5
with:
Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/unitcoverage.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,8 @@ jobs:
fetch-depth: 1
- name: Install Go
uses: actions/setup-go@v5
with:
cache: false
- name: Run Unit Tests
run: |
go test -coverpkg ./pkg/... -coverprofile coverage.out ./pkg/... -run Unit
Expand Down

0 comments on commit 303350d

Please sign in to comment.