Skip to content

Add debugLogs parameter to helm chart #2624

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 22, 2025
+48 −1
Merged

Add debugLogs parameter to helm chart #2624

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 8 additions & 0 deletions charts/aws-ebs-csi-driver/templates/_node-windows.tpl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,11 @@ spec:
{{- with .Values.node.loggingFormat }}
- --logging-format={{ . }}
{{- end }}
{{- if .Values.debugLogs }}
- --v=7
{{- else }}
- --v={{ .Values.node.logLevel }}
{{- end }}
{{- if .Values.node.otelTracing }}
- --enable-otel-tracing=true
{{- end}}
Expand Down Expand Up @@ -181,7 +185,11 @@ spec:
{{- if .Values.node.windowsHostProcess }}
- --plugin-registration-path=$(PLUGIN_REG_DIR)
{{- end }}
{{- if .Values.debugLogs }}
- --v=7
{{- else }}
- --v={{ .Values.sidecars.nodeDriverRegistrar.logLevel }}
{{- end }}
env:
- name: ADDRESS
{{- if .Values.node.windowsHostProcess }}
Expand Down
8 changes: 8 additions & 0 deletions charts/aws-ebs-csi-driver/templates/_node.tpl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,11 @@ spec:
{{- with .Values.node.loggingFormat }}
- --logging-format={{ . }}
{{- end }}
{{- if .Values.debugLogs }}
- --v=7
{{- else }}
- --v={{ .Values.node.logLevel }}
{{- end }}
{{- if .Values.node.otelTracing }}
- --enable-otel-tracing=true
{{- end}}
Expand Down Expand Up @@ -181,7 +185,11 @@ spec:
args:
- --csi-address=$(ADDRESS)
- --kubelet-registration-path=$(DRIVER_REG_SOCK_PATH)
{{- if .Values.debugLogs }}
- --v=7
{{- else }}
- --v={{ .Values.sidecars.nodeDriverRegistrar.logLevel }}
{{- end }}
{{- range .Values.sidecars.nodeDriverRegistrar.additionalArgs }}
- {{ . }}
{{- end }}
Expand Down
26 changes: 25 additions & 1 deletion charts/aws-ebs-csi-driver/templates/controller.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,7 @@ spec:
{{- with .Values.controller.httpEndpoint }}
- --http-endpoint={{ . }}
{{- end }}
{{- if .Values.controller.sdkDebugLog }}
{{- if or .Values.controller.sdkDebugLog .Values.debugLogs }}
- --aws-sdk-debug-log=true
{{- end}}
{{- if .Values.controller.batching }}
Expand All @@ -103,7 +103,11 @@ spec:
{{- if .Values.controller.otelTracing }}
- --enable-otel-tracing=true
{{- end}}
{{- if .Values.debugLogs }}
- --v=7
{{- else }}
- --v={{ .Values.controller.logLevel }}
{{- end }}
{{- range .Values.controller.additionalArgs }}
- {{ . }}
{{- end }}
Expand Down Expand Up @@ -211,7 +215,11 @@ spec:
- --timeout=60s
{{- end }}
- --csi-address=$(ADDRESS)
{{- if .Values.debugLogs }}
- --v=7
{{- else }}
- --v={{ .Values.sidecars.provisioner.logLevel }}
{{- end }}
- --feature-gates=Topology=true
{{- if .Values.controller.extraCreateMetadata }}
- --extra-create-metadata
Expand Down Expand Up @@ -275,7 +283,11 @@ spec:
- --timeout=6m
{{- end }}
- --csi-address=$(ADDRESS)
{{- if .Values.debugLogs }}
- --v=7
{{- else }}
- --v={{ .Values.sidecars.attacher.logLevel }}
{{- end }}
- --leader-election={{ .Values.sidecars.attacher.leaderElection.enabled | required "leader election state for csi-attacher is required, must be set to true || false." }}
{{- if .Values.sidecars.attacher.leaderElection.enabled }}
{{- if .Values.sidecars.attacher.leaderElection.leaseDuration }}
Expand Down Expand Up @@ -330,7 +342,11 @@ spec:
args:
- --csi-address=$(ADDRESS)
- --leader-election=true
{{- if .Values.debugLogs }}
- --v=7
{{- else }}
- --v={{ .Values.sidecars.snapshotter.logLevel }}
{{- end }}
{{- if .Values.controller.extraCreateMetadata }}
- --extra-create-metadata
{{- end}}
Expand Down Expand Up @@ -379,7 +395,11 @@ spec:
- --timeout=60s
{{- end }}
- --csi-address=$(ADDRESS)
{{- if .Values.debugLogs }}
- --v=7
{{- else }}
- --v={{ .Values.sidecars.volumemodifier.logLevel }}
{{- end }}
- --leader-election={{ .Values.sidecars.volumemodifier.leaderElection.enabled | required "leader election state for csi-volumemodifier is required, must be set to true || false." }}
{{- if .Values.sidecars.volumemodifier.leaderElection.enabled }}
{{- if .Values.sidecars.volumemodifier.leaderElection.leaseDuration }}
Expand Down Expand Up @@ -439,7 +459,11 @@ spec:
- --extra-modify-metadata
{{- end}}
- --csi-address=$(ADDRESS)
{{- if .Values.debugLogs }}
- --v=7
{{- else }}
- --v={{ .Values.sidecars.resizer.logLevel }}
{{- end }}
- --handle-volume-inuse-error=false
{{- with .Values.sidecars.resizer.leaderElection }}
- --leader-election={{ .enabled | default true }}
Expand Down
5 changes: 5 additions & 0 deletions charts/aws-ebs-csi-driver/values.schema.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -126,6 +126,11 @@
"description": "Instruct the AWS SDK to use AWS FIPS endpoints, and deploy container built with BoringCrypto (a FIPS-validated cryptographic library) instead of the Go default. The EBS CSI Driver FIPS images have not undergone FIPS certification, and no official guarantee is made about the compliance of these images under the FIPS standard. Users relying on these images for FIPS compliance should perform their own independent evaluation",
"default": "false"
},
"debugLogs": {
"type": "boolean",
"description": "Set maximum verbosity for logs of each container and other recommended debugging parameters such as enabling AWS SDK debug logging",
"default": "false"
},
"fullnameOverride": {
"type": ["string", "null"],
"default": ""
Expand Down
2 changes: 2 additions & 0 deletions charts/aws-ebs-csi-driver/values.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -513,6 +513,8 @@ useOldCSIDriver: false
nodeAllocatableUpdatePeriodSeconds: 10
# Deploy EBS CSI Driver without controller and associated resources
nodeComponentOnly: false
# Set maximum verbosity for logs of each container and other recommended debugging parameters such as enabling AWS SDK debug logging
debugLogs: false
helmTester:
enabled: true
# Supply a custom image to the ebs-csi-driver-test pod in helm-tester.yaml
Expand Down
Loading