Skip to content

feat(examples): simplify k8s Helm chart to MVP#234

Merged
MaojiaSheng merged 3 commits intomainfrom
feature/k8s-helm-chart-mvp
Feb 21, 2026
Merged

feat(examples): simplify k8s Helm chart to MVP#234
MaojiaSheng merged 3 commits intomainfrom
feature/k8s-helm-chart-mvp

Conversation

@ZaynJarvis
Copy link
Collaborator

Summary

  • Strips the helm chart down to 3 templates: Deployment, Service, Secret — removing HPA, PVC, ServiceAccount, and all cloud-provider templating
  • Fixes the config format: only embedding and vlm are valid confmap fields; server/storage blocks caused a pydantic validation error at startup
  • Passes --host/--port as CLI args to openviking serve instead of embedding them in the config file
  • Switches service default to ClusterIP (use kubectl port-forward for local access)
  • Uses emptyDir for data — no PVC needed for a quick trial

Test plan

  • helm lint passes (0 failures)
  • helm template renders all 3 resources cleanly
  • Deployed to EKS default namespace via helm install
  • Pod reaches 1/1 Running (packages cached on second run)
  • curl /health returns {"status":"ok"}
  • ov health returns healthy true via port-forward
  • ov status shows healthy queue + vikingdb components
  • ov ls viking://user lists directories correctly

ZaynJarvis and others added 3 commits February 15, 2026 12:00
Add a production-ready Helm chart for deploying OpenViking on Kubernetes
with support for GCP and AWS cloud providers.

Features:
- LoadBalancer service with cloud-specific annotations
- Support for GCP and AWS via values.yaml cloudProvider setting
- Secret management for ov.conf configuration
- Health checks (liveness and readiness probes)
- Optional HPA for autoscaling
- Optional PVC for persistent storage
- Uses uv container for streamlined deployment
- Comprehensive documentation

Usage:
  helm install openviking ./deploy/helm/openviking \
    --set cloudProvider=gcp \
    --set openviking.config.embedding.dense.api_key=YOUR_KEY
Remove production-only complexity to make the chart easy to try locally:
- Drop HPA, PVC, and ServiceAccount templates
- Remove security contexts, cloud provider annotations, node affinity
- Fix config: only embedding/vlm are valid confmap fields (no server/storage blocks)
- Pass --host/--port as CLI args to openviking serve
- Switch service default to ClusterIP with port-forward instructions
- Use emptyDir for data volume
- Simplify _helpers.tpl and values.yaml

Tested against EKS: ov health and ov status both pass.
@MaojiaSheng MaojiaSheng merged commit 5bb7709 into main Feb 21, 2026
5 checks passed
@MaojiaSheng MaojiaSheng deleted the feature/k8s-helm-chart-mvp branch February 21, 2026 01:14
@github-project-automation github-project-automation bot moved this from Backlog to Done in OpenViking project Feb 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants