-
Notifications
You must be signed in to change notification settings - Fork 118
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
Remove CSI NodeUnpublish lstat and attachment check #1379
Remove CSI NodeUnpublish lstat and attachment check #1379
Conversation
Signed-off-by: Grant Griffiths <grant@portworx.com>
Testing this change now. |
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
codes.Internal, | ||
"Unknown error while verifying target location %s: %s", | ||
// Mount volume onto the path | ||
if err = s.driver.Unmount(req.GetVolumeId(), req.GetTargetPath(), nil); err != nil { |
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.
Need to check if the that this call is idempotent
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.
discussed this on the call, it is idempotent with the sched filter
3970ea0
to
f258852
Compare
This check was originally in place due to container-storage-interface/spec#285, which appears to be fixed.
We also use the file info to check for if a path is a block path or not. We may need to perform a different check when we implement CSI Raw block.