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
feat: update cache support for starndard and vmss flex #2950
feat: update cache support for starndard and vmss flex #2950
Conversation
✅ Deploy Preview for kubernetes-sigs-cloud-provide-azure ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
/retest |
207d140
to
dd58e4d
Compare
/retest |
/retest |
2 similar comments
/retest |
/retest |
refactor add more logging refine logging refine vmss DeleteCacheForNode refactor DeleteCacheForNode fix DeleteCacheForNode issue vmclient fix update vmssflex cache fix vmssflex cache update refine updateCache in WaitForUpdateResult fix ut fix refine logging test: add unit test for updateCache func revert fix golint fix
92459bf
to
f91eb41
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: andyzhangx, ZeroMagic 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 |
return err | ||
} | ||
// get sync.Map cache and remove the node from the cache | ||
cacheKey := getVMSSVMCacheKey(node.resourceGroup, node.vmssName) | ||
ss.lockMap.LockEntry(cacheKey) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks this PR introduces a race condition in the unit test: https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/kubernetes-sigs_cloud-provider-azure/2984/pull-cloud-provider-azure-unit/1605470798791839744
What type of PR is this?
/kind feature
What this PR does / why we need it:
feat: update cache support for starndard and vmss flex
This PR is a preceding fix of #2608, also refines original update cache logic in VMSS
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: