Skip to content

Conversation

acornett21
Copy link
Contributor

Description of the change:

  • Updating k8s to 1.33
  • Updating go to 1.24

Motivation for the change:

Copy link

codecov bot commented Jun 20, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.98%. Comparing base (426fc03) to head (30c1a0b).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #241      +/-   ##
==========================================
- Coverage   88.55%   87.98%   -0.58%     
==========================================
  Files          18       18              
  Lines         699      699              
==========================================
- Hits          619      615       -4     
- Misses         67       71       +4     
  Partials       13       13              

☔ View full report in Codecov by Sentry.
📢 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.

}

c, err := controller.NewUnmanaged("pod", mgr, controller.Options{
c, err := controller.NewUnmanaged("pod", controller.Options{
Copy link
Contributor

Choose a reason for hiding this comment

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

It seems right since the mgr option was removed here: kubernetes-sigs/controller-runtime#3141

Copy link
Contributor

@camilamacedo86 camilamacedo86 left a comment

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 Jun 21, 2025
@camilamacedo86 camilamacedo86 added this pull request to the merge queue Jun 21, 2025
Merged via the queue into operator-framework:main with commit 44e6f81 Jun 21, 2025
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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