-
Notifications
You must be signed in to change notification settings - Fork 552
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
refactor: Migrate gcs cache to opendal based #1474
Conversation
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
Codecov ReportBase: 29.86% // Head: 29.98% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #1474 +/- ##
==========================================
+ Coverage 29.86% 29.98% +0.12%
==========================================
Files 47 47
Lines 16974 16572 -402
Branches 8056 7827 -229
==========================================
- Hits 5069 4969 -100
+ Misses 6503 6349 -154
+ Partials 5402 5254 -148
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo github@xuanwo.io
This PR will introduce the following user face changes:
SCCACHE_GCS_SERVICE_ACCOUNT
to allow the user to specify the service account to fetch tokens from VM metadata.SCCACHE_GCS_CREDENTIALS_URL
andSCCACHE_GCS_OAUTH_URL
; hence we will build the correct URL via the service account instead.And, yes, remove a lot of code 🚀