Adding resources on all containers

stakater · May 19, 2022 · ce999e9 · ce999e9
1 parent 7627728
commit ce999e9
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/charts/ingressmonitorcontroller/templates/deployment.yaml b/charts/ingressmonitorcontroller/templates/deployment.yaml
@@ -59,9 +59,9 @@ spec:
         envFrom:
         {{- toYaml .Values.envFrom | nindent 8 }}
         {{- end }}
-        {{- if .Values.kube-rbac-proxy.image.resources }}
+        {{- if .Values.resources }}
          resources:
-{{ toYaml .Values.kube-rbac-proxy.image.resources | indent 10 }}
+{{ toYaml .Values.resources | indent 10 }}
         {{- end }}   
         name: manager
         image: "{{ .Values.image.repository }}:{{ .Values.image.tag | default .Chart.AppVersion }}"
@@ -78,8 +78,10 @@ spec:
             port: 8081
           initialDelaySeconds: 5
           periodSeconds: 10
+        {{- if .Values.resources }}
         resources:
-          {{- toYaml .Values.resources | nindent 10 }}
+{{ toYaml .Values.resources | indent 10 }}
+        {{- end }}   
       terminationGracePeriodSeconds: 10
       {{- with .Values.nodeSelector }}
       nodeSelector:

diff --git a/charts/ingressmonitorcontroller/values.yaml b/charts/ingressmonitorcontroller/values.yaml
@@ -15,8 +15,6 @@ kube-rbac-proxy:
     repository: gcr.io/kubebuilder/kube-rbac-proxy
     tag: v0.8.0
     pullPolicy: IfNotPresent
-    resources:
-      {}
 
 # Partial override for ingress-monitor-controller.fullname template (will keep the release name)
 nameOverride: ""