Renovate: Update module sigs.k8s.io/structured-merge-diff/v6 to v7 - #358
Renovate: Update module sigs.k8s.io/structured-merge-diff/v6 to v7#358renovate[bot] wants to merge 1 commit into
Conversation
There was a problem hiding this comment.
🟡 Changes recommended
The v7 parser types are incompatible with Kubernetes managedfields APIs still typed against v6, so the project no longer compiles.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
Updates sigs.k8s.io/structured-merge-diff from v6.4.2 to v7.0.0 and switches the generated parser import.
Changes:
- Adds the v7 module and checksums.
- Retains v6 transitively.
- Updates generated apply-configuration parsing.
Critical finding: the v7 parser is incompatible with Kubernetes APIs still typed against v6, causing compilation failure.
File summaries
| File | Reviewed changes |
|---|---|
go.sum |
Adds v7 dependency checksums. |
go.mod |
Adds v7 while retaining v6 indirectly. |
applyconfigurations/internal/internal.go |
Switches to the v7 typed parser, causing the identified incompatibility. |
Review details
Files not reviewed (1)
- applyconfigurations/internal/internal.go: Generated file
- Files reviewed: 1/3 changed files
- Comments generated: 1
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| sync "sync" | ||
|
|
||
| typed "sigs.k8s.io/structured-merge-diff/v6/typed" | ||
| typed "sigs.k8s.io/structured-merge-diff/v7/typed" |
5272e2e to
ea54208
Compare
Merging this branch will not change overall coverage
Coverage by fileChanged files (no unit tests)
Please note that the "Total", "Covered", and "Missed" counts above refer to code statements instead of lines of code. The value in brackets refers to the test coverage of that file in the old version of the code. |
There was a problem hiding this comment.
🔵 Needs a closer look
The generated parser change causes type incompatibilities with the current managedfields API and will fail to compile.
Review details
Files not reviewed (1)
- applyconfigurations/internal/internal.go: Generated file
Suppressed comments (1)
applyconfigurations/internal/internal.go:9
- This generated package now exposes
*structured-merge-diff/v7/typed.Parser, but thek8s.io/apimachinery v0.37.0managedfieldsAPI is built against the v6 typed package (the retained v6 indirect requirement is evidence of that dependency). As a result, calls such asmanagedfields.NewSchemeTypeConverter(scheme, internal.Parser())andmanagedfields.ExtractInto(..., internal.Parser().Type(...), ...)will fail to compile because the v7 types are not assignable to the v6 types. Keep the generated code on v6, or upgrade the Kubernetes dependency set to a release whose managedfields API uses v7 before switching this import.
- Files reviewed: 1/3 changed files
- Comments generated: 0 new
- Review effort level: Lite
This PR contains the following updates:
v6.4.2→v7.0.0Release Notes
kubernetes-sigs/structured-merge-diff (sigs.k8s.io/structured-merge-diff/v6)
v7.0.0Compare Source
Configuration
📅 Schedule: (UTC)
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.