Skip to content

Commit

Permalink
[CHORE] Fix CI caching to cache integration test builds separately (#…
Browse files Browse the repository at this point in the history
…1101)

Fixes our integration test build to use a separate (different) cache
from other builds, by changing the cache key

Driveby: Fix what seems to be a typo in the Rust test cache's
`restore-keys` argument as well

Co-authored-by: Jay Chia <jaychia94@gmail.com@users.noreply.github.com>
  • Loading branch information
jaychia and Jay Chia authored Jun 27, 2023
1 parent c08f6b2 commit a0932d3
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions .github/workflows/python-package.yml
Original file line number Diff line number Diff line change
Expand Up @@ -134,10 +134,10 @@ jobs:
path: |
~/.cargo/registry
~/.cargo/git
key: ${{ runner.os }}-build-${{ env.cache-name }}-${{ hashFiles('**/Cargo.lock') }}
key: ${{ runner.os }}-integration-build-${{ env.cache-name }}-${{ hashFiles('**/Cargo.lock') }}
restore-keys: |
${{ runner.os }}-build-${{ env.cache-name }}-
${{ runner.os }}-build-
${{ runner.os }}-integration-build-${{ env.cache-name }}-
${{ runner.os }}-integration-build-
${{ runner.os }}-
# NOTE: we don't build with all the actual release optimizations to avoid hellish CI times
- name: Build wheels
Expand Down Expand Up @@ -231,8 +231,8 @@ jobs:
~/.cargo/git
key: ${{ runner.os }}-rust-package-${{ env.cache-name }}-${{ hashFiles('**/Cargo.lock') }}
restore-keys: |
${{ runner.os }}-build-${{ env.cache-name }}-
${{ runner.os }}-build-
${{ runner.os }}-rust-package-${{ env.cache-name }}-
${{ runner.os }}-rust-package-
${{ runner.os }}-
- name: Install cargo-llvm-cov
uses: taiki-e/install-action@cargo-llvm-cov
Expand Down

0 comments on commit a0932d3

Please sign in to comment.