Skip to content

feat: add multi-container support to apexkube helm chart (#145)#145

Merged
Luv7804 merged 2 commits into
mainfrom
feat/apexkube-multi-container-support
Jun 10, 2026
Merged

feat: add multi-container support to apexkube helm chart (#145)#145
Luv7804 merged 2 commits into
mainfrom
feat/apexkube-multi-container-support

Conversation

@Luv7804

@Luv7804 Luv7804 commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Description:

Summary

Add additionalContainers support to the apexkube helm chart, enabling sidecar/multi-container pod deployments. This aligns with the existing implementation in the kube-oidc-proxy chart (see #140).

Changes

charts/apexkube/values.yaml

  • Added serviceTemplate.additionalContainers: [] with commented usage example
  • Supports both serviceTemplate.additionalContainers (applied to all services) and per-service additionalContainers overrides

charts/apexkube/templates/service.yaml

  • Added additionalContainers rendering block after the healthcheck section and before dnsConfig
  • When both serviceTemplate.additionalContainers and per-service additionalContainers are defined, they are stacked together
  • Bug fix: Fixed nil pointer error in healthcheck condition, changed .healthcheck.enabled to and .healthcheck .healthcheck.enabled to prevent template failure when .healthcheck is nil

@Luv7804 Luv7804 merged commit e344890 into main Jun 10, 2026
4 checks passed
@Luv7804 Luv7804 deleted the feat/apexkube-multi-container-support branch June 10, 2026 13:56
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