-
Notifications
You must be signed in to change notification settings - Fork 16
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.
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
Ensure local builds can use CI cache #38
Comments
One additional change that will need to be made across repositories for this to work is to add this SCCACHE_NO_CREDENTIALS environment variable to the conda recipes' We should also check that the inclusion of that environment variable doesn't interfere with CI's regular use of My local testing suggests that, as long as that environment variable is not set in CI, it will not be passed through to the That environment variable is documented here: https://github.com/mozilla/sccache/blob/f8972dcc5570b9308847ea7c9b7c25e86e293fbf/docs/S3.md#L29. |
This Once that PR is merged and released, we can add a This way CI builds will not use the gha-tools/tools/rapids-env-update Lines 13 to 19 in 01c94b2
Then we can add the After these changes, local builds should be able to seamlessly use the |
I also updated our I've confirmed that enabling this setting allows local builds to use the |
mozilla/sccache#1724 has been merged and released in https://github.com/mozilla/sccache/releases/tag/v0.4.2. The next action items are: |
closing this issue now that all the PRs above are merged. |
The changes in #35 allow users to run CI test scripts locally by enabling the artifact transfer commands to work in a non-CI environment.
One thing that's still left to confirm is whether local builds are able to properly utilize the
sccache
that's been populated by CI.This workflow is useful for the following scenario:
The changes from #35 should enable this to be possible, but I haven't yet been able to verify it locally.
Opening this issue as a reminder to confirm this.
The text was updated successfully, but these errors were encountered: