Skip to content

Comments

feat: add dnsConfig and clickhouse.database values; update docs#285

Merged
Steffen911 merged 4 commits intolangfuse:mainfrom
mozesgaal-seon:feat/additional-config-values
Dec 2, 2025
Merged

feat: add dnsConfig and clickhouse.database values; update docs#285
Steffen911 merged 4 commits intolangfuse:mainfrom
mozesgaal-seon:feat/additional-config-values

Conversation

@mozesgaal-seon
Copy link
Contributor

@mozesgaal-seon mozesgaal-seon commented Nov 28, 2025

Important

Add DNS configuration and ClickHouse database settings, update documentation for ClickHouse password issues.

  • Configuration:
    • Add dnsConfig to web/deployment.yaml and worker/deployment.yaml for DNS settings.
    • Add clickhouse.database value in values.yaml and _helpers.tpl for specifying ClickHouse database.
  • Documentation:
    • Update TROUBLESHOOTING.md to include more problematic characters for ClickHouse passwords and add a new error message example.
  • Misc:
    • Update values.yaml to include comments for new dnsConfig and clickhouse.database settings.

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

@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Nov 28, 2025
@CLAassistant
Copy link

CLAassistant commented Nov 28, 2025

CLA assistant check
All committers have signed the CLA.

Copy link
Member

@Steffen911 Steffen911 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you a lot for those additions!

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Dec 2, 2025
@Steffen911 Steffen911 merged commit 6f7b39a into langfuse:main Dec 2, 2025
3 checks passed
@mozesgaal-seon mozesgaal-seon deleted the feat/additional-config-values branch December 4, 2025 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants