helm icon indicating copy to clipboard operation
helm copied to clipboard

PriorityClassName missing from deployment

Open timbrd opened this issue 4 years ago • 2 comments

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.

timbrd avatar Dec 25 '21 11:12 timbrd

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.

2fst4u avatar May 18 '23 07:05 2fst4u

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

jessebot avatar Jul 26 '24 07:07 jessebot