-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Added documentation about recommended Kubernetes labels #9854
Conversation
Signed-off-by: Paolo Patierno <ppatierno@live.com>
Signed-off-by: Paolo Patierno <ppatierno@live.com>
Signed-off-by: Paolo Patierno <ppatierno@live.com>
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.
Thanks Paolo. I've left some suggestions. I thought it might be better to describe the constraints earlier.
@ppatierno -- I wonder if we should move this section to sit as a sub-section under here instead? It's in the same chapter, but has content related to Kubernetes resources. |
@PaulRMellor I see your point but it's about "customizing" so still same fear like the title. |
@PaulRMellor I noticed your thumbs up on my latest comment about leaving the PR as it is. Are you going to approve as well? Or do you have any other comments? Did you make any changes based on my suggestions? Don't see any? |
Signed-off-by: Paolo Patierno <ppatierno@live.com>
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.
Thanks
Signed-off-by: Paolo Patierno <ppatierno@live.com>
The |
Signed-off-by: Paolo Patierno <ppatierno@live.com> Signed-off-by: Steffen Karlsson <steffen.karlsson@maersk.com>
Signed-off-by: Paolo Patierno <ppatierno@live.com> Signed-off-by: Steffen Karlsson <steffen.karlsson@maersk.com>
Signed-off-by: Paolo Patierno <ppatierno@live.com>
This PR adds some documentation around the recommended Kubernetes labels that we set on operands and cannot be overwritten by users when using templates.
This is related to #9735