mirror of
https://github.com/helm/charts.git
synced 2026-08-23 22:37:45 +00:00
[incubator/fluentd-cloudwatch] Remove default resources requests and limits (#13181)
Resources should not be limited by default because nobody knows where the chart will be installed to. Especially worth not to set CPU limits by default as per https://github.com/kubernetes/kubernetes/issues/67577 Signed-off-by: Eugene Glotov <kivagant@gmail.com>
This commit is contained in:
committed by
Kubernetes Prow Robot
parent
81e1d29578
commit
d7d9513fd2
@@ -1,6 +1,6 @@
|
||||
apiVersion: v1
|
||||
name: fluentd-cloudwatch
|
||||
version: 0.8.1
|
||||
version: 0.9.0
|
||||
appVersion: v1.3.3-debian-cloudwatch-1.0
|
||||
description: A Fluentd CloudWatch Helm chart for Kubernetes.
|
||||
home: https://www.fluentd.org/
|
||||
|
||||
@@ -9,13 +9,17 @@ image:
|
||||
## Configure resource requests and limits
|
||||
## ref: http://kubernetes.io/docs/user-guide/compute-resources/
|
||||
##
|
||||
resources:
|
||||
limits:
|
||||
cpu: 100m
|
||||
memory: 200Mi
|
||||
requests:
|
||||
cpu: 100m
|
||||
memory: 200Mi
|
||||
resources: {}
|
||||
# We usually recommend not to specify default resources and to leave this as a conscious
|
||||
# choice for the user. This also increases chances charts run on environments with little
|
||||
# resources, such as Minikube. If you do want to specify resources, uncomment the following
|
||||
# lines, adjust them as necessary, and remove the curly braces after 'resources:'.
|
||||
# limits:
|
||||
# cpu: 100m
|
||||
# memory: 200Mi
|
||||
# requests:
|
||||
# cpu: 100m
|
||||
# memory: 200Mi
|
||||
|
||||
# hostNetwork: false
|
||||
|
||||
|
||||
Reference in New Issue
Block a user