🌱 Bump controller-tools from 0.20.0 to 0.20.1#2534
🌱 Bump controller-tools from 0.20.0 to 0.20.1#2534openshift-merge-bot[bot] merged 2 commits intooperator-framework:mainfrom
Conversation
Bumps the k8s-dependencies group with 1 update: [sigs.k8s.io/controller-tools](https://github.com/kubernetes-sigs/controller-tools). Updates `sigs.k8s.io/controller-tools` from 0.20.0 to 0.20.1 - [Release notes](https://github.com/kubernetes-sigs/controller-tools/releases) - [Changelog](https://github.com/kubernetes-sigs/controller-tools/blob/main/RELEASE.md) - [Commits](kubernetes-sigs/controller-tools@v0.20.0...v0.20.1) --- updated-dependencies: - dependency-name: sigs.k8s.io/controller-tools dependency-version: 0.20.1 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: k8s-dependencies ... Signed-off-by: dependabot[bot] <support@github.com>
✅ Deploy Preview for olmv1 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Pull request overview
Bumps sigs.k8s.io/controller-tools to v0.20.1 and refreshes generated CRD manifests to reflect the new controller-gen version annotation.
Changes:
- Update
sigs.k8s.io/controller-toolsdependency fromv0.20.0tov0.20.1ingo.mod/go.sum - Regenerate/refresh CRD manifests so
controller-gen.kubebuilder.io/versionannotations reflectv0.20.1 - Apply the same CRD annotation update across standard/experimental and e2e/Helm CRD copies
Reviewed changes
Copilot reviewed 10 out of 11 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| manifests/standard.yaml | Updates CRD controller-gen version annotation to v0.20.1. |
| manifests/standard-e2e.yaml | Updates CRD controller-gen version annotation to v0.20.1 for e2e manifests. |
| manifests/experimental.yaml | Updates CRD controller-gen version annotation to v0.20.1 for experimental manifests. |
| manifests/experimental-e2e.yaml | Updates CRD controller-gen version annotation to v0.20.1 for experimental e2e manifests. |
| helm/olmv1/base/operator-controller/crd/standard/olm.operatorframework.io_clusterextensions.yaml | Updates Helm-packaged standard CRD annotation to v0.20.1. |
| helm/olmv1/base/operator-controller/crd/experimental/olm.operatorframework.io_clusterextensions.yaml | Updates Helm-packaged experimental CRD annotation to v0.20.1. |
| helm/olmv1/base/operator-controller/crd/experimental/olm.operatorframework.io_clusterextensionrevisions.yaml | Updates Helm-packaged experimental CRD annotation to v0.20.1. |
| helm/olmv1/base/catalogd/crd/standard/olm.operatorframework.io_clustercatalogs.yaml | Updates Helm-packaged standard CRD annotation to v0.20.1. |
| helm/olmv1/base/catalogd/crd/experimental/olm.operatorframework.io_clustercatalogs.yaml | Updates Helm-packaged experimental CRD annotation to v0.20.1. |
| go.sum | Updates module sums for controller-tools v0.20.1. |
| go.mod | Bumps controller-tools requirement to v0.20.1. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Update the bingo tool, test files and CRDs Signed-off-by: Todd Short <tshort@redhat.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 17 out of 18 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
rashmigottipati
left a comment
There was a problem hiding this comment.
I see that this PR modifies the controller-gen.kubebuilder.io/version annotation from v0.20.0 to v0.20.1 which the dependabot PR was missing.
Changes look good to me.
/lgtm
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2534 +/- ##
==========================================
+ Coverage 68.58% 68.60% +0.02%
==========================================
Files 131 131
Lines 9288 9288
==========================================
+ Hits 6370 6372 +2
+ Misses 2432 2428 -4
- Partials 486 488 +2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
/approve |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: perdasilva, rashmigottipati The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
55473d8
into
operator-framework:main
Alternative to #2533; this includes additional controller-tools updates.
Description
Reviewer Checklist