Add resource and fix matchLabel

- Add mem and CPU limit/req - matchLabel are immutable
rhpds · Jun 13, 2023 · 284aeeb · 284aeeb
1 parent f1b6707
commit 284aeeb
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 1 deletion.
diff --git a/deploy/helm-api/templates/deployment.yaml b/deploy/helm-api/templates/deployment.yaml
@@ -11,7 +11,6 @@ spec:
   selector:
     matchLabels:
       app.kubernetes.io/name: {{ .Values.appName }}
-      app.kubernetes.io/version: {{ .Chart.AppVersion | quote }}
   strategy:
     type: {{ .Values.deployment.strategy }}
   template:
@@ -24,6 +23,8 @@ spec:
       - name: sandbox-api
         image: {{ .Values.deployment.image }}:{{ .Values.image_tag | default .Values.deployment.tag }}
         imagePullPolicy: {{ .Values.deployment.pullPolicy }}
+        resources:
+          {{- toYaml .Values.resources | nindent 10 }}
         livenessProbe:
           initialDelaySeconds: 10
           tcpSocket:

diff --git a/deploy/helm-api/values.yaml b/deploy/helm-api/values.yaml
@@ -15,3 +15,11 @@ service:
 
 dynamodb_table: accounts-dev
 dynamodb_region: us-east-1
+
+resources:
+  limits:
+    cpu: 1
+    memory: 1Gi
+  requests:
+    cpu: 50m
+    memory: 128Mi
diff --git a/readme.adoc b/readme.adoc
@@ -56,6 +56,11 @@ sandbox_api_secrets:
 helm install  -f secrets.yaml sandbox-api deploy/helm-api/
 ----
 
+.Upgrade chart
+----
+helm upgrade  -f secrets.yaml sandbox-api deploy/helm-api/
+----
+
 To initialize or update the postgresql schema, use the following:
 
 .Run the DB migration