[tempo] switch to tempo-distributed and check if tempoQuery.enabled=true is still needed #1011
Comments
|
According to grafana/helm-charts#3417 (comment) this tempoQuery.enabled is only needed when integrated with jaeger, not with grafana. So we can remove this setting and ignore the breaking change in the PR #824 |
jkleinlercher
added a commit
that referenced
this issue
Jan 24, 2025
jkleinlercher
added a commit
that referenced
this issue
Jan 24, 2025
…1018) * fix(tempo): queryEnabled not needed with grafana integration (#1011) * updated container image list * run commit even when Scan is skipped so deleted reports get deleted too * updated trivy scan results --------- Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
See https://grafana.com/docs/tempo/latest/setup/helm-chart/ and https://grafana.com/docs/helm-charts/tempo-distributed/next/get-started-helm-charts/
and check if #824 (comment) can be ignored because
The text was updated successfully, but these errors were encountered: