Skip to content

feat: helm chart for kubernetes deployment - #68

Merged
kacy merged 2 commits into
mainfrom
feat/helm-chart
Feb 8, 2026
Merged

feat: helm chart for kubernetes deployment#68
kacy merged 2 commits into
mainfrom
feat/helm-chart

Conversation

@kacy

@kacy kacy commented Feb 8, 2026

Copy link
Copy Markdown
Owner

summary

  • adds a minimal helm chart under helm/ember/ for deploying ember to kubernetes
  • chart creates a deployment, service, and serviceaccount
  • all ember configuration flows through EMBER_* environment variables, matching the existing dockerfile support
  • adds helm-lint and helm-template targets to the makefile
  • adds a kubernetes section to the README with install examples

what was tested

  • helm lint helm/ember — passes clean (only an info about missing icon)
  • helm template ember helm/ember — renders valid manifests with default values
  • helm template with overrides (--set ember.maxMemory=512M, --set ember.appendonly=true, etc.) — all conditionals render correctly
  • verified serviceaccount is omitted when serviceAccount.create=false

design considerations

  • kept the chart minimal: no ingress, PVC, or HPA. users can extend as needed
  • uses emptyDir for /data — persistence in k8s would need a PVC, which is out of scope for v1
  • metrics port is conditionally exposed in both the deployment and service
  • TCP probes on the cache port for liveness/readiness (no HTTP endpoint needed)

kacy added 2 commits February 8, 2026 18:58
minimal chart with deployment, service, and serviceaccount.
configures ember via EMBER_* env vars from values.yaml.
@kacy
kacy merged commit 1935a73 into main Feb 8, 2026
5 checks passed
@kacy
kacy deleted the feat/helm-chart branch February 8, 2026 23:59
kacy added a commit that referenced this pull request Feb 11, 2026
* feat: add helm chart for kubernetes deployment

minimal chart with deployment, service, and serviceaccount.
configures ember via EMBER_* env vars from values.yaml.

* docs: add helm targets to makefile and kubernetes section to readme
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