csm icon indicating copy to clipboard operation
csm copied to clipboard

[FEATURE]: CSI-PowerScale DeleteVolume improvements

Open santhoshatdell opened this issue 1 year ago • 1 comments

Describe the solution you'd like Based on what I noted with certain environments, it would be better if we can correct the below items in the DeleteVolume() function.

Current DeleteVolume() implementation does

  1. If there is no NFS export with expected ID available at that time, the function simply returns success. - We should still continue and delete the directory if there are no other exports.
  2. Delete Quota by retrieving quota ID from export’s description. - May I know what was the reason we had to depend on the description property of NFS export for Quota ID? We could check for quota on a directory path and use that ID to delete.

Without these changes, the PV may get stuck in Terminating state.

santhoshatdell avatar Apr 28 '23 15:04 santhoshatdell

@santhoshatdell: Thank you for submitting this issue!

The issue is currently awaiting triage. Please make sure you have given us as much context as possible.

If the maintainers determine this is a relevant issue, they will remove the needs-triage label and assign an appropriate priority label.


We want your feedback! If you have any questions or suggestions regarding our contributing process/workflow, please reach out to us at [email protected].

csmbot avatar Apr 28 '23 15:04 csmbot