grafana / tempo-operator

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

[chore] fix upgrade e2e test #943

Closed andreasgerstmayr closed 3 months ago

andreasgerstmayr commented 3 months ago

The test retrieved the wrong version (0.10.0 comes before 0.8.0 when sorting alphabetically), built a bundle with the wrong version, and then OLM could not determine an upgrade path.

codecov-commenter commented 3 months ago

Codecov Report

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

Project coverage is 76.41%. Comparing base (c202a04) to head (06935aa).

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