yunify / qingcloud-csi

Kubernetes volume plugin based on CSI specification which support block storage of qingcloud
Apache License 2.0
36 stars 22 forks source link

Fix the wrong check of whether node expansion is required #197

Closed dkeven closed 2 years ago

dkeven commented 2 years ago

Signed-off-by: dkeven keven@kubesphere.io

What type of PR is this? /kind bug

What this PR does / why we need it: https://github.com/yunify/qingcloud-csi/pull/195 adds support for block volume. In the ControllerExpandVolume interface response, node expansion is required only when the volume mode is not block, which means the req.GetVolumeCapability().GetBlock() is nil. But this check is incorrectly set to the opposite.