Skip to content
This repository was archived by the owner on Mar 3, 2025. It is now read-only.

🐛 test/e2e/metrics_endpoint_test.go: discover catalogd namespace and improve logging#503

Merged
joelanford merged 1 commit into
operator-framework:mainfrom
joelanford:fix-metrics-e2e
Jan 7, 2025
Merged

🐛 test/e2e/metrics_endpoint_test.go: discover catalogd namespace and improve logging#503
joelanford merged 1 commit into
operator-framework:mainfrom
joelanford:fix-metrics-e2e

Conversation

@joelanford

Copy link
Copy Markdown
Member

Same PR as in operator-controller (see operator-framework/operator-controller#1547)

…prove logging

Signed-off-by: Joe Lanford <joe.lanford@gmail.com>
@joelanford joelanford requested a review from a team as a code owner January 7, 2025 19:27
@codecov

codecov Bot commented Jan 7, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 37.20%. Comparing base (1dbab8e) to head (ce83da7).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #503   +/-   ##
=======================================
  Coverage   37.20%   37.20%           
=======================================
  Files          15       15           
  Lines        1258     1258           
=======================================
  Hits          468      468           
  Misses        740      740           
  Partials       50       50           

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

@tmshort tmshort left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Jan 7, 2025
@joelanford joelanford added this pull request to the merge queue Jan 7, 2025
Merged via the queue into operator-framework:main with commit be1e4d6 Jan 7, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants