add upstream support of JSON logging

[dmpe]

Is your feature request related to a problem? Please describe.

Following containers have JSON support build in. It just needs to be enabled in the helm chart via some if/else construct.
So my question is: could it be done ?

• csi-node-driver-registrar -> https://github.com/kubernetes-csi/node-driver-registrar/blob/release-2.12/CHANGELOG/CHANGELOG-2.11.md
• livenessprobe -> https://github.com/kubernetes-csi/livenessprobe/blob/release-2.14/CHANGELOG/CHANGELOG-2.13.md

and I also guess all images under https://github.com/rancher/vsphere-charts/blob/main/charts/rancher-vsphere-csi/values.yaml#L247

Example: csi attacher has added JSON logging in https://github.com/kubernetes-csi/external-attacher/blob/master/CHANGELOG/CHANGELOG-4.6.md version.

Describe the solution you'd like

I would like that all CSI containers log in JSON format, instead of default text format. It should also configurable in the helm chart.

Rancher GUI 2.9.2 + downstream cluster v1.30.4+rke2r1

[snasovich]

@dmpe , thank you for submitting this enhancement request.
I guess it should be possible by adding support to provide additional args to these containers via values.yaml (at a glance they should probably go here -

vsphere-charts/charts/rancher-vsphere-csi/values.yaml

Lines 189 to 200 in a5c99d7

	nodeDriverRegistrar:
	repository: rancher/mirrored-sig-storage-csi-node-driver-registrar
	tag: latest
	imagePullPolicy: ""
	resources: {}
	#resources:
	# limits:
	# cpu: 100m
	# memory: 256Mi
	# requests:
	# cpu: 50m
	# memory: 128Mi

for registrar and Helm template around

vsphere-charts/charts/rancher-vsphere-csi/templates/node/daemonset.yaml

Lines 72 to 75 in a5c99d7

	args:
	- "--v=5"
	- "--csi-address=$(ADDRESS)"
	- "--kubelet-registration-path=$(DRIVER_REG_SOCK_PATH)"

should be updated to use these additional args) - that's one of possible solutions. However, at this time this enhancement is not prioritized - but we welcome community contributions. ;)