search

nextcloud / helm

A community maintained helm chart for deploying Nextcloud on Kubernetes.
GNU Affero General Public License v3.0
332 stars 269 forks source link

PriorityClassName missing from deployment #178

Open timbrd opened 2 years ago

timbrd commented 2 years ago

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.

2fst4u commented 1 year ago

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.

jessebot commented 3 months ago

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