Skip to content

webhook: bump k8s API to v0.35.0 to support restartPolicyRules - #579

Merged
kubernetes-prow[bot] merged 1 commit into
kubernetes-sigs:masterfrom
rzlink:bump-webhook-api-v0.37
Jul 9, 2026
Merged

webhook: bump k8s API to v0.35.0 to support restartPolicyRules#579
kubernetes-prow[bot] merged 1 commit into
kubernetes-sigs:masterfrom
rzlink:bump-webhook-api-v0.37

Conversation

@rzlink

@rzlink rzlink commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

What this does

Bumps the hyper-v mutating webhook's Kubernetes dependencies from v0.31.0 to
v0.35.0 so the typed corev1.Container recognizes the container-level
restartPolicyRules field (KEP-5307 ContainerRestartRules). The older API
lacked the field, so the typed decode path silently dropped it.

  • go.mod: k8s.io/{api,apimachinery,client-go,apiextensions-apiserver} v0.31.0 → v0.35.0; controller-runtime v0.19.0 → v0.23.0; go 1.24.0 → 1.25.0
  • helpers/Makefile + Dockerfile.hyperv: build Go 1.24 → 1.25 (required by controller-runtime v0.23.0)
  • webhook_test.go: add TestTypedAPIKnowsRestartPolicyRules

v0.35.0 is the earliest release that contains the field and pairs with
controller-runtime v0.23.0, keeping the required Go toolchain at 1.25.
go build, go vet, and go test ./... pass.

Complements #576 (which preserves the field via raw-JSON mutation) by keeping
the typed decode path current as well.

@kubernetes-prow
kubernetes-prow Bot requested a review from marosset July 7, 2026 22:37
@kubernetes-prow kubernetes-prow Bot added the tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges. label Jul 7, 2026
@kubernetes-prow
kubernetes-prow Bot requested a review from zylxjtu July 7, 2026 22:37
@kubernetes-prow kubernetes-prow Bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Jul 7, 2026
Bump the hyper-v mutating webhook's Kubernetes dependencies from v0.31.0 to
v0.35.0, so the typed corev1.Container knows the container-level
restartPolicyRules field (KEP-5307 ContainerRestartRules). Previously the
typed API lacked it, which is the class of bug the raw-JSON mutation fix
already guards against; this keeps the typed decode path current too. v0.35.0
is the earliest release with the field and pairs with controller-runtime
v0.23.0, keeping the required Go toolchain at 1.25.

- go.mod: k8s.io/{api,apimachinery,client-go,apiextensions-apiserver}
  v0.31.0 -> v0.35.0; controller-runtime v0.19.0 -> v0.23.0; go 1.24.0 -> 1.25.0
- helpers/Makefile + Dockerfile.hyperv: build Go 1.24 -> 1.25
  (controller-runtime v0.23.0 requires go 1.25)
- webhook_test.go: add TestTypedAPIKnowsRestartPolicyRules asserting the typed
  API decodes/round-trips restartPolicyRules

Signed-off-by: Dawei Wei <wei.dawei.cn@gmail.com>
@rzlink
rzlink force-pushed the bump-webhook-api-v0.37 branch from 443ff86 to 5d61565 Compare July 8, 2026 00:36
@rzlink rzlink changed the title webhook: bump k8s API to v0.36.2 to support restartPolicyRules webhook: bump k8s API to v0.35.0 to support restartPolicyRules Jul 8, 2026
@zylxjtu

zylxjtu commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

/lgtm
/approve

@kubernetes-prow kubernetes-prow Bot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jul 9, 2026
@kubernetes-prow

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: linliu-msft-git, rzlink, zylxjtu

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kubernetes-prow kubernetes-prow Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jul 9, 2026
@kubernetes-prow
kubernetes-prow Bot merged commit 7f991a6 into kubernetes-sigs:master Jul 9, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants