Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add status field to TempoMonolithic CR #787

Merged

Conversation

andreasgerstmayr
Copy link
Collaborator

Add status field to TempoMonolithic CR

Signed-off-by: Andreas Gerstmayr <agerstmayr@redhat.com>
Signed-off-by: Andreas Gerstmayr <agerstmayr@redhat.com>
@codecov-commenter
Copy link

codecov-commenter commented Feb 8, 2024

Codecov Report

Attention: 51 lines in your changes are missing coverage. Please review.

Comparison is base (b334efd) 73.76% compared to head (09e7fce) 73.78%.
Report is 3 commits behind head on main.

Files Patch % Lines
internal/status/monolithic.go 74.07% 40 Missing and 2 partials ⚠️
controllers/tempo/tempomonolithic_controller.go 40.00% 8 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #787      +/-   ##
==========================================
+ Coverage   73.76%   73.78%   +0.02%     
==========================================
  Files          82       84       +2     
  Lines        6010     6188     +178     
==========================================
+ Hits         4433     4566     +133     
- Misses       1350     1393      +43     
- Partials      227      229       +2     
Flag Coverage Δ
unittests 73.78% <74.24%> (+0.02%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Signed-off-by: Andreas Gerstmayr <agerstmayr@redhat.com>
Signed-off-by: Andreas Gerstmayr <agerstmayr@redhat.com>
@andreasgerstmayr andreasgerstmayr merged commit 5d01d0a into grafana:main Feb 12, 2024
11 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants