-
Notifications
You must be signed in to change notification settings - Fork 14.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
Improvement for k8s.io/docs/concepts/policy/limit-range/ #46899
Comments
/language en |
Mentioning the namespace in a |
We can also add a comment into the manifest about the namespace being implied. Using a comment helps people test things out in namespaces other than |
Hey @vsagar100 I think It's already mentioned clearly about the constraints and the steps of using it in |
@Shubhdeep02, I am talking about the below page. It might have been mentioned on the page you mentioned. I was following LimitRange page but found it missing hence requested update. |
/assign |
/triage accepted |
PR working on this #46950 |
@subhals, |
@T-Lakshmi Thank you! Please feel free to pull since I am not working on this. |
@subhals, |
/assign |
PR Raised #48766 |
LimitRange is namespace specific, however the given example does not contain namespace attribute. I know it will pick default namespace however mentioning it will be more clear & helpful for beginners like me.
The text was updated successfully, but these errors were encountered: