Skip to content

chore: fix noisy operator logs - #1574

Merged
andreasgerstmayr merged 1 commit into
grafana:mainfrom
andreasgerstmayr:fix-noisy-logs
Aug 4, 2026
Merged

andreasgerstmayr merged 1 commit into
grafana:mainfrom
andreasgerstmayr:fix-noisy-logs

Conversation

@andreasgerstmayr

@andreasgerstmayr andreasgerstmayr commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator
  • "certificate rotation completed" and "discovered Kubernetes API server endpoints" was printed at every TempoStack reconcile
  • public port name was used twice in the gateway pod, triggering a duplicate port name warning in the logs

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

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 75.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 59.53%. Comparing base (0b0e615) to head (1826ae8).
⚠️ Report is 199 commits behind head on main.

Files with missing lines Patch % Lines
internal/certrotation/handlers/rotate_certs.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1574      +/-   ##
==========================================
+ Coverage   57.64%   59.53%   +1.88%     
==========================================
  Files         121      135      +14     
  Lines       11277    10947     -330     
==========================================
+ Hits         6501     6517      +16     
+ Misses       4418     4041     -377     
- Partials      358      389      +31     
Flag Coverage Δ
unittests 59.53% <75.00%> (+1.88%) ⬆️

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

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@andreasgerstmayr
andreasgerstmayr merged commit bf3726b into grafana:main Aug 4, 2026
23 of 24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants