Skip to content

Fix native binary cache misses on Depot runners - #1205

Merged
theoephraim merged 1 commit into
mainfrom
native-bin-cache-on-github
Oct 13, 2026
Merged

theoephraim merged 1 commit into
mainfrom
native-bin-cache-on-github

Conversation

@theoephraim

Copy link
Copy Markdown
Member

Since #1184, every PR that releases varlock rebuilds (and signs) the macOS app and the Rust helpers on every push.

The native build jobs run on GitHub runners and save the binaries to GitHub's Actions cache. The cache lookups in build-and-test and the restores in release-preview-packages now run on Depot runners, and Depot redirects actions/cache to Depot Cache (this redirect isn't affected by the org's automatic cache connection setting), so they always miss.

  • The lookup-only checks move into a small native-cache-check job on ubuntu-latest.
  • release-preview-packages moves back to ubuntu-latest so it restores binaries from GitHub's cache. This also keeps the binaries in published preview packages out of Depot's cache, which has no branch isolation.

@pullfrog pullfrog Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ No new issues found.

Reviewed changes: Reviewed the native cache lookup extraction, dependent build conditions, and preview runner change in .github/workflows/test.yaml against the existing cache producers and Depot's documented routing behavior.

  • GitHub-hosted cache checks: Moves lookup-only checks into native-cache-check, preserving source-change gating and the shared Rust source hash.
  • Native build dependencies: Updates both native build callers to consume the new job's cache-hit outputs.
  • Preview cache restores: Runs preview publishing on GitHub-hosted Ubuntu while retaining the separate Depot Turbo cache configuration.

CI passed, but the native cache and preview jobs were skipped on this workflow-only PR, so the live cache-hit and publication paths remain untested here.

Pullfrog  | View workflow run | Using gpt-6.1-sol | 𝕏

@theoephraim
theoephraim merged commit 1c65e21 into main Oct 13, 2026
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant