grafana / tempo-operator

Grafana Tempo Kubernetes operator
https://grafana.com/docs/tempo/latest/setup/operator/
GNU Affero General Public License v3.0
61 stars 30 forks source link

[Chore] Fix tests after standalone Tempo query change #1028

Closed IshwarKanse closed 1 month ago

IshwarKanse commented 1 month ago

We have few bugs that need to be fixed. Due to which the tests need to be updated in a seperate PR after the fixes. [Upstream] Tempo query frontend failing with error unknown flag: --prometheus.query.support-spanmetrics-connector https://issues.redhat.com/browse/TRACING-4702 [Upstream] Tempo query container crashing with error opening /var/run/ca/service-ca.crt CA https://issues.redhat.com/browse/TRACING-4703

codecov-commenter commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 73.08%. Comparing base (21054a4) to head (ab06f93).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1028 +/- ## ======================================= Coverage 73.08% 73.08% ======================================= Files 106 106 Lines 6624 6624 ======================================= Hits 4841 4841 Misses 1493 1493 Partials 290 290 ``` | [Flag](https://app.codecov.io/gh/grafana/tempo-operator/pull/1028/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=grafana) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/grafana/tempo-operator/pull/1028/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=grafana) | `73.08% <ø> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=grafana#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.