-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
KEP: Custom Resource Field Selectors #4359
Conversation
Skipping CI for Draft Pull Request. |
ef8d169
to
6ee44a0
Compare
8d013cc
to
7142484
Compare
keps/sig-api-machinery/4358-custom-resource-field-selectors/README.md
Outdated
Show resolved
Hide resolved
keps/sig-api-machinery/4358-custom-resource-field-selectors/README.md
Outdated
Show resolved
Hide resolved
keps/sig-api-machinery/4358-custom-resource-field-selectors/README.md
Outdated
Show resolved
Hide resolved
keps/sig-api-machinery/4358-custom-resource-field-selectors/README.md
Outdated
Show resolved
Hide resolved
keps/sig-api-machinery/4358-custom-resource-field-selectors/README.md
Outdated
Show resolved
Hide resolved
keps/sig-api-machinery/4358-custom-resource-field-selectors/README.md
Outdated
Show resolved
Hide resolved
keps/sig-api-machinery/4358-custom-resource-field-selectors/README.md
Outdated
Show resolved
Hide resolved
keps/sig-api-machinery/4358-custom-resource-field-selectors/README.md
Outdated
Show resolved
Hide resolved
afce6cb
to
7c3f593
Compare
7c3f593
to
86af44c
Compare
keps/sig-api-machinery/4358-custom-resource-field-selectors/README.md
Outdated
Show resolved
Hide resolved
"org.example.v1.CustomResource": { | ||
"type": "object", | ||
"x-kubernetes-group-version-kind": [ ... ], | ||
"x-kubernetes-selectable-fields": [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm on openapi extension naming, added one question comment
keps/sig-api-machinery/4358-custom-resource-field-selectors/README.md
Outdated
Show resolved
Hide resolved
the KEP lgtm, but needs comments on the openapi changes from someone with more familiarity. I've suggested Jeffrey and Stefan. |
Hi, could it support context: kube-state-metrics collects custom resource metrics. Some users use |
keps/sig-api-machinery/4358-custom-resource-field-selectors/README.md
Outdated
Show resolved
Hide resolved
keps/sig-api-machinery/4358-custom-resource-field-selectors/README.md
Outdated
Show resolved
Hide resolved
keps/sig-api-machinery/4358-custom-resource-field-selectors/README.md
Outdated
Show resolved
Hide resolved
"org.example.v1.CustomResource": { | ||
"type": "object", | ||
"x-kubernetes-group-version-kind": [ ... ], | ||
"x-kubernetes-selectable-fields": [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm on openapi extension naming, added one question comment
keps/sig-api-machinery/4358-custom-resource-field-selectors/README.md
Outdated
Show resolved
Hide resolved
keps/sig-api-machinery/4358-custom-resource-field-selectors/README.md
Outdated
Show resolved
Hide resolved
c2b2178
to
dde20fa
Compare
discussion looks resolved. Design and PRR both lgtm /lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: deads2k, jpbetz 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 |
Implementation: kubernetes/kubernetes#122717