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

🌱 Add check for client in e2e#485

Merged
tmshort merged 1 commit into
operator-framework:mainfrom
tmshort:skip-metrics
Dec 17, 2024
Merged

🌱 Add check for client in e2e#485
tmshort merged 1 commit into
operator-framework:mainfrom
tmshort:skip-metrics

Conversation

@tmshort

@tmshort tmshort commented Dec 17, 2024

Copy link
Copy Markdown
Contributor

Depending on how the e2e is run, we may not have a k8s client available.

Depending on how the e2e is run, we may not have a k8s client available.

Signed-off-by: Todd Short <todd.short@me.com>
@codecov

codecov Bot commented Dec 17, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 38.23%. Comparing base (f91558f) to head (152e49e).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #485   +/-   ##
=======================================
  Coverage   38.23%   38.23%           
=======================================
  Files          15       15           
  Lines        1224     1224           
=======================================
  Hits          468      468           
  Misses        706      706           
  Partials       50       50           

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

@camilamacedo86 camilamacedo86 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 Dec 17, 2024
@tmshort tmshort added this pull request to the merge queue Dec 17, 2024
Merged via the queue into operator-framework:main with commit 0dbef9b Dec 17, 2024
@tmshort tmshort deleted the skip-metrics branch February 25, 2025 14:41
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