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
code-generator: fix the bug that ApplyConfiguration constructor missing WithKind/WithAPIVersion methods #107834
Conversation
…ng WithKind/WithAPIVersion methods
38a8355
to
ce50eed
Compare
integration timeout flake tracked in #107857 |
/assign @jpbetz |
/retest |
/triage accepted |
/test pull-kubernetes-integration |
kindly ping @jpbetz |
/lgtm This looks right |
/priority important-soon |
kindly ping @liggitt for approve |
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
/approve
@@ -36,7 +36,10 @@ type DeleteOptionsApplyConfiguration struct { | |||
// DeleteOptionsApplyConfiguration constructs an declarative configuration of the DeleteOptions type for use with | |||
// apply. | |||
func DeleteOptions() *DeleteOptionsApplyConfiguration { |
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.
pre-existing, but apply configuration for delete options is weird, right? we can't actually submit this object to server-side apply
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.
I remember squinting at this one when I originally added apply configurations. The apply configurations generator uses a reachability routine to figure out which types to include. It includes delete options because of
DeleteOptions *metav1.DeleteOptions `json:"deleteOptions,omitempty" protobuf:"bytes,2,opt,name=deleteOptions"` |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: liggitt, SataQiu The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What type of PR is this?
/kind bug
What this PR does / why we need it:
code-generator: fix the bug that ApplyConfiguration constructor missing WithKind/WithAPIVersion methods
Which issue(s) this PR fixes:
Fixes #107795
Special notes for your reviewer:
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: