Skip to content

Commit

Permalink
chore(ci): save cache on main at end of workflow (#17354)
Browse files Browse the repository at this point in the history
Seems like our caching was totally broken. We need to save the cache
after building and not before.

```
Warning: Path Validation Error: Path(s) specified in the action for caching do(es) not exist, hence no cache is being saved.
Warning: Cache save failed.
```
  • Loading branch information
dsherret committed Jan 11, 2023
1 parent b0e0e4f commit b0eaa0b
Show file tree
Hide file tree
Showing 2 changed files with 29 additions and 29 deletions.
36 changes: 18 additions & 18 deletions .github/workflows/ci.generate.ts
Original file line number Diff line number Diff line change
Expand Up @@ -361,26 +361,9 @@ const ci = {
"18-cargo-home-${{ matrix.os }}-${{ hashFiles('Cargo.lock') }}",
},
},
{
// In main branch, always creates fresh cache
name: "Cache build output (main)",
uses: "actions/cache/save@v3",
if:
"(matrix.profile == 'release' || matrix.profile == 'fastci') && github.ref == 'refs/heads/main'",
with: {
path: [
"./target",
"!./target/*/gn_out",
"!./target/*/*.zip",
"!./target/*/*.tar.gz",
].join("\n"),
key:
"18-cargo-target-${{ matrix.os }}-${{ matrix.profile }}-${{ github.sha }}",
},
},
{
// Restore cache from the latest 'main' branch build.
name: "Cache build output (PR)",
name: "Restore cache build output (PR)",
uses: "actions/cache/restore@v3",
if:
"github.ref != 'refs/heads/main' && !startsWith(github.ref, 'refs/tags/')",
Expand Down Expand Up @@ -790,6 +773,23 @@ const ci = {
draft: true,
},
},
{
// In main branch, always creates fresh cache
name: "Save cache build output (main)",
uses: "actions/cache/save@v3",
if:
"(matrix.profile == 'release' || matrix.profile == 'fastci') && github.ref == 'refs/heads/main'",
with: {
path: [
"./target",
"!./target/*/gn_out",
"!./target/*/*.zip",
"!./target/*/*.tar.gz",
].join("\n"),
key:
"18-cargo-target-${{ matrix.os }}-${{ matrix.profile }}-${{ github.sha }}",
},
},
]),
],
},
Expand Down
22 changes: 11 additions & 11 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -240,17 +240,7 @@ jobs:
~/.cargo/git/db
key: '18-cargo-home-${{ matrix.os }}-${{ hashFiles(''Cargo.lock'') }}'
if: steps.exit_early.outputs.EXIT_EARLY != 'true'
- name: Cache build output (main)
uses: actions/cache/save@v3
if: steps.exit_early.outputs.EXIT_EARLY != 'true' && ((matrix.profile == 'release' || matrix.profile == 'fastci') && github.ref == 'refs/heads/main')
with:
path: |-
./target
!./target/*/gn_out
!./target/*/*.zip
!./target/*/*.tar.gz
key: '18-cargo-target-${{ matrix.os }}-${{ matrix.profile }}-${{ github.sha }}'
- name: Cache build output (PR)
- name: Restore cache build output (PR)
uses: actions/cache/restore@v3
if: 'steps.exit_early.outputs.EXIT_EARLY != ''true'' && (github.ref != ''refs/heads/main'' && !startsWith(github.ref, ''refs/tags/''))'
with:
Expand Down Expand Up @@ -532,6 +522,16 @@ jobs:
target/release/lib.deno.d.ts
body_path: target/release/release-notes.md
draft: true
- name: Save cache build output (main)
uses: actions/cache/save@v3
if: steps.exit_early.outputs.EXIT_EARLY != 'true' && ((matrix.profile == 'release' || matrix.profile == 'fastci') && github.ref == 'refs/heads/main')
with:
path: |-
./target
!./target/*/gn_out
!./target/*/*.zip
!./target/*/*.tar.gz
key: '18-cargo-target-${{ matrix.os }}-${{ matrix.profile }}-${{ github.sha }}'
publish-canary:
name: publish canary
runs-on: ubuntu-20.04
Expand Down

0 comments on commit b0eaa0b

Please sign in to comment.