From df67548ed016626b9216934501e555f079be0ebd Mon Sep 17 00:00:00 2001
From: Macharia Muguku <machariamuguku@gmail.com>
Date: Mon, 18 Sep 2023 02:29:32 +0300
Subject: [PATCH] add resource requests and limits for airbyte minio container

---
 charts/airbyte/README.md            |  2 ++
 charts/airbyte/templates/minio.yaml |  3 +++
 charts/airbyte/values.yaml          | 20 ++++++++++++++++++++
 3 files changed, 25 insertions(+)

diff --git a/charts/airbyte/README.md b/charts/airbyte/README.md
index f9df04ca652..e1e7c9ed394 100644
--- a/charts/airbyte/README.md
+++ b/charts/airbyte/README.md
@@ -152,6 +152,8 @@ Helm chart to deploy airbyte
 | minio.image.repository | string | `"minio/minio"`                      |  |
 | minio.image.tag | string | `"latest"`                           |  |
 | minio.storage.volumeClaimValue | string | `500Mi`                              |  |
+| minio.resources.limits | object | `{}`                                 |  |
+| minio.resources.requests | object | `{}`                                 |  |
 | nameOverride | string | `""`                                 |  |
 | pod-sweeper.affinity | object | `{}`                                 |  |
 | pod-sweeper.containerSecurityContext | object | `{}`                                 |  |
diff --git a/charts/airbyte/templates/minio.yaml b/charts/airbyte/templates/minio.yaml
index f25068d4882..ffc8e7db86d 100644
--- a/charts/airbyte/templates/minio.yaml
+++ b/charts/airbyte/templates/minio.yaml
@@ -48,6 +48,9 @@ spec:
           volumeMounts:
             - name: airbyte-minio-pv-claim # must match the volume name, above
               mountPath: "/data"
+          {{- if .Values.minio.resources }}
+          resources: {{- toYaml .Values.minio.resources | nindent 12 }}
+          {{- end }}
       {{- with .Values.minio.nodeSelector }}
       nodeSelector:
         {{- toYaml . | nindent 8 }}
diff --git a/charts/airbyte/values.yaml b/charts/airbyte/values.yaml
index d1410db3db3..c30e06ffc92 100644
--- a/charts/airbyte/values.yaml
+++ b/charts/airbyte/values.yaml
@@ -1308,6 +1308,26 @@ minio:
   storage:
     volumeClaimValue: 500Mi
 
+  ## Minio resource requests and limits
+  ## ref: http://kubernetes.io/docs/user-guide/compute-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:'.
+  ##  minio.resources.limits [object] The resources limits for minio pod(s)
+  ##  minio.resources.requests [object] The requested resources for minio pod(s)
+  resources:
+    ## Example:
+    ## requests:
+    ##    memory: 256Mi
+    ##    cpu: 250m
+    requests: {}
+    ## Example:
+    ## limits:
+    ##    cpu: 200m
+    ##    memory: 1Gi
+    limits: {}
+
 ## @section cron parameters
 
 cron: