-
Notifications
You must be signed in to change notification settings - Fork 364
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
Fix MarshalJSON error caused by incorrect conversion function #2302
Conversation
Codecov Report
@@ Coverage Diff @@
## main #2302 +/- ##
==========================================
+ Coverage 62.05% 62.06% +0.01%
==========================================
Files 277 278 +1
Lines 21588 21589 +1
==========================================
+ Hits 13396 13400 +4
+ Misses 6792 6788 -4
- Partials 1400 1401 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
|
controlplane.Service couldn't be converted to v1beta1.Service directly as it has an extra field. This patch fixes it by using the auto-generated conversion function. Signed-off-by: Quan Tian <[email protected]>
/test-all |
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.
LGTM, yes please backport to the 3 releases you listed in the PR description
BTW, I remember you saying that this issue affected networking.antrea.tanzu.vmware.com
but not controlplane.antrea.tanzu.vmware.com
. From the fix, I don't see how it's possible?
Actually looks like it's explained by #2305 :) |
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.
Thanks for the fix
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.
LGTM
controlplane.Service couldn't be converted to v1beta1.Service directly as it has an extra field. This patch fixes it by using the auto-generated conversion function.
Signed-off-by: Quan Tian [email protected]
Fixes #2303
It may need to be backported to 0.13, 1.0, and 1.1.