-
Notifications
You must be signed in to change notification settings - Fork 216
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
Update comments and suggested defaults #651
Conversation
I haven't created an issue, as it seems to be an easy fix. However, if that's the proper workflow, and if these changes are worth going forward, I can do that. |
It seems like documentation was not updated on this MR, so you will see one related line being removed in this MR. |
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! ❤️
Could you sign the commit please? |
Signed-off-by: Kamil Aliev <[email protected]>
Now done :) |
Signed-off-by: Kamil Aliev <[email protected]>
Signed-off-by: Kamil Aliev <[email protected]> Signed-off-by: Jorge Turrado <[email protected]>
* fix: labels on cloudeventsources CRD should be the same as other CRDs (#656) Signed-off-by: Joel Dimbernat <[email protected]> Signed-off-by: Jorge Turrado <[email protected]> * chore: Update comments and suggested defaults (#651) Signed-off-by: Kamil Aliev <[email protected]> Signed-off-by: Jorge Turrado <[email protected]> * fix: broken "| default" logic for automountServiceAccountToken (#665) When setting (.Values.serviceAccount.operator).automountServiceAccountToken to boolean false, the "| default" part kicks in. Fix is strongly inspired from helm/helm#12080 (comment) Signed-off-by: François Blondel <[email protected]> Co-authored-by: François Blondel <[email protected]> Signed-off-by: Jorge Turrado <[email protected]> * fix: add forgotten crds.additionalAnnotations to crd-cloudeventsources.yaml (#668) Signed-off-by: François Blondel <[email protected]> Co-authored-by: François Blondel <[email protected]> Signed-off-by: Jorge Turrado <[email protected]> * Prepare release v2.14.3 Signed-off-by: Jorge Turrado <[email protected]> --------- Signed-off-by: Joel Dimbernat <[email protected]> Signed-off-by: Jorge Turrado <[email protected]> Signed-off-by: Kamil Aliev <[email protected]> Signed-off-by: François Blondel <[email protected]> Co-authored-by: JoelDimbernat <[email protected]> Co-authored-by: Kamil Aliev <[email protected]> Co-authored-by: François Blondel <[email protected]> Co-authored-by: François Blondel <[email protected]>
It seems that currently the description about root affinity setting is not correct, based on template here.
Also I assume that suggested root affinity would like to split all components across the nodes, not only operator and metrics-server.
Lastly for consistency I applied name of the admissions webhook to its affinity suggestion to follow similar suggestions for operator and metrics-server.
Checklist