Merge pull request #36 from Financial-Times/fix/CPH-331-antiaffinity

Distribute load between nodes in cluster
Financial-Times · Jan 5, 2021 · f41fdec · f41fdec
2 parents 81d404b + 9178f85
commit f41fdec
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 2 deletions.
diff --git a/helm/internal-content-api/templates/deployment.yaml b/helm/internal-content-api/templates/deployment.yaml
@@ -28,6 +28,16 @@ spec:
                 values:
                 - {{ .Values.service.name }}
             topologyKey: "kubernetes.io/hostname"
+          preferredDuringSchedulingIgnoredDuringExecution:
+          - weight: 100
+            podAffinityTerm:
+              labelSelector:
+                matchExpressions:
+                - key: app
+                  operator: In
+                  values:
+                  - content-search-api-port
+              topologyKey: "kubernetes.io/hostname"
       containers:
       - name: {{ .Values.service.name }}
         image: "{{ .Values.image.repository }}:{{ .Chart.Version }}"

diff --git a/helm/internal-content-api/values.yaml b/helm/internal-content-api/values.yaml
@@ -10,7 +10,8 @@ image:
   pullPolicy: IfNotPresent
 resources:
   requests:
-    memory: 20Mi
+    memory: 14Mi
+    cpu: 64m
   limits:
     memory: 128Mi
 env: # environment variable for the app
@@ -31,5 +32,5 @@ env: # environment variable for the app
   CONTENT_UNROLLER_APP_NAME: ""
   CONTENT_UNROLLER_APP_HEALTH_URI: ""
   CONTENT_UNROLLER_APP_PANIC_GUIDE: ""
-  CONTENT_UNROLLER_APP_BUSINESS_IMPACT: ""  
+  CONTENT_UNROLLER_APP_BUSINESS_IMPACT: ""
   APP_SYSTEM_CODE: ""