kubernetes/kubernetes

1.30 tag also breaks PodIP.IP (which should be marked required)

tatsuhiro-t opened this issue · 3 comments

What happened?

We have a CRD that contains PodStatus. After generating CRD with k8s v1.30.1, applying it failed with the message:

spec.validation.openAPIV3Schema.properties[status].properties[podIPs].items.properties[ip].default: Required value: this property is in x-kubernetes-list-map-keys, so it must have a default or be a required property

Similar issue reported in #124540 but different struct field.

What did you expect to happen?

CRD is applied successfully.

How can we reproduce it (as minimally and precisely as possible)?

Create CRD that has PodStatus and then apply it.

Anything else we need to know?

This is a leftover of #124553
PodIP.IP should also be marked required because it is +listMapKey=ip in PodStatus.PodIPs.

Kubernetes version

$ kubectl version
Client Version: v1.30.0
Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3

Cloud provider

OS version

# On Linux:
$ cat /etc/os-release
# paste output here
$ uname -a
# paste output here

# On Windows:
C:\> wmic os get Caption, Version, BuildNumber, OSArchitecture
# paste output here

Install tools

Container runtime (CRI) and version (if applicable)

Related plugins (CNI, CSI, ...) and versions (if applicable)

This issue is currently awaiting triage.

If a SIG or subproject determines this is a relevant issue, they will accept it by applying the triage/accepted label and provide further guidance.

The triage/accepted label can be added by org members by writing /triage accepted in a comment.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

/sig api-machinery