Helm chart should default enableVolumeScheduling: true

[aaroniscode]

/kind bug

What happened?
Deployed the default helm chart and topology aware dynamic provisioning did not work. The kustomize based manifest default to true: https://github.com/kubernetes-sigs/aws-ebs-csi-driver/blob/master/deploy/kubernetes/values/controller.yaml#L4

What you expected to happen?
With the CSI Topology sub-feature at GA status, I would expect this feature gate to default to true.

Thoughts on this? Is there still a good reason to default the Helm chart to not use the Topology feature?

[ayberk]

Agreed we can default to true.

/good-first-issue

[k8s-ci-robot]

@ayberk:
This request has been marked as suitable for new contributors.

Please ensure the request meets the requirements listed here.

If this request no longer meets these requirements, the label can be removed
by commenting with the /remove-good-first-issue command.

In response to this:

Agreed we can default to true.

/good-first-issue

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.