-
Notifications
You must be signed in to change notification settings - Fork 36
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
Fixup cache key for projects #425
Fixup cache key for projects #425
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: vishesh92 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
✅ Deploy Preview for kubernetes-sigs-cluster-api-cloudstack ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
/run-e2e -c 4.20 |
@vishesh92 a jenkins job has been kicked to run test with following paramaters:
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #425 +/- ##
==========================================
- Coverage 25.66% 25.61% -0.06%
==========================================
Files 59 69 +10
Lines 5563 6699 +1136
==========================================
+ Hits 1428 1716 +288
- Misses 3996 4824 +828
- Partials 139 159 +20 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Test Results : (tid-633)
|
Test Results : (tid-635)
|
/lgtm |
Issue #, if available:
Description of changes:
This pull request includes changes to the
pkg/cloud/client.go
file to improve the client caching mechanism by incorporating the project identifier into the cache key generation.Improvements to client caching:
pkg/cloud/client.go
: Modified thegenerateClientCacheKey
function to include theproject
parameter, ensuring that the cache key is unique per project.pkg/cloud/client.go
: Updated theNewClientFromConf
function to pass theproject
parameter to thegenerateClientCacheKey
function.Testing performed:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.