add resources to gateway and scheduler containers (#1357)

* add resources to gateway and scheduler containers Signed-off-by: Akihiko Kuroda <akihikokuroda2020@gmail.com>
Qiskit · Jun 7, 2024 · 3fc78e4 · 3fc78e4
1 parent babc84e
commit 3fc78e4
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 11 deletions.
diff --git a/.github/workflows/kubernetes-deploy.yaml b/.github/workflows/kubernetes-deploy.yaml
@@ -53,6 +53,8 @@ jobs:
             --set gateway.application.ray.cpu=1 \
             --set gateway.application.limits.keepClusterOnComplete=false \
             .
+          GATEWAY=$(kubectl get pod -l app.kubernetes.io/name=gateway -o name)
+          kubectl wait --for=condition=Ready "$GATEWAY" --timeout 5m 
       - name: setup python
         uses: actions/setup-python@v4
         with:

diff --git a/charts/qiskit-serverless/charts/gateway/templates/deployment.yaml b/charts/qiskit-serverless/charts/gateway/templates/deployment.yaml
@@ -78,6 +78,8 @@ spec:
             httpGet:
               path: /readiness
               port: http
+            initialDelaySeconds: 60
+            periodSeconds: 20
           volumeMounts:
             - mountPath: "/usr/src/app/media/"
               name: gateway-pv-storage
@@ -253,6 +255,8 @@ spec:
               name: gateway-pv-storage
             - mountPath: "/tmp/templates/"
               name: ray-cluster-template
+          resources:
+            {{- toYaml .Values.scheduler.resources | nindent 12 }}
           env:
             - name: DEBUG
               value: {{ .Values.application.debug | quote }}

diff --git a/charts/qiskit-serverless/charts/gateway/values.yaml b/charts/qiskit-serverless/charts/gateway/values.yaml
@@ -145,17 +145,21 @@ 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:
+  limits:
+    cpu: "1500m"
+    memory: "1Gi"
+  requests:
+    cpu: "500m"
+    memory: "700Mi"
+scheduler:
+  resources:
+    limits:
+      cpu: "1500m"
+      memory: "270Mi"
+    requests:
+      cpu: "100m"
+      memory: "180Mi"
 
 autoscaling:
   enabled: false