NetApp / trident

Storage orchestrator for containers
Apache License 2.0
762 stars 222 forks source link

No limits or requests set on Trident resources #853

Open cjreyn opened 1 year ago

cjreyn commented 1 year ago

Describe the bug No limits or requests are set for any Trident pods except the operator pod. This is bad practice, as an issue in Trident could take down the cluster.

Environment Provide accurate information about the environment to help us reproduce the issue.

To Reproduce Install trident with the helm chart. Query the daemonset named "daemonset.apps/trident-node-linux" and all pods it starts. No sign of limits or requests.

Expected behavior The daemonset should set some sensible cpu, mem, and ephemeral storage limits and requests.

Additional context Add any other context about the problem here.

m0untains commented 2 months ago

We bumped into this issue as well, as we have a number of cluster policies (i.e. kyverno) for enforcing various standards around how resources and limits are set across the cluster.

It would be great if the helm chart let us specify (override) the resource block for each pod deployed (i.e. app, daemonset, and all sidecars). This flexibility is a fairly common paradigm across the k8s helm ecosystem, so we were surprised to not find it here.

sjpeeris commented 3 weeks ago

Hi @cjreyn Please let us know if this issue still exists with the newer versions of Trident. If this has been resolved, please close the issue.

cjreyn commented 3 weeks ago

Thanks for addressing! I'll wait for it to trickle into a tagged release unless I find time to roll out the fixed branch on our test cluster.