Skip to content

Comments

Fix initContainers resource rendering when not defined in values#64

Closed
Copilot wants to merge 1 commit intofix-0.8.3from
copilot/sub-pr-62-again
Closed

Fix initContainers resource rendering when not defined in values#64
Copilot wants to merge 1 commit intofix-0.8.3from
copilot/sub-pr-62-again

Conversation

Copy link

Copilot AI commented Jan 13, 2026

The initContainer template files attempted to render initContainers.resources without checking if the parent initContainers object exists, causing template evaluation failures when users omit these optional configurations.

Changes

  • Template safety checks: Added conditional guards in _initContainers-wait-for-infrastructure-dependencies.tpl and _initContainers-wait-for-other-components.tpl to verify both initContainers and initContainers.resources exist before rendering
  • Chart version: Bumped to 0.8.4

Technical detail

# Before: Would fail if initContainers undefined
resources:
  {{- toYaml (get $ctx.Values $component).initContainers.resources | nindent 4 }}

# After: Safely handles missing configuration
{{- if and (get $ctx.Values $component).initContainers (get $ctx.Values $component).initContainers.resources }}
resources:
  {{- toYaml (get $ctx.Values $component).initContainers.resources | nindent 4 }}
{{- end }}

This allows deployments to proceed without explicitly defining initContainers.resources for each component (api, ui, evalServer, daServer).


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Fix initContainers resource bug in v0.8.4 Fix initContainers resource rendering when not defined in values Jan 13, 2026
Copilot AI requested a review from cosmic-flood January 13, 2026 11:03
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.

2 participants