Skip to content

[DEV-15058] Add signoz, disabled by default#552

Merged
ltellesfl merged 3 commits intomainfrom
dev-15058
Mar 27, 2026
Merged

[DEV-15058] Add signoz, disabled by default#552
ltellesfl merged 3 commits intomainfrom
dev-15058

Conversation

@ltellesfl
Copy link
Copy Markdown
Contributor

@ltellesfl ltellesfl commented Mar 27, 2026

Note

Medium Risk
Introduces a new monitoring mode that conditionally disables existing Prometheus/Loki/Tempo/alerting components in favor of SigNoz, which could change observability coverage if misconfigured. Default is off, limiting impact unless enable_signoz is enabled and endpoints/tokens are set.

Overview
Adds an optional SigNoz integration (disabled by default) via new Terraform variables (enable_signoz, signoz_otel_collector_endpoint, signoz_bearer_token) and new Helm values for signoz plus enabling the opentelemetry-operator.

When enable_signoz is true, the PR turns off the existing monitoring stack pieces (kube-prometheus-stack, opentelemetry-collector, tempo, loki/fluent-bit, and Alertmanager-based alerting) to avoid running both stacks simultaneously.

Written by Cursor Bugbot for commit 7c1458b. This will update automatically on new commits. Configure here.

Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

@ltellesfl ltellesfl merged commit a8a7d4e into main Mar 27, 2026
19 of 20 checks passed
@ltellesfl ltellesfl deleted the dev-15058 branch March 27, 2026 19:36
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.

2 participants