helm
helm copied to clipboard
PriorityClassName missing from deployment
I would like to define a priorityClassName for the nextcloud deployment and the cronjob. Pod priority ensures that the pod is not being evicted by other pods.
One minor point, pod Priority doesn't affect the likelihood of being evicted. QoS does that.
But yes, this chart need a priorityclass field. Not having one is bad design.
This sounds good! We would be happy to review a PR for this feature :) You can read the contributing doc here: https://github.com/nextcloud/helm/blob/main/CONTRIBUTING.md#pull-requests