-
Notifications
You must be signed in to change notification settings - Fork 1
feat(hindsight): wire real vectorize-io OCI Helm chart #4913
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,15 +1,9 @@ | ||
| # Hindsight — agent persistent memory system for Hermes. | ||
| # Standalone Helm chart deployed via ArgoCD. | ||
| # Hindsight (vectorize-io) — agent persistent memory system for Hermes. | ||
| # Real chart wired: OCI Helm chart at ghcr.io/vectorize-io/charts/hindsight. | ||
| # | ||
| # TODO(maintainer): provide the Helm chart URL + chart name + version. | ||
| # Confirm which Hindsight chart this refers to: | ||
| # - public OSS chart (helm repo URL) | ||
| # - private chart (repoURL + auth) | ||
| # - in-repo chart (sibling repo URL + path) | ||
| # | ||
| # Once repoURL + chart name + version are provided, this Application | ||
| # wires up directly. Until then, this placeholder declares the | ||
| # namespace + intent so the structure is in place. | ||
| # Pairs with hermes/Application.yaml — Hermes' deployment.yaml | ||
| # sets `HINDSIGHT_URL=http://hindsight.hindsight.svc.cluster.local` | ||
| # which matches this Application's namespace + chart-default service name. | ||
|
|
||
| apiVersion: argoproj.io/v1alpha1 | ||
| kind: Application | ||
|
|
@@ -20,27 +14,55 @@ metadata: | |
| spec: | ||
| project: default | ||
| source: | ||
| # TODO(maintainer): replace with the real Helm repo + chart name. | ||
| # Example shape: | ||
| # repoURL: https://your-org.github.io/hindsight-chart/ | ||
| # chart: hindsight | ||
| # targetRevision: 1.0.0 | ||
| # helm: | ||
| # releaseName: hindsight | ||
| # valuesObject: | ||
| # persistence: | ||
| # storageClass: longhorn | ||
| # size: 20Gi | ||
| # hermesIntegration: | ||
| # enabled: true | ||
| repoURL: https://github.com/Lucent-Financial-Group/Zeta | ||
| targetRevision: main | ||
| path: full-ai-cluster/k8s/applications/hindsight | ||
| directory: | ||
| include: 'namespace.yaml' | ||
| repoURL: ghcr.io/vectorize-io/charts | ||
| chart: hindsight | ||
| targetRevision: 0.3.0 # bump to latest as vectorize-io publishes | ||
| helm: | ||
| releaseName: hindsight | ||
| valuesObject: | ||
| # Bundled PostgreSQL for storage. Swap to false + | ||
| # api.database.url for an external CockroachDB/Postgres | ||
| # backing store once the cluster's CockroachDB Application | ||
| # is healthy. | ||
| postgresql: | ||
| enabled: true | ||
| primary: | ||
| persistence: | ||
| storageClass: longhorn | ||
| size: 20Gi | ||
|
|
||
| api: | ||
| # LLM provider Hindsight uses for its own entity/relation | ||
| # extraction (separate from Hermes' LLM choice). Sourced | ||
| # from a Vault-backed ExternalSecret rather than hardcoded. | ||
| llm: | ||
| provider: groq # change per maintainer preference | ||
| existingSecret: | ||
| name: hindsight-llm-api-key | ||
| key: api-key | ||
|
|
||
| # Database URL inferred from bundled postgresql above when | ||
| # `postgresql.enabled: true`. Override here for external | ||
| # CockroachDB: | ||
| # database: | ||
| # url: postgresql://hindsight:$(PG_PASSWORD)@cockroachdb-public.cockroachdb.svc.cluster.local:26257/hindsight?sslmode=require | ||
| # existingSecret: | ||
| # name: hindsight-db-credentials | ||
| # passwordKey: password | ||
|
|
||
| # Service for Hermes to reach Hindsight at: | ||
| # http://hindsight.hindsight.svc.cluster.local | ||
| service: | ||
| type: ClusterIP | ||
| port: 80 | ||
|
Comment on lines
+53
to
+57
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
This block is using a top-level Useful? React with 👍 / 👎. |
||
|
|
||
| destination: | ||
| server: https://kubernetes.default.svc | ||
| namespace: hindsight | ||
| syncPolicy: | ||
| automated: { prune: false, selfHeal: true } | ||
| syncOptions: [ CreateNamespace=true, ServerSideApply=true ] | ||
| automated: | ||
| prune: false # never prune — memory store is precious | ||
| selfHeal: true | ||
| syncOptions: | ||
| - CreateNamespace=true | ||
| - ServerSideApply=true | ||
This file was deleted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The chart does not consume
api.llm.providerorapi.llm.existingSecret; in v0.3.0 it expects LLM config viaapi.envand either top-levelexistingSecretorapi.secrets(templates/api-deployment.yaml/secret.yaml). With these unsupported keys, noHINDSIGHT_API_LLM_*env vars are populated, so retain/reflect operations that require an LLM provider key will fail at runtime.Useful? React with 👍 / 👎.