Skip to content

Conversation

@ArjunDandagi
Copy link

@ArjunDandagi ArjunDandagi commented Jul 25, 2025

This PR:

  • Allows more fields to be referenced from secretRef.

NOTE:

  • please mention if anymore needs to be considered.

Important

Add support for fetching PostgreSQL and Redis configuration fields from Kubernetes secrets in Helm chart.

  • Behavior:
    • Extend langfuse.getValueOrSecret usage in _helpers.tpl to support postgresql and redis fields: host, port, username, and database.
    • Add secret reference support for postgresql and redis configurations in values.yaml.
  • Examples:
    • Add secret-refs-example.yaml to demonstrate using secretKeyRef for PostgreSQL and Redis configurations.
  • Misc:
    • Update REDIS_CONNECTION_STRING logic in _helpers.tpl to handle secret references for host, port, and username.

This description was created by Ellipsis for f10a7bd. You can customize this summary. It will automatically update as commits are pushed.

@ArjunDandagi ArjunDandagi requested a review from Steffen911 as a code owner July 25, 2025 07:28
@CLAassistant
Copy link

CLAassistant commented Jul 25, 2025

CLA assistant check
All committers have signed the CLA.

@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Jul 25, 2025
@ArjunDandagi ArjunDandagi marked this pull request as draft August 14, 2025 21:01
@yonik-lr
Copy link

hey, can you consider adding langfuse.smtp.connectionUrl, as in #234 🙏

@tom-mav9
Copy link

Hey has this gone stale? We'd really appreciate this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants