Skip to content

Commit

Permalink
CI: fix rust and go cache (#217)
Browse files Browse the repository at this point in the history
* rust-cache: add more target dirs
* Don't set custom TMPDIR

This seems to be the culprit for the extremely long path names that
are too long for IPC paths.
  • Loading branch information
sveitser committed Sep 3, 2024
1 parent 251b2bf commit b56dc56
Showing 1 changed file with 3 additions and 13 deletions.
16 changes: 3 additions & 13 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,14 +33,6 @@ jobs:
test-mode: [defaults, race, challenge, stylus, long, redis]

steps:
- name: Move repo to avoid too long file paths
run: |
mkdir /tmp/gha-work
cd /tmp/gha-work
rm -rf $GITHUB_WORKSPACE
# Link the original workspace to the new workspace directory
ln -sv /tmp/gha-work $GITHUB_WORKSPACE
- name: Checkout
uses: actions/checkout@v4
with:
Expand Down Expand Up @@ -137,11 +129,11 @@ jobs:
path: |
~/.cargo/registry/
~/.cargo/git/
arbitrator/target/
arbitrator/**/target/
arbitrator/wasm-libraries/target/
arbitrator/wasm-libraries/soft-float/SoftFloat/build
target/etc/initial-machine-cache/
key: v1-${{ runner.os }}-cargo-${{ steps.install-rust.outputs.rustc_hash }}-min-${{ hashFiles('arbitrator/Cargo.lock') }}-${{ matrix.test-mode }}
key: v2-${{ runner.os }}-cargo-${{ steps.install-rust.outputs.rustc_hash }}-min-${{ hashFiles('arbitrator/Cargo.lock') }}-${{ matrix.test-mode }}
restore-keys: v1-${{ runner.os }}-cargo-${{ steps.install-rust.outputs.rustc_hash }}-

- name: Cache cbrotli
Expand Down Expand Up @@ -188,8 +180,6 @@ jobs:
- name: Set environment variables
run: |
mkdir -p target/tmp/x
echo "TMPDIR=$(pwd)/target/tmp/x" >> "$GITHUB_ENV"
echo "GOMEMLIMIT=6GiB" >> "$GITHUB_ENV"
echo "GOGC=80" >> "$GITHUB_ENV"
Expand Down Expand Up @@ -245,7 +235,7 @@ jobs:

- name: Upload coverage to Codecov
uses: codecov/codecov-action@v2
if: matrix.test-mode == 'defaults'
if: false
with:
fail_ci_if_error: false
files: ./coverage.txt,./coverage-redis.txt
Expand Down

0 comments on commit b56dc56

Please sign in to comment.