spec issues

Maybe add c# namespace into the spec

3

c# users of the spec need to edit it to generate the source-code to include the csharp namespace. If the option is not allowed to be included then please remove...

Zetanova

Promote GetVolume and VolumeCondition to GA

2

This PR promotes volume health monitoring related capabilities, RPCs, fields to GA. Volume health monitoring is implemented in Kubernetes CSI External Health Monitor Controller and Kubelet. The feature on the...

xing-yang

Volume Lifecycle is not correspond to actual k8s behavior

8

Current CSI volume lifecycle is not designed for the case when `Node` is unreachable. When `Node` is shutdown or in a non-recoverable state such as hardware failure or broken OS,...

YuikoTakada

Dependency tree were found to be deprecated github.com/golang/protobuf

4

Is there any plan to instead `github.com/golang/protobuf` with `google.golang.org/protobuf`. FYI: https://pkg.go.dev/github.com/golang/protobuf/proto

vincentgoat

Add note to Volume Lifecycle figure

1

This PR adds note to Volume Lifecycle figure in `spec.md` in order to explain actual Kubernetes volume lifecycle behavior which has changed by non-graceful node shutdown feature of Kubernetes ....

YuikoTakada

Secrets feild addition to NodeGetVolumeStats Spec

14

At present, there is no `secret` field available on NodeGetVolumeStats RPC, however this is required at times for specific storage vendor to map volumeID to the `targetPath`. As per the...

humblec

add `secrets` field support for ControllerGetVolume Request

4

At present `secrets` are not part of this request and some consumers need this field for its operation.: That said, storage backends like Ceph Cluster require the access to be...

humblec

Need secrets in ControllerGetVolumeRequest{}

2

It would be nice to have secrets available in the ControllerGetVolumeRequest{} In our use case, we need secrets to access the volume to check the current information of a volume...

pkalever