Skip to content

Cache guest binaries #1077

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
ludfjig wants to merge 1 commit into from
Closed

Cache guest binaries #1077

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
24 changes: 24 additions & 0 deletions .github/workflows/dep_rust.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -121,6 +121,30 @@ jobs:
run: (Get-Service actions.runner.*) | Foreach { $_.Name, $_.UserName, $_.ServiceType }
shell: pwsh

# If exact key match is found, it will be restored and the below `just build-rust-guests` step will be much faster
# If a partial key match is found (restore-keys), it will be restored but `just build-rust-guests` will perform an incremental build
# If neither, `just build-rust-guests` will perform a full build
- name: Cache Guest Binaries
id: cache-guests
uses: actions/cache@v4
with:
path: |
src/tests/rust_guests/simpleguest/target/x86_64-unknown-none/${{ inputs.config }}
src/tests/rust_guests/dummyguest/target/x86_64-unknown-none/${{ inputs.config }}
src/tests/rust_guests/witguest/target/x86_64-unknown-none/${{ inputs.config }}
src/tests/rust_guests/simpleguest/target/${{ inputs.config }}
src/tests/rust_guests/dummyguest/target/${{ inputs.config }}
src/tests/rust_guests/witguest/target/${{ inputs.config }}
src/tests/c_guests/c_simpleguest/out/${{ inputs.config }}
key: guest-binaries-${{ runner.os }}-${{ inputs.config }}-${{ hashFiles('src/tests/rust_guests/**/*', 'src/tests/c_guests/**/*') }} # cargo.lock is not sufficent as it is not always up to date due to dependabot PRs
restore-keys: |
guest-binaries-${{ runner.os }}-${{ inputs.config }}-
enableCrossOsArchive: false # binaries are compatible but build artifacts contain absolute paths

- name: Display cache hit info
run: |
echo "Cache hit: ${{ steps.cache-guests.outputs.cache-hit }}"

- name: Build and move Rust guests
run: |
# use these commands in favor of build-and-move-rust-guests to avoid building both configs
Expand Down
6 changes: 5 additions & 1 deletion Justfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -59,15 +59,19 @@ build-rust-guests target=default-target features="": (witguest-wit)
build-and-move-rust-guests: (build-rust-guests "debug") (move-rust-guests "debug") (build-rust-guests "release") (move-rust-guests "release")
build-and-move-c-guests: (build-c-guests "debug") (move-c-guests "debug") (build-c-guests "release") (move-c-guests "release")

clean: clean-rust
clean: clean-rust clean-c

clean-rust:
cargo clean
cd src/tests/rust_guests/simpleguest && cargo clean
cd src/tests/rust_guests/dummyguest && cargo clean
{{ if os() == "windows" { "cd src/tests/rust_guests/witguest -ErrorAction SilentlyContinue; cargo clean" } else { "[ -d src/tests/rust_guests/witguest ] && cd src/tests/rust_guests/witguest && cargo clean || true" } }}
{{ if os() == "windows" { "Remove-Item src/tests/rust_guests/witguest/interface.wasm -Force -ErrorAction SilentlyContinue" } else { "rm -f src/tests/rust_guests/witguest/interface.wasm" } }}

clean-c:
git clean -fdx src/tests/c_guests/bin src/tests/rust_guests/bin
git clean -fdx src/tests/c_guests/c_simpleguest/out src/tests/c_guests/c_callbackguest/out


################
### TESTING ####
Expand Down
Loading