Merge pull request #2 from nico151999/feat/resources

feat: add resource quotas to s3 jobs
mcsps · Apr 3, 2024 · bcdc0c5 · bcdc0c5
2 parents 2bbe93d + 6da6327
commit bcdc0c5
Show file tree

Hide file tree

Showing 5 changed files with 142 additions and 117 deletions.
diff --git a/charts/s3-register/Chart.yaml b/charts/s3-register/Chart.yaml
@@ -1,9 +1,9 @@
 apiVersion: v2
-appVersion: "0.0.1"
+appVersion: "0.0.2"
 description: Job for create a bucket and a user on S3 backend
 maintainers:
   - name: eumel8 # Frank Kloeker
     email: [email protected]
     url: https://www.telekom.com
 name: s3-register
-version: 0.0.1
+version: 0.0.2
diff --git a/charts/s3-register/templates/register.yaml b/charts/s3-register/templates/register.yaml
@@ -64,6 +64,8 @@ spec:
         image:  {{ .Values.image }}
         imagePullPolicy: Always
         name: register
+        resources:
+          {{- .Values.resources | toYaml | nindent 10 }}
         env:
         - name: S3URL
           value: {{ .Values.s3.url }}
@@ -124,6 +126,8 @@ spec:
         image:  {{ .Values.image }}
         imagePullPolicy: Always
         name: register
+        resources:
+          {{- .Values.resources | toYaml | nindent 10 }}
         env:
         - name: S3URL
           value: {{ .Values.s3.url }}

diff --git a/charts/s3-register/values.yaml b/charts/s3-register/values.yaml
@@ -15,3 +15,11 @@ s3:
 # bucket:
 # accesskey:
 # secretkey:
+
+resources:
+  limits:
+    cpu: "500m"
+    memory: "1Gi"
+  requests:
+    cpu: "200m"
+    memory: "256Mi"