newrelic / helm-charts

Helm charts for New Relic applications
Apache License 2.0
95 stars 201 forks source link

Remove resources cpu limits default (newrelic-logging) #1377

Open ciechonp opened 1 month ago

ciechonp commented 1 month ago

Description

We would like to remove cpu limits default from newrelic-logging helm chart as it is best practise for setting resources. It is also preventing us for doing so as we are using nri-bundle helm chart and because of helm bug related to dependent helm charts it is impossible to remove it. https://github.com/newrelic/helm-charts/blob/master/charts/newrelic-logging/values.yaml#L248

Acceptance Criteria

Remove cpu limit from newrelic-logging helm chart defaults

workato-integration[bot] commented 1 month ago

https://new-relic.atlassian.net/browse/NR-273815