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 multitenancy and component-replicas test #1006

Closed IshwarKanse closed 2 months ago

IshwarKanse commented 3 months ago
--- PASS: chainsaw (0.00s)
    --- PASS: chainsaw/multitenancy (139.97s)
    --- PASS: chainsaw/component-replicas (221.59s)
PASS
Tests Summary...
- Passed  tests 2
- Failed  tests 0
- Skipped tests 0
Done.

The tests are failing in OCP CI job as --traces.tls.watch-certs=true is now added to the gateway deployment. Also changed the TempoStack name in component-replicas test case.

codecov-commenter commented 3 months ago

Codecov Report

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

Project coverage is 73.16%. Comparing base (020ed7d) to head (25e54b4).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1006 +/- ## ======================================= Coverage 73.16% 73.16% ======================================= Files 106 106 Lines 6607 6607 ======================================= Hits 4834 4834 Misses 1480 1480 Partials 293 293 ``` | [Flag](https://app.codecov.io/gh/grafana/tempo-operator/pull/1006/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/1006/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=grafana) | `73.16% <ø> (ø)` | | 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.

pavolloffay commented 3 months ago

CI failed

IshwarKanse commented 3 months ago

@pavolloffay We are debugging in this PR for the monolithic-receivers-tls test https://github.com/grafana/tempo-operator/pull/1007 Its failing for min k8s version. I'll check on OCP 4.12 once.