From f3b261e4e2e1ae9d6409fde962aff5829930f40c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Julian=20B=C3=A4ume?= <julian.baeume@open-xchange.com>
Date: Fri, 17 Dec 2021 16:38:55 +0100
Subject: [PATCH] [CTR-159] Default resource requests and limits

since we don't expect admins to collect metrics for those services,
we provide them with some sensible defaults.
---
 helm/core-manifest-service/values.yaml | 20 +++++++++-----------
 1 file changed, 9 insertions(+), 11 deletions(-)

diff --git a/helm/core-manifest-service/values.yaml b/helm/core-manifest-service/values.yaml
index a87d172..968b74f 100644
--- a/helm/core-manifest-service/values.yaml
+++ b/helm/core-manifest-service/values.yaml
@@ -58,17 +58,15 @@ ingress:
   #    hosts:
   #      - chart-example.local
 
-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: 128Mi
-  # requests:
-  #   cpu: 100m
-  #   memory: 128Mi
+resources:
+  # We recommend to always specify resources after actually measuring the values under
+  # production-like circumstances.
+  limits:
+    cpu: 1
+    memory: 256Mi
+  requests:
+    cpu: 500m
+    memory: 196Mi
 
 autoscaling:
   enabled: false
-- 
GitLab