Skip to content

feat(foundryctl): missing env vars for cluster, replication and cardinality on ingester - #133

Open
mvaldes14 wants to merge 5 commits into
mainfrom
feat/docker-variables
Open

feat(foundryctl): missing env vars for cluster, replication and cardinality on ingester#133
mvaldes14 wants to merge 5 commits into
mainfrom
feat/docker-variables

Conversation

@mvaldes14

@mvaldes14 mvaldes14 commented Jun 10, 2026

Copy link
Copy Markdown
Collaborator

Features

  • Includes the missing variables as a default for ingester and migrator.
    • LOW_CARDINAL_EXCEPTION_GROUPING=false
    • SIGNOZ_OTEL_COLLECTOR_CLICKHOUSE_CLUSTER=cluster
    • SIGNOZ_OTEL_COLLECTOR_CLICKHOUSE_REPLICATION=true

@mvaldes14
mvaldes14 requested a review from therealpandey as a code owner June 10, 2026 17:46
Nageshbansal
Nageshbansal previously approved these changes Jun 11, 2026
View reviewed changes
therealpandey
therealpandey requested changes Jun 11, 2026
View reviewed changes
Comment thread docs/examples/coolify/stack/pours/deployment/coolify.yaml Outdated
@therealpandey

therealpandey commented Jun 11, 2026

Copy link
Copy Markdown
Member

Also what is the use of the env variable LOW....? Please check

@mvaldes14

mvaldes14 commented Jun 11, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator Author

We could leave it as is, already defaults to false.

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

Reviewers

@therealpandey therealpandey therealpandey requested changes
@Nageshbansal Nageshbansal Nageshbansal left review comments

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mvaldes14 @therealpandey @Nageshbansal