-
Notifications
You must be signed in to change notification settings - Fork 35
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
bump controller gen #911
bump controller gen #911
Conversation
In case you pre-installed it, make sure to have v0.15.0 or remove it from your GOBIN path. Signed-off-by: Liran Rotenberg <[email protected]>
Quality Gate passedIssues Measures |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #911 +/- ##
=======================================
Coverage 15.94% 15.94%
=======================================
Files 106 106
Lines 19711 19711
=======================================
Hits 3143 3143
Misses 16292 16292
Partials 276 276
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@liranr23 shall we have this also in the release-v2.6.3 branch?
@@ -3,8 +3,7 @@ apiVersion: apiextensions.k8s.io/v1 | |||
kind: CustomResourceDefinition | |||
metadata: | |||
annotations: | |||
controller-gen.kubebuilder.io/version: v0.10.0 | |||
creationTimestamp: null |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good, I saw a warning about creationTimestamp
somewhere
we can, should be harmless. |
In case you pre-installed it, make sure to have v0.15.0 or remove it from your GOBIN path.