Add resource limits to containers (#223)

ascoderu · Aug 25, 2019 · ee30adb · ee30adb
1 parent 893549d
commit ee30adb
Show file tree

Hide file tree

Showing 4 changed files with 28 additions and 4 deletions.
diff --git a/helm/templates/api-deployment.yaml b/helm/templates/api-deployment.yaml
@@ -51,6 +51,12 @@ spec:
               key: REGISTRATION_PASSWORD
         ports:
         - containerPort: 8080
-        resources: {}
+        resources:
+          limits:
+            memory: "512Mi"
+            cpu: "500m"
+          requests:
+            memory: "256Mi"
+            cpu: "250m"
       restartPolicy: Always
 status: {}
diff --git a/helm/templates/flower-deployment.yaml b/helm/templates/flower-deployment.yaml
@@ -32,6 +32,12 @@ spec:
               key: FLOWER_PASSWORD
         ports:
         - containerPort: 8080
-        resources: {}
+        resources:
+          limits:
+            memory: "256Mi"
+            cpu: "250m"
+          requests:
+            memory: "128Mi"
+            cpu: "125m"
       restartPolicy: Always
 status: {}
diff --git a/helm/templates/nginx-deployment.yaml b/helm/templates/nginx-deployment.yaml
@@ -34,7 +34,13 @@ spec:
           value: "{{ .Release.Name }}-api:8080"
         ports:
         - containerPort: 8888
-        resources: {}
+        resources:
+          limits:
+            memory: "128Mi"
+            cpu: "250m"
+          requests:
+            memory: "64Mi"
+            cpu: "125m"
       - name: dnsmasq
         image: "janeczku/go-dnsmasq:release-1.0.7"
         args:

diff --git a/helm/templates/worker-deployment.yaml b/helm/templates/worker-deployment.yaml
@@ -26,6 +26,12 @@ spec:
         {{ include "opwen.environment.shared" . }}
         ports:
         - containerPort: 80
-        resources: {}
+        resources:
+          limits:
+            memory: "512Mi"
+            cpu: "500m"
+          requests:
+            memory: "256Mi"
+            cpu: "250m"
       restartPolicy: Always
 status: {}