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
update smd to 4.2.3 #111557
update smd to 4.2.3 #111557
Conversation
/lgtm thanks for the fix! |
🤔 |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: alexzielenski, aojea, liggitt 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 |
ahh, it needs rebase, I was surprised the PR didn't pick the approve and lgtm labels from Jordan |
3c8abaf
to
e77ed0b
Compare
rebased and also changed 4.2.2 to 4.2.3 |
/lgtm |
/triage accepted |
cherry pick pr #111557 to 1.23 :update smd to 4.2.3
cherry pick pr #111557 to 1.24 :update smd to 4.2.3
brings in fixes to exponential recursion bug, adds field level override ability to support another backport effort
Comparison between two versions: https://github.com/kubernetes-sigs/structured-merge-diff/compare/v4.2.1..v4.2.3
What type of PR is this?
/kind bug
What this PR does / why we need it:
Fixes exponentially expanding recursion bug when first creating an object with a fully deduced type. Details: Fix exponential merge recursion kubernetes-sigs/structured-merge-diff#222.
Adds ability to override
structType
annotation with field-level referencesWhich issue(s) this PR fixes:
fixes #109099
required for #110495
Special notes for your reviewer:
Ran
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:
/cc @apelisse