-
Notifications
You must be signed in to change notification settings - Fork 152
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
chore: remove webhooks of all crds #7573
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #7573 +/- ##
==========================================
- Coverage 66.02% 64.66% -1.36%
==========================================
Files 343 329 -14
Lines 42227 41486 -741
==========================================
- Hits 27880 26828 -1052
- Misses 11916 12314 +398
+ Partials 2431 2344 -87
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
cbf16d1
to
2d61885
Compare
2d61885
to
a7d7edd
Compare
a7d7edd
to
6d5868f
Compare
6d5868f
to
853b32b
Compare
config/webhook/manifests.yaml
Outdated
@@ -24,26 +24,6 @@ webhooks: | |||
resources: |
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.
Why keeps /mutate-workloads-kubeblocks-io-v1alpha1-instanceset
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.
All cleaned up.
This file is maintained and updated by controller-gen, it seems that it behaves weirdly when the webhook marker is removed.
c99ed4d
to
bacfd4a
Compare
return errors.New("invalid destination object") | ||
} | ||
return convertToImpl(cc, ccv1) | ||
return errors.New("not implemented") |
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.
Why keeps this two "ConverTo" methods?
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.
Deleted
bacfd4a
to
3844eb6
Compare
No description provided.