Closed andyzhangx closed 2 months ago
[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
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
pkg/nfs/nfs.go | 1 | 3 | 33.33% | ||
pkg/nfs/controllerserver.go | 11 | 20 | 55.0% | ||
<!-- | Total: | 12 | 23 | 52.17% | --> |
Totals | |
---|---|
Change from base Build 10641312236: | -0.5% |
Covered Lines: | 914 |
Relevant Lines: | 1170 |
What type of PR is this? /kind bug
What this PR does / why we need it: fix: delete volume error in archive deletion mode
when onDelete: archive, there could be delete volume error with error msg rename xxx: no such file or directory since there is a bug in csi-provisioner that DeleteVolume may be performed twice, this PR adds a 1 min cache and lock to workaround this issue.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?: