Skip to content

Commit

Permalink
chore: don't use cashapp/activate-hermit's native caching
Browse files Browse the repository at this point in the history
We only write to the cache on main.
  • Loading branch information
alecthomas committed Nov 1, 2024
1 parent 21d6bf3 commit 844b112
Show file tree
Hide file tree
Showing 4 changed files with 36 additions and 16 deletions.
15 changes: 14 additions & 1 deletion .github/actions/build-cache/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,20 @@ runs:
- id: find-go-build-cache
shell: bash
run: echo "cache=$(go env GOCACHE)" >> $GITHUB_OUTPUT
- name: Cache Docker Images
- id: bin-hash
shell: bash
run: |
hash="$(find ./${{ inputs.working-directory }}/bin ! -type d | sort | xargs openssl sha256 | openssl sha256 -r | cut -d' ' -f1)"
echo "hash=$hash" >> "$GITHUB_OUTPUT"
- name: Restore Hermit Cache
uses: actions/cache/restore@v4
with:
path: ${{ runner.os == 'macOS' && '~/Library/Caches/hermit/pkg' || '~/.cache/hermit/pkg' }}
key: ${{ runner.os }}-hermit-cache-${{ steps.bin-hash.outputs.hash }}
restore-keys: |
${{ runner.os }}-hermit-cache-${{ steps.bin-hash.outputs.hash }}
${{ runner.os }}-hermit-cache-
- name: Restore Docker Images
uses: ScribeMD/docker-cache@0.5.0
with:
key: docker-${{ runner.os }}-${{ hashFiles('docker-compose.yml') }}
Expand Down
6 changes: 0 additions & 6 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -170,8 +170,6 @@ jobs:
uses: actions/checkout@v4
- name: Init Hermit
uses: cashapp/activate-hermit@v1.1.3
with:
cache: true
- name: Build Cache
uses: ./.github/actions/build-cache
- name: Install Java
Expand Down Expand Up @@ -303,8 +301,6 @@ jobs:
uses: actions/checkout@v4
- name: Init Hermit
uses: cashapp/activate-hermit@v1.1.3
with:
cache: true
- name: Build Cache
uses: ./.github/actions/build-cache
- name: Docker Compose
Expand Down Expand Up @@ -350,8 +346,6 @@ jobs:
uses: actions/checkout@v4
- name: Init Hermit
uses: cashapp/activate-hermit@v1.1.3
with:
cache: true
- name: Build Cache
uses: ./.github/actions/build-cache
- name: Docker Compose
Expand Down
14 changes: 6 additions & 8 deletions .github/workflows/smoketest.yml
Original file line number Diff line number Diff line change
Expand Up @@ -56,17 +56,15 @@ jobs:
rm -r ./tmpsmoketest
- name: Init Hermit
uses: cashapp/activate-hermit@v1
with:
cache: true
- name: Build Cache
uses: ./.github/actions/build-cache
- name: Download Go Modules
run: go mod download
- name: Set up a kube cluster with the tagged release dockerhub image
run: |
set -euo pipefail
echo "Deploying the tagged release to the cluster"
cd deployment && just deploy-version ${{ env.LATEST_VERSION }} && cd ..
set -euo pipefail
echo "Deploying the tagged release to the cluster"
cd deployment && just deploy-version ${{ env.LATEST_VERSION }} && cd ..
# We skip this since it requires an integration test change to skip the full kube deploy.
# Re-enable this step when the tagged release has the integration test change.
# - name: Smoke test the tagged release images
Expand All @@ -81,9 +79,9 @@ jobs:
fetch-depth: 1
- name: Smoke test HEAD with a full deploy to test upgrade path
run: |
set -euo pipefail
echo "Running smoke test on the HEAD images"
USE_DB_CONFIG=true go test -v -timeout 15m -tags smoketest -run '^Test' ./smoketest
set -euo pipefail
echo "Running smoke test on the HEAD images"
USE_DB_CONFIG=true go test -v -timeout 15m -tags smoketest -run '^Test' ./smoketest
- name: Archive Report
uses: actions/upload-artifact@v4
if: always()
Expand Down
17 changes: 16 additions & 1 deletion .github/workflows/writecache.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,8 @@ on:
push:
branches:
- main
- aat/hermit-cache-fix
workflow_dispatch:
concurrency:
group: ${{ github.ref }}-writecache
cancel-in-progress: true
Expand All @@ -15,6 +17,8 @@ jobs:
uses: actions/checkout@v4
- name: Init Hermit
uses: cashapp/activate-hermit@v1.1.3
- name: Install Hermit Packages
run: hermit install
- name: Docker Compose
run: docker compose up -d --wait
- name: Init DB
Expand All @@ -28,6 +32,17 @@ jobs:
- id: find-go-build-cache
shell: bash
run: echo "cache=$(go env GOCACHE)" >> "$GITHUB_OUTPUT"
- id: bin-hash
shell: bash
working-directory: .
run: |

Check failure on line 38 in .github/workflows/writecache.yml

View workflow job for this annotation

GitHub Actions / Lint

property "working-directory" is not defined in object type {} [expression]
hash="$(find ./${{ inputs.working-directory }}/bin ! -type d | sort | xargs openssl sha256 | openssl sha256 -r | cut -d' ' -f1)"
echo "hash=$hash" >> "$GITHUB_OUTPUT"
- name: Save Hermit Cache
uses: actions/cache/save@v4
with:
path: ${{ runner.os == 'macOS' && '~/Library/Caches/hermit/pkg' || '~/.cache/hermit/pkg' }}
key: ${{ runner.os }}-hermit-cache-${{ steps.bin-hash.outputs.hash }}
- name: Save Go Module Cache
uses: actions/cache/save@v4
with:
Expand Down Expand Up @@ -56,4 +71,4 @@ jobs:
- name: Save Docker Images
uses: ScribeMD/docker-cache@0.5.0
with:
key: docker-${{ runner.os }}-${{ hashFiles('docker-compose.yml') }}
key: docker-${{ runner.os }}-${{ hashFiles('docker-compose.yml') }}

0 comments on commit 844b112

Please sign in to comment.