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
fix: add lock to avoid concurrent update on the cache #4968
fix: add lock to avoid concurrent update on the cache #4968
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: andyzhangx 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 |
/retest |
Cloud object will be removed from csi drivers if we adopt track2 sdk. I'm wondering if there is any other cache implementation in csi driver? |
related: #4973 |
Can you add unit tests to cover the race condition? |
4b9b161
to
0f031f5
Compare
@nilo19 ut added, let's fix it with sdk v1 first, and then port the change to sdk v2 |
3b578df
to
7e1006d
Compare
/retest |
2 similar comments
/retest |
/retest |
|
/retest |
2 similar comments
/retest |
/retest |
lgtm, please don't forget to squash the commits. |
fix test: add parallel tests
cc75107
to
24417d7
Compare
@nilo19 done, there are two commits now: one bug fix and one ut commit. |
/retest |
5 similar comments
/retest |
/retest |
/retest |
/retest |
/retest |
/retest |
/retest job should be fixed now |
@andyzhangx: The following test failed, say
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
/retest |
What type of PR is this?
/kind bug
What this PR does / why we need it:
fix: add lock to avoid concurrent update on the cache
find more details here: kubernetes-sigs/azurefile-csi-driver#1556
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.: