Mock ControllerGetVolume/NodeGetVolumeStats/ListVolumes RPCs to test controller/agent here

[NickrenREN]

No description provided.

[NickrenREN]

/assign @fengzixu

[k8s-ci-robot]

@NickrenREN: GitHub didn't allow me to assign the following users: fengzixu.

Note that only kubernetes-csi members, repo collaborators and people who have commented on this issue/PR can be assigned. Additionally, issues/PRs can only have 10 assignees at the same time.
For more information please see the contributor guide

In response to this:

/assign @fengzixu

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[fengzixu]

Please assign to me

[NickrenREN]

@fengzixu You may need to add support in csi-test repo for volume health monitor.
Example: kubernetes-csi/csi-test#160

[fengzixu]

Implementing

[fengzixu]

When I resolved this issue, I found the GET_VOLUME_HEALTH capability doesn't exist in spec repository. I think it was replaced by GET_VOLUME_CONDITION. Right?

• KEP: https://github.com/kubernetes/enhancements/pull/1077/files#diff-077b91d1804a891f8b79c30eeaa4964fR187
• Spec: https://github.com/container-storage-interface/spec/pull/415/files#diff-41f1638f086beb93f65a22cf27bb8be2R219

And, I found the ListVolumes RPC interface has been mocked. https://github.com/kubernetes-csi/csi-test/blob/613e9285fbb8552aae5827286600146dff19db62/mock/service/controller.go#L329

[fengzixu]

Not sure if this PR can satisfy our mock requirement. Please check: kubernetes-csi/csi-test#268

[xing-yang]

Yes, it is GET_VOLUME_CONDITION.

ListVolumes is an existing RPC. We just added volume_condition in its response.

[xing-yang]

You can take a look of this PR to see what changes are added to CSI spec to support volume health: container-storage-interface/spec#415