Skip to content

feat: add support for extraInitContainers in deployment template#5

Open
aehrt55 wants to merge 1 commit into
mainfrom
refactor/standardize-init-containers
Open

feat: add support for extraInitContainers in deployment template#5
aehrt55 wants to merge 1 commit into
mainfrom
refactor/standardize-init-containers

Conversation

@aehrt55
Copy link
Copy Markdown

@aehrt55 aehrt55 commented Apr 15, 2026

Summary

This PR introduces the extraInitContainers field to the deployment template, allowing users to inject custom initialization logic (e.g., plugin installation, complex environment setup) after the built-in init containers.

Key Changes

  • Added extraInitContainers block in templates/deployment.yaml between init-skills and the main containers.
  • Defined extraInitContainers: [] as a default in values.yaml with explanatory comments.

This provides the necessary hook for environment-specific customizations while keeping the core chart generic and stable.

🤖 Generated with Claude Code

@aehrt55 aehrt55 force-pushed the refactor/standardize-init-containers branch from d6fd139 to 518bada Compare April 15, 2026 05:33
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces the extraInitContainers field to the Helm chart, allowing users to define additional init containers in the deployment. I have no feedback to provide.

Allow users to inject custom init containers after the built-in
init-config and init-skills containers via .Values.extraInitContainers.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@aehrt55 aehrt55 force-pushed the refactor/standardize-init-containers branch from 518bada to 184f7ce Compare April 15, 2026 06:43
@aehrt55 aehrt55 changed the title refactor: remove hard-coded init-plugins in favor of extraInitContainers feat: add support for extraInitContainers in deployment template Apr 15, 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.

1 participant