Skip to content

Comments

v0.8.4: fix initContainers resource bug#62

Merged
cosmic-flood merged 1 commit intomainfrom
fix-0.8.3
Jan 13, 2026
Merged

v0.8.4: fix initContainers resource bug#62
cosmic-flood merged 1 commit intomainfrom
fix-0.8.3

Conversation

@cosmic-flood
Copy link
Contributor

@cosmic-flood cosmic-flood commented Jan 13, 2026

This pull request makes improvements to the Helm chart templates for resource handling in init containers and bumps the chart version to reflect these changes. The main focus is on ensuring that resource specifications for init containers are only included when both the initContainers and their resources fields are present, which increases template robustness.

Helm chart template improvements:

  • Updated both _initContainers-wait-for-infrastructure-dependencies.tpl and _initContainers-wait-for-other-components.tpl to only render the resources block for init containers if both initContainers and initContainers.resources are defined, preventing errors from missing fields. [1] [2]

Version update:

  • Bumped the chart version from 0.8.3 to 0.8.4 in Chart.yaml to reflect the new changes.

@cosmic-flood
Copy link
Contributor Author

@copilot give me the title and description of this update

Copy link

Copilot AI commented Jan 13, 2026

@cosmic-flood I've opened a new pull request, #63, to work on those changes. Once the pull request is ready, I'll request review from you.

@cosmic-flood
Copy link
Contributor Author

@copilot help me to summarize this pull request. actually I fixed a bug that .tpl file should safely process the siutations that initContainers doesn't exist in the values.

Copy link

Copilot AI commented Jan 13, 2026

@cosmic-flood I've opened a new pull request, #64, to work on those changes. Once the pull request is ready, I'll request review from you.

@cosmic-flood cosmic-flood merged commit 33d64eb into main Jan 13, 2026
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