chore: add resources limits and requests to containers (#329)

Signed-off-by: SdgJlbl <sarah.diot-girard@owkin.com>
Substra · May 31, 2024 · 44afc50 · 44afc50
1 parent 1d67af1
commit 44afc50
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 13 deletions.
diff --git a/charts/substra-frontend/CHANGELOG.md b/charts/substra-frontend/CHANGELOG.md
@@ -2,6 +2,13 @@
 
 <!-- towncrier release notes start -->
 
+## [1.1.0] - 2024-04-05
+
+### Added
+
+- Resources limits and requests (CPU and memory) for all containers.
+
+
 ## [1.0.28] - 2024-03-27
 
 
@@ -21,7 +28,7 @@
 ### Changed
 
 - Upgrade AppVersion to 0.47.0
-  
+
 ## 1.0.25
 
 ### Changed

diff --git a/charts/substra-frontend/Chart.yaml b/charts/substra-frontend/Chart.yaml
@@ -10,6 +10,6 @@ type: application
 maintainers:
   - name: Substra Team
     email: support@substra.org
-version: 1.0.28
+version: 1.1.0
 appVersion: 0.49.0 # should be same as in package.json
 kubeVersion: '>= 1.19.0-0'
diff --git a/charts/substra-frontend/values.yaml b/charts/substra-frontend/values.yaml
@@ -50,17 +50,14 @@ 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:
+  requests:
+    memory: "200Mi"
+    cpu: "100m"
+  limits:
+    memory: "800Mi"
+    cpu: "100m"
+
 
 nodeSelector: {}