Skip to content

default controllers[].logLevel to prevent empty -v= crashloop (chart 0.3.1)#33

Merged
bborbe merged 1 commit into
masterfrom
fix/controller-loglevel-default
Jul 4, 2026
Merged

default controllers[].logLevel to prevent empty -v= crashloop (chart 0.3.1)#33
bborbe merged 1 commit into
masterfrom
fix/controller-loglevel-default

Conversation

@bborbe

@bborbe bborbe commented Jul 4, 2026

Copy link
Copy Markdown
Owner

Robustness fix found during the quant crossover: a controllers: entry that omits logLevel rendered - -v= (empty), which glog rejects at startup → the controller CrashLoopBackOff'd printing its usage. Defaulting it to "2" (matching the executor's existing default) makes the chart safe for any consumer who omits it.

  • controller-statefulset.yaml: -v={{ $c.logLevel }}-v={{ $c.logLevel | default "2" }}
  • Chart 0.3.00.3.1

Verified

helm template with a controller entry that omits logLevel now renders args: ['-v=2']; lint clean; make precommit passes.

🤖 Generated with Claude Code

@bborbe bborbe merged commit 992ac65 into master Jul 4, 2026
1 check passed
@bborbe bborbe deleted the fix/controller-loglevel-default branch July 4, 2026 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant