Update sccache bucket
#289
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This PR updates the `sccache` configuration settings to use a new bucket, `rapids-sccache-east`. Unlike the previous `rapids-sccache` bucket, `rapids-sccache-east` resides in the same AWS region as the rest of our CI infrastructure (`us-east-2`). This should result in faster, more reliable `sccache` connections and will also help keep our data transfer costs down. **Important Note**: The changes from [this `sccache` PR](mozilla/sccache#1403) are required to use a bucket in `us-east-2`. These changes were incorporated in `sccache` `v0.3.2`, so you'll need that version or later (preferably the latest, `v0.3.3`). Please ensure that your CI images satisfy this constraint before merging this PR. Additionally, this PR updates the `SCCACHE_REGION` to the region of the bucket. This value is not meant to be a variable.
ajschmidt8
added
improvement
Adopt updated CI runner with sccache 0.3.3
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## branch-23.03 #289 +/- ##
=============================================
Coverage 72.95% 72.95%
=============================================
Files 381 381
Lines 13117 13117
Branches 993 992 -1
=============================================
Hits 9569 9569
Misses 3548 3548
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
dagardner-nv
approved these changes
Feb 14, 2023
|
/merge |
ghost
merged commit
This pull request was closed.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR updates the
sccacheconfiguration settings to use a new bucket,rapids-sccache-east.Unlike the previous
rapids-sccachebucket,rapids-sccache-eastresides in the same AWS region as the rest of our CI infrastructure (us-east-2).This should result in faster, more reliable
sccacheconnections and will also help keep our data transfer costs down.Important Note: The changes from this
sccachePR are required to use a bucket inus-east-2. These changes were incorporated insccachev0.3.2, so you'll need that version or later (preferably the latest,v0.3.3). Please ensure that your CI images satisfy this constraint before merging this PR.Additionally, this PR updates the
SCCACHE_REGIONto the region of the bucket. This value is not meant to be a variable.