cinder-csi: consider using ClusterFirstWithHostNet DNSPolicy

[rptaylor]

Is this a BUG REPORT or FEATURE REQUEST?:

/kind feature

What happened:
Cinder CSI is using ClusterFirst DNSPolicy, but also appears to use hostNetwork.
Following up from #1387 (comment) , it seems like using ClusterFirstWithHostNet could be better but it's not entirely clear to me. FYI @jichenjc

Anything else we need to know?:
See https://kubernetes.io/docs/concepts/services-networking/dns-pod-service/#pod-s-dns-policy

Environment:

• openstack-cloud-controller-manager(or other related binary) version:
• OpenStack version:
• Others:

[dulek]

Seems like the chart has this configurable [1] and the default is in fact ClusterFirstWithHostNet [2]. This got fixed in #2483. What am I missing?

[1]

cloud-provider-openstack/charts/cinder-csi-plugin/templates/nodeplugin-daemonset.yaml

Line 27 in 8a156e5

dnsPolicy: {{ .Values.csi.plugin.nodePlugin.dnsPolicy }}

[2]

cloud-provider-openstack/charts/cinder-csi-plugin/values.yaml

Line 72 in d64cd51

dnsPolicy: ClusterFirstWithHostNet

[rptaylor]

Oh, good to see it was already discussed. I guess the recent comment #1387 (comment) mislead us then.