-
Notifications
You must be signed in to change notification settings - Fork 14.4k
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
Glossary list of core objects is wrong #45972
Comments
We should fix this. /priority backlog |
/language en |
PRs that cover just one entry at a time are very welcome. |
/assign |
In general, do the glossary categories correspond to the API categories? If they aren't, should they be so that new additions are easier to put into buckets? If they are, are we socializing that information well (like with comments in the source directory or style guide)? |
We don't really have API categories. https://kubernetes.io/docs/reference/kubernetes-api/ does have some categories, but (IIRC) this SIG and its tech leads at the time were keen to de-emphasize or drop them. |
Putting APIs into categories or a taxonomy might be useful, but it makes work for maintainers that I think isn't justified by the value you get from it. |
BTW I'm taking “core object” to mean “API kinds built in to Kubernetes”, and not “API kinds in the core API group”. |
@mguan2020 Are you still working on this issue? |
/assign |
As the PR #48333 has been merged so we can close this issue |
/close |
@vaibhav2107: Closing this issue. In response to this:
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. |
This is a Bug Report
Problem:
The glossary listing for core objects includes several things that aren't core object kinds, such as:
kubectl
etc)kubectl
etc)Proposed Solution:
Edit those glossary entries. Optionally add in glossary entries for all core object kinds; for example, NetworkPolicy, Node and PodTemplate should be in the list.
Page to Update:
https://k8s.io/docs/reference/glossary/?core-object=true
Additional Information:
/language en
The text was updated successfully, but these errors were encountered: