Skip to content

Commit cfaf8da

Browse files
committed
fix(newrelic-logging): correct logLevel default to enable global.verboseLog inheritance
- Changed fluentBit.logLevel default from "info" to "" (empty) - This allows global.verboseLog=true to correctly set LOG_LEVEL=debug - Added clear precedence comments in values.yaml Root cause: values.yaml had logLevel: "info" as default, causing the template's if $logLevel condition to always be true, preventing global.verboseLog from being evaluated. Test Results: All 107 helm-unittest tests pass Template validation: - Default (no settings) → LOG_LEVEL="info" - global.verboseLog=true → LOG_LEVEL="debug" - Explicit fluentBit.logLevel → takes precedence
1 parent 9730bda commit cfaf8da

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

charts/newrelic-logging/values.yaml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,10 @@
3030
proxy: ""
3131

3232
fluentBit:
33-
logLevel: "info"
33+
# logLevel: Set the log level for Fluent Bit (trace, debug, info, warn, error, off)
34+
# When empty, defaults to "info" unless global.verboseLog is true (then "debug")
35+
# Override this to explicitly set a log level, which takes precedence over global.verboseLog
36+
logLevel: ""
3437
path: "/var/log/containers/*.log"
3538
linuxMountPath: /var
3639
windowsPath: "C:\\var\\log\\containers\\*.log"

0 commit comments

Comments
 (0)