Problem
When log_ingestor: null is set in values.yaml, the Pydantic model parser falls back to default values, which leaves the log-ingestor configured even though it should be disabled.
Impact
When clpConfig.package.storage is configured as "clp" (CLP-Text, which doesn't support the log-ingestor), the configuration is deemed invalid and most Helm chart deployments won't start.
Solution
The configuration parser should explicitly handle null values for log_ingestor to prevent fallback to default values, ensuring the feature remains disabled when set to null.
References
Files Affected
tools/deployment/package-helm/values.yaml- Configuration parser/validator (likely Python Pydantic models)
Problem
When
log_ingestor: nullis set invalues.yaml, the Pydantic model parser falls back to default values, which leaves the log-ingestor configured even though it should be disabled.Impact
When
clpConfig.package.storageis configured as"clp"(CLP-Text, which doesn't support the log-ingestor), the configuration is deemed invalid and most Helm chart deployments won't start.Solution
The configuration parser should explicitly handle
nullvalues forlog_ingestorto prevent fallback to default values, ensuring the feature remains disabled when set to null.References
log-ingestordeployment. #1856log-ingestordeployment. #1856 (comment)Files Affected
tools/deployment/package-helm/values.yaml