disk: verify VolumeCapabilities

[huww98]

What type of PR is this?

/kind api-change
/kind feature

What this PR does / why we need it:

Reject CreateVolume if capability cannot be supported: MULTI_NODE_MULTI_WRITER
and MULTI_NODE_SINGLE_WRITER for mount volumes.

Require MultiAttach to be enabled in parameters if multi-node access is requested.
We do not enable it automatically because multi-attach implies additional limits,
which may be unexpected to user.

Also implement the ValidateVolumeCapabilities controller RPC call.

Which issue(s) this PR fixes:

Fixes CSI sanity case:

• controller service [controller server] validatevolumecapabilities [it] should fail when the requested volume does not exist

Special notes for your reviewer:

PVC with volumeMode: Filesystem and accessModes: [ ReadWriteMany ] will fail with this event:

Warning ProvisioningFailed 2s (x3 over 3s) diskplugin.csi.alibabacloud.com_csi-provisioner-647f8bccfb-w22zq_d72ba154-98bf-4c6d-b276-8cb0120fcbbc failed to provision volume with StorageClass "csi-disk-essd-immediate": rpc error: code = InvalidArgument desc = Invalid parameters from input: disk-ebddaed2-f256-4d6b-8855-785a1072fc11, with error: multi-node writing is only supported for block volume

Does this PR introduce a user-facing change?

action required. New mount volume cannot be created if it has ReadWriteMany access mode. multiAttach is now required if other multi-node access is requested in access mode.

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:

[k8s-ci-robot]

Hi @huww98. Thanks for your PR.

I'm waiting for a kubernetes-sigs member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

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.

[huww98]

I've modified this PR to NOT enable multi-attach automatically. Because that implies additional limits.

[huww98]

rebased for csi-sanity
/ok-to-test

[huww98]

/auto-cc

[mowangdk]

/lgtm
/approve

Add e2e test later

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: huww98, mowangdk

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [mowangdk]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment