Skip to content

feat: configurable fulltext search backend in Helm chart#3

Open
dnplkndll wants to merge 1 commit intofeat/helm-chartfrom
feat/multi-search-backend
Open

feat: configurable fulltext search backend in Helm chart#3
dnplkndll wants to merge 1 commit intofeat/helm-chartfrom
feat/multi-search-backend

Conversation

@dnplkndll
Copy link
Copy Markdown

@dnplkndll dnplkndll commented Mar 27, 2026

Summary

Add Helm chart values for configuring the fulltext search backend and observability.

Fulltext backend selection

  • fulltext.backend: "elastic" (default), "opensearch", or "typesense"
  • fulltext.dbUrl: Override FULLTEXT_DB_URL for backends needing a different URL format (e.g. Typesense with ?apiKey=)
  • FULLTEXT_BACKEND set directly on fulltext deployment (not configmap — only fulltext pod needs it)
  • FULLTEXT_DB_URL defaults to ELASTIC_URL when dbUrl is empty

Observability

  • observability.otlpEndpoint: OTLP trace/metrics export endpoint
  • observability.otlpHeaders: Auth headers for OTLP (e.g. Authorization=Basic ...)

No breaking changes

All new values default to empty strings — existing deployments are unaffected.

Depends on

Test plan

  • helm template with default values renders correctly (no new env vars)
  • helm template with fulltext.backend=typesense renders FULLTEXT_BACKEND + FULLTEXT_DB_URL override
  • helm template with observability.otlpEndpoint renders OTEL vars in configmap
  • Helm CI: Lint & Template pass, kind-selfhost pass
  • Production: deployed with Typesense backend, 565 docs indexed, auto-reindex working

🤖 Generated with Claude Code

Add values for selecting the fulltext search backend:
  - fulltext.backend: "elastic" (default), "opensearch", or "typesense"
  - fulltext.dbUrl: override for non-ES backends (e.g. Typesense URL)
  - observability.otlpEndpoint + otlpHeaders: OTLP trace export config

The FULLTEXT_BACKEND env var is set directly on the fulltext deployment
(not via configmap, since only the fulltext pod needs it). FULLTEXT_DB_URL
defaults to ELASTIC_URL but can be overridden via fulltext.dbUrl for
backends that use a different URL format (e.g. Typesense with apiKey).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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