helm: make log level configurable

instead of keeping the log level at 5, which
is required only for tracing the errors. this commit
adds an option for users to configure the log level
for all containers.

Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
This commit is contained in:
Madhu Rajanna 2020-10-20 12:19:10 +05:30 committed by mergify[bot]
parent 70358c8eb7
commit 551a5018d0
6 changed files with 24 additions and 14 deletions

View File

@ -41,7 +41,7 @@ spec:
image: "{{ .Values.nodeplugin.registrar.image.repository }}:{{ .Values.nodeplugin.registrar.image.tag }}" image: "{{ .Values.nodeplugin.registrar.image.repository }}:{{ .Values.nodeplugin.registrar.image.tag }}"
imagePullPolicy: {{ .Values.nodeplugin.registrar.image.pullPolicy }} imagePullPolicy: {{ .Values.nodeplugin.registrar.image.pullPolicy }}
args: args:
- "--v=5" - "--v={{ .Values.logLevel }}"
- "--csi-address=/csi/{{ .Values.pluginSocketFile }}" - "--csi-address=/csi/{{ .Values.pluginSocketFile }}"
- "--kubelet-registration-path={{ .Values.socketDir }}/{{ .Values.pluginSocketFile }}" - "--kubelet-registration-path={{ .Values.socketDir }}/{{ .Values.pluginSocketFile }}"
env: env:
@ -68,7 +68,7 @@ spec:
- "--forcecephkernelclient={{ .Values.nodeplugin.forcecephkernelclient }}" - "--forcecephkernelclient={{ .Values.nodeplugin.forcecephkernelclient }}"
{{- end }} {{- end }}
- "--endpoint=$(CSI_ENDPOINT)" - "--endpoint=$(CSI_ENDPOINT)"
- "--v=5" - "--v={{ .Values.logLevel }}"
- "--drivername=$(DRIVER_NAME)" - "--drivername=$(DRIVER_NAME)"
{{- if .Values.topology.enabled }} {{- if .Values.topology.enabled }}
- "--domainlabels={{ .Values.topology.domainLabels | join "," }}" - "--domainlabels={{ .Values.topology.domainLabels | join "," }}"

View File

@ -48,7 +48,7 @@ spec:
imagePullPolicy: {{ .Values.provisioner.provisioner.image.pullPolicy }} imagePullPolicy: {{ .Values.provisioner.provisioner.image.pullPolicy }}
args: args:
- "--csi-address=$(ADDRESS)" - "--csi-address=$(ADDRESS)"
- "--v=5" - "--v={{ .Values.logLevel }}"
- "--timeout={{ .Values.provisioner.timeout }}" - "--timeout={{ .Values.provisioner.timeout }}"
- "--enable-leader-election=true" - "--enable-leader-election=true"
- "--leader-election-type=leases" - "--leader-election-type=leases"
@ -69,7 +69,7 @@ spec:
imagePullPolicy: {{ .Values.provisioner.snapshotter.image.pullPolicy }} imagePullPolicy: {{ .Values.provisioner.snapshotter.image.pullPolicy }}
args: args:
- "--csi-address=$(ADDRESS)" - "--csi-address=$(ADDRESS)"
- "--v=5" - "--v={{ .Values.logLevel }}"
- "--timeout={{ .Values.provisioner.timeout }}" - "--timeout={{ .Values.provisioner.timeout }}"
- "--leader-election=true" - "--leader-election=true"
env: env:
@ -87,7 +87,7 @@ spec:
image: "{{ .Values.provisioner.attacher.image.repository }}:{{ .Values.provisioner.attacher.image.tag }}" image: "{{ .Values.provisioner.attacher.image.repository }}:{{ .Values.provisioner.attacher.image.tag }}"
imagePullPolicy: {{ .Values.provisioner.attacher.image.pullPolicy }} imagePullPolicy: {{ .Values.provisioner.attacher.image.pullPolicy }}
args: args:
- "--v=5" - "--v={{ .Values.logLevel }}"
- "--csi-address=$(ADDRESS)" - "--csi-address=$(ADDRESS)"
- "--leader-election=true" - "--leader-election=true"
- "--retry-interval-start=500ms" - "--retry-interval-start=500ms"
@ -106,7 +106,7 @@ spec:
image: "{{ .Values.provisioner.resizer.image.repository }}:{{ .Values.provisioner.resizer.image.tag }}" image: "{{ .Values.provisioner.resizer.image.repository }}:{{ .Values.provisioner.resizer.image.tag }}"
imagePullPolicy: {{ .Values.provisioner.resizer.image.pullPolicy }} imagePullPolicy: {{ .Values.provisioner.resizer.image.pullPolicy }}
args: args:
- "--v=5" - "--v={{ .Values.logLevel }}"
- "--csi-address=$(ADDRESS)" - "--csi-address=$(ADDRESS)"
- "--csiTimeout={{ .Values.provisioner.timeout }}" - "--csiTimeout={{ .Values.provisioner.timeout }}"
- "--leader-election" - "--leader-election"
@ -130,7 +130,7 @@ spec:
- "--controllerserver=true" - "--controllerserver=true"
- "--pidlimit=-1" - "--pidlimit=-1"
- "--endpoint=$(CSI_ENDPOINT)" - "--endpoint=$(CSI_ENDPOINT)"
- "--v=5" - "--v={{ .Values.logLevel }}"
- "--drivername=$(DRIVER_NAME)" - "--drivername=$(DRIVER_NAME)"
env: env:
- name: POD_IP - name: POD_IP

View File

@ -29,6 +29,11 @@ serviceAccounts:
# subvolumeGroup: "csi" # subvolumeGroup: "csi"
csiConfig: [] csiConfig: []
# Set logging level for csi containers.
# Supported values from 0 to 5. 0 for general useful logs,
# 5 for trace level verbosity.
logLevel: 5
nodeplugin: nodeplugin:
name: nodeplugin name: nodeplugin
# if you are using ceph-fuse client set this value to OnDelete # if you are using ceph-fuse client set this value to OnDelete

View File

@ -42,7 +42,7 @@ spec:
image: "{{ .Values.nodeplugin.registrar.image.repository }}:{{ .Values.nodeplugin.registrar.image.tag }}" image: "{{ .Values.nodeplugin.registrar.image.repository }}:{{ .Values.nodeplugin.registrar.image.tag }}"
imagePullPolicy: {{ .Values.nodeplugin.registrar.image.pullPolicy }} imagePullPolicy: {{ .Values.nodeplugin.registrar.image.pullPolicy }}
args: args:
- "--v=5" - "--v={{ .Values.logLevel }}"
- "--csi-address=/csi/{{ .Values.pluginSocketFile }}" - "--csi-address=/csi/{{ .Values.pluginSocketFile }}"
- "--kubelet-registration-path={{ .Values.socketDir }}/{{ .Values.pluginSocketFile }}" - "--kubelet-registration-path={{ .Values.socketDir }}/{{ .Values.pluginSocketFile }}"
env: env:
@ -66,7 +66,7 @@ spec:
- "--nodeserver=true" - "--nodeserver=true"
- "--pidlimit=-1" - "--pidlimit=-1"
- "--endpoint=$(CSI_ENDPOINT)" - "--endpoint=$(CSI_ENDPOINT)"
- "--v=5" - "--v={{ .Values.logLevel }}"
- "--drivername=$(DRIVER_NAME)" - "--drivername=$(DRIVER_NAME)"
{{- if .Values.topology.enabled }} {{- if .Values.topology.enabled }}
- "--domainlabels={{ .Values.topology.domainLabels | join "," }}" - "--domainlabels={{ .Values.topology.domainLabels | join "," }}"

View File

@ -48,7 +48,7 @@ spec:
imagePullPolicy: {{ .Values.provisioner.provisioner.image.pullPolicy }} imagePullPolicy: {{ .Values.provisioner.provisioner.image.pullPolicy }}
args: args:
- "--csi-address=$(ADDRESS)" - "--csi-address=$(ADDRESS)"
- "--v=5" - "--v={{ .Values.logLevel }}"
- "--timeout={{ .Values.provisioner.timeout }}" - "--timeout={{ .Values.provisioner.timeout }}"
- "--enable-leader-election=true" - "--enable-leader-election=true"
- "--leader-election-type=leases" - "--leader-election-type=leases"
@ -69,7 +69,7 @@ spec:
image: "{{ .Values.provisioner.resizer.image.repository }}:{{ .Values.provisioner.resizer.image.tag }}" image: "{{ .Values.provisioner.resizer.image.repository }}:{{ .Values.provisioner.resizer.image.tag }}"
imagePullPolicy: {{ .Values.provisioner.resizer.image.pullPolicy }} imagePullPolicy: {{ .Values.provisioner.resizer.image.pullPolicy }}
args: args:
- "--v=5" - "--v={{ .Values.logLevel }}"
- "--csi-address=$(ADDRESS)" - "--csi-address=$(ADDRESS)"
- "--csiTimeout={{ .Values.provisioner.timeout }}" - "--csiTimeout={{ .Values.provisioner.timeout }}"
- "--leader-election" - "--leader-election"
@ -88,7 +88,7 @@ spec:
imagePullPolicy: {{ .Values.provisioner.snapshotter.image.pullPolicy }} imagePullPolicy: {{ .Values.provisioner.snapshotter.image.pullPolicy }}
args: args:
- "--csi-address=$(ADDRESS)" - "--csi-address=$(ADDRESS)"
- "--v=5" - "--v={{ .Values.logLevel }}"
- "--timeout={{ .Values.provisioner.timeout }}" - "--timeout={{ .Values.provisioner.timeout }}"
- "--leader-election=true" - "--leader-election=true"
env: env:
@ -106,7 +106,7 @@ spec:
image: "{{ .Values.provisioner.attacher.image.repository }}:{{ .Values.provisioner.attacher.image.tag }}" image: "{{ .Values.provisioner.attacher.image.repository }}:{{ .Values.provisioner.attacher.image.tag }}"
imagePullPolicy: {{ .Values.provisioner.attacher.image.pullPolicy }} imagePullPolicy: {{ .Values.provisioner.attacher.image.pullPolicy }}
args: args:
- "--v=5" - "--v={{ .Values.logLevel }}"
- "--csi-address=$(ADDRESS)" - "--csi-address=$(ADDRESS)"
- "--leader-election=true" - "--leader-election=true"
- "--retry-interval-start=500ms" - "--retry-interval-start=500ms"
@ -128,7 +128,7 @@ spec:
- "--controllerserver=true" - "--controllerserver=true"
- "--pidlimit=-1" - "--pidlimit=-1"
- "--endpoint=$(CSI_ENDPOINT)" - "--endpoint=$(CSI_ENDPOINT)"
- "--v=5" - "--v={{ .Values.logLevel }}"
- "--drivername=$(DRIVER_NAME)" - "--drivername=$(DRIVER_NAME)"
- "--rbdhardmaxclonedepth={{ .Values.provisioner.hardMaxCloneDepth }}" - "--rbdhardmaxclonedepth={{ .Values.provisioner.hardMaxCloneDepth }}"
- "--rbdsoftmaxclonedepth={{ .Values.provisioner.softMaxCloneDepth }}" - "--rbdsoftmaxclonedepth={{ .Values.provisioner.softMaxCloneDepth }}"

View File

@ -41,6 +41,11 @@ csiConfig: []
# vaultCAVerify: "false" # vaultCAVerify: "false"
encryptionKMSConfig: {} encryptionKMSConfig: {}
# Set logging level for csi containers.
# Supported values from 0 to 5. 0 for general useful logs,
# 5 for trace level verbosity.
logLevel: 5
nodeplugin: nodeplugin:
name: nodeplugin name: nodeplugin
# if you are using rbd-nbd client set this value to OnDelete # if you are using rbd-nbd client set this value to OnDelete