grafana / tempo-operator

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

[Chore] Test scale-up/scale-down of TempoStack components replica #1000

Closed IshwarKanse closed 1 month ago

IshwarKanse commented 1 month ago
--- PASS: chainsaw (0.00s)
    --- PASS: chainsaw/component-replicas (224.28s)
PASS
Tests Summary...
- Passed  tests 1
- Failed  tests 0
- Skipped tests 0
Done.
codecov-commenter commented 1 month ago

Codecov Report

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

Project coverage is 73.16%. Comparing base (1ef6b26) to head (24009e7).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1000 +/- ## ======================================= Coverage 73.16% 73.16% ======================================= Files 106 106 Lines 6606 6606 ======================================= Hits 4833 4833 Misses 1480 1480 Partials 293 293 ``` | [Flag](https://app.codecov.io/gh/grafana/tempo-operator/pull/1000/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/1000/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.

andreasgerstmayr commented 1 month ago

I re-ran the failing action 3 times but it's always failing with

FAIL: chainsaw/monolithic-receivers-tls (207.08s)

It doesn't look directly related to this PR, but this test doesn't fail with other PRs :thinking:

IshwarKanse commented 1 month ago

@andreasgerstmayr That test has been flaky on upstream, it has failed with other PRs as well and passes after few retries. It always passed in OCP CI jobs. Resource issues ? It failed in this one also https://github.com/grafana/tempo-operator/pull/1002

andreasgerstmayr commented 1 month ago

hm yeah I just saw it failing with another PR. I'd prefer fixing it upstream, I guess it's just a timeout.

IshwarKanse commented 1 month ago

@andreasgerstmayr Ok, I'll take a look in the morning for that failing one.

andreasgerstmayr commented 1 month ago

I gave it a try with #1005, let's see :)