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
apiextensions: validate list-type and map-type "atomic" #84722
apiextensions: validate list-type and map-type "atomic" #84722
Conversation
sttts
commented
Nov 4, 2019
This PR may require API review. If so, when the changes are ready, complete the pre-review checklist and request an API review. Status of requested reviews is tracked in the API Review project. |
/assign @apelisse I need you confirmation that my understanding of "atomic" is correct here. |
staging/src/k8s.io/apiextensions-apiserver/pkg/apis/apiextensions/validation/atomic.go
Outdated
Show resolved
Hide resolved
c2e14f9
to
5b13a54
Compare
/milestone v1.17 |
19b1ab1
to
4570038
Compare
4570038
to
61ae424
Compare
staging/src/k8s.io/apiextensions-apiserver/pkg/apis/apiextensions/validation/validation.go
Show resolved
Hide resolved
staging/src/k8s.io/apiextensions-apiserver/pkg/apis/apiextensions/validation/validation.go
Outdated
Show resolved
Hide resolved
staging/src/k8s.io/apiextensions-apiserver/pkg/apis/apiextensions/validation/validation.go
Outdated
Show resolved
Hide resolved
one comment on nil check, lgtm otherwise |
61ae424
to
b3b15a9
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: liggitt, sttts 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 |
/retest Review the full test history for this PR. Silence the bot with an |
release note should clarify this is for CRs and should use the actual API field names (x-kubernetes-…) |
…cation In kube 1.17, the openapi structural schema validation was updated to validate CRDs according to the documented API semantics of x-kubernetes-list-type and x-kubernetes-map-type atomic to reject non-atomic sub-types. This PR updates the API types to follow the specification. See kubernetes/kubernetes#84722 for more details.
…cation In kube 1.17, the openapi structural schema validation was updated to validate CRDs according to the documented API semantics of x-kubernetes-list-type and x-kubernetes-map-type atomic to reject non-atomic sub-types. This PR updates the API types to follow the specification. See kubernetes/kubernetes#84722 for more details.