community icon indicating copy to clipboard operation
community copied to clipboard

[CSI]DeleteVolumeRequest:A parameter must be added to transfer the PV name of the volume to be deleted

Open redriverhong opened this issue 1 year ago • 5 comments

Describe the issue

Is your feature request related to a problem?/Why is this needed Currently, the DeleteVolumeRequest is only one parameter VolumeId, which is used to transfer the volume to be deleted. However, this requirement cannot be met in the scenario where PV volumes are preset.

image

for example: apiVersion: v1 kind: PersistentVolume metadata: annotations: pv.kubernetes.io/provisioned-by: com.localcsi pv.kubernetes.io/create-by: manual // Identifier Manually Preset name: pv-test-0 spec: capacity: storage: 256Mi accessModes:

ReadWriteOnce claimRef: apiVersion: v1 kind: PersistentVolumeClaim name: data-test-0 namespace: default persistentVolumeReclaimPolicy: Retain volumeMode: Filesystem storageClassName: local csi: driver: com.localcsi fsType: ext4 volumeHandle: caasvg_lv // volumeId nodeAffinity: required: nodeSelectorTerms: matchExpressions: key: kubernetes.io/hostname operator: In values: master1 apiVersion: v1 kind: PersistentVolume metadata: annotations: pv.kubernetes.io/provisioned-by: com.localcsi pv.kubernetes.io/create-by: manual // Identifier Manually Preset name: pv-test-1 spec: capacity: storage: 256Mi accessModes:

ReadWriteOnce claimRef: apiVersion: v1 kind: PersistentVolumeClaim name: data-test-1 namespace: default persistentVolumeReclaimPolicy: Retain volumeMode: Filesystem storageClassName: local csi: driver: com.localcsi fsType: ext4 volumeHandle: caasvg_lv // volumeId nodeAffinity: required: nodeSelectorTerms: matchExpressions: key: kubernetes.io/hostname operator: In values: master2 For the preceding two PVs, if I want to delete the first PV, only one volumeId is sent to me when I invoke the volume deletion interface. but the volumeId of the two PVs is the same and the volume of the node to be deleted cannot be identified.

So raised this issue to add a parameter pvName in DeleteVolumeRequest.

Describe the solution you'd like in detail image

redriverhong avatar Dec 06 '23 06:12 redriverhong