codecov[bot]
commented
2 months ago Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 22.81%. Comparing base (
ced1d92) to head (e59065d). Report is 1 commits behind head on master.
Additional details and impacted files
```diff @@ Coverage Diff @@ ## master #5208 +/- ## ========================================== - Coverage 22.83% 22.81% -0.03% ========================================== Files 420 420 Lines 45302 45302 ========================================== - Hits 10344 10334 -10 - Misses 34163 34172 +9 - Partials 795 796 +1 ```:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
What this PR does / why we need it:
Without this PR, the traces are lost when the tempo pod is evicted. The StatefulSet manages the tempo's pod, so usually it does not occur, but it occurs with the times such as when the configmap is changed.
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?: Yes only if users use the control plane version v0.49.0-rc0
How are users affected by this change: When users using control plane helm chart with monitoring.enabled as true, they cannot upgrade the control plane without manual operation.
Is this breaking change: Yes
How to migrate (if breaking change): Delete the Grafana Tempo's StatefulSet before applying the Helm Chart.