Cherry-pick to 0.9: check path existence and lock in NodeUnpublishVolume (#3284) #3450
Conversation
…ve#3284) * check path existence and lock in NodeUnpublishVolume Signed-off-by: wangshulin <wangshulin@smail.nju.edu.cn> * use path level lock and lock for NodePublishVolume Signed-off-by: wangshulin <wangshulin@smail.nju.edu.cn> * small fix Signed-off-by: wangshulin <wangshulin@smail.nju.edu.cn> * return codes.aborted when the lock is not acquired Signed-off-by: wangshulin <wangshulin@smail.nju.edu.cn> * check target path validity Signed-off-by: wangshulin <wangshulin@smail.nju.edu.cn> * adjust order of check path and acquire lock Signed-off-by: wangshulin <wangshulin@smail.nju.edu.cn> --------- Signed-off-by: wangshulin <wangshulin@smail.nju.edu.cn>
|
Kudos, SonarCloud Quality Gate passed!
|
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cheyang 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 |
Codecov Report
@@ Coverage Diff @@
## release-0.9 #3450 +/- ##
===============================================
- Coverage 65.52% 65.49% -0.04%
===============================================
Files 397 398 +1
Lines 23107 23121 +14
===============================================
Hits 15142 15142
- Misses 6183 6197 +14
Partials 1782 1782
|
fluid-e2e-bot
bot
merged commit 76e2621
into
fluid-cloudnative:release-0.9
Ⅰ. Describe what this PR does
Cherry-pick for PR #3284
Ⅱ. Does this pull request fix one issue?
fixes #XXXX
Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews