Merge pull request #32 from small-hack/feat/add-resource-limits

add resource limits for clusters
small-hack · Mar 30, 2024 · 9d9b167 · 9d9b167
2 parents 041b2a0 + d69fa24
commit 9d9b167
Show file tree

Hide file tree

Showing 4 changed files with 17 additions and 4 deletions.
diff --git a/charts/cloudnative-pg-cluster/Chart.yaml b/charts/cloudnative-pg-cluster/Chart.yaml
@@ -2,7 +2,7 @@ apiVersion: v2
 name: cnpg-cluster
 description: Create postgres tenant clusters managed by the CNPG Operator
 type: application
-version: 0.3.12
+version: 0.3.13
 
 maintainers:
   - name: "cloudymax"

diff --git a/charts/cloudnative-pg-cluster/README.md b/charts/cloudnative-pg-cluster/README.md
@@ -1,6 +1,6 @@
 # cnpg-cluster
 
-![Version: 0.3.12](https://img.shields.io/badge/Version-0.3.12-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square)
+![Version: 0.3.13](https://img.shields.io/badge/Version-0.3.13-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square)
 
 Create postgres tenant clusters managed by the CNPG Operator
 
@@ -43,6 +43,8 @@ Create postgres tenant clusters managed by the CNPG Operator
 | name | string | `"cnpg"` |  |
 | postgresql.pg_hba | list | `["hostnossl all all 0.0.0.0/0 reject","hostssl all all 0.0.0.0/0 cert clientcert=verify-full"]` | records for the pg_hba.conf file. ref: https://www.postgresql.org/docs/current/auth-pg-hba-conf.html |
 | primaryUpdateStrategy | string | `"unsupervised"` |  |
+| resources.limits | object | `{"cpu":1000,"memory":"1024Mi"}` | resource limit for pods |
+| resources.requests | object | `{"cpu":500,"memory":"512Mi"}` | minimum resources guaranteed for pods |
 | scheduledBackup.name | string | `"example-backup"` | name to use for your scheduled backup job |
 | scheduledBackup.spec.backupOwnerReference | string | `"self"` |  |
 | scheduledBackup.spec.cluster.name | string | `"pg-backup"` |  |
@@ -52,4 +54,4 @@ Create postgres tenant clusters managed by the CNPG Operator
 | testApp.enabled | bool | `false` |  |
 
 ----------------------------------------------
-Autogenerated from chart metadata using [helm-docs v1.11.3](https://github.com/norwoodj/helm-docs/releases/v1.11.3)
+Autogenerated from chart metadata using [helm-docs v1.12.0](https://github.com/norwoodj/helm-docs/releases/v1.12.0)
diff --git a/charts/cloudnative-pg-cluster/templates/cnpg_cluster.yaml b/charts/cloudnative-pg-cluster/templates/cnpg_cluster.yaml
@@ -28,6 +28,8 @@ spec:
   postgresql:
     {{- toYaml . | nindent 4 }}
   {{- end }}
+  resources:
+  {{- toYaml .Values.resources | nindent 4 }}
   storage:
     size: {{ .Values.storage.size }}
   bootstrap:
@@ -72,4 +74,3 @@ spec:
     {{- toYaml . | nindent 6 }}
   {{- end }}
   {{- end }}
-
diff --git a/charts/cloudnative-pg-cluster/values.yaml b/charts/cloudnative-pg-cluster/values.yaml
@@ -145,6 +145,16 @@ storage:
   # -- how much storage to allocate to the postgresql cluster
   size: 1Gi
 
+resources:
+  # -- minimum resources guaranteed for pods
+  requests:
+    cpu: 500
+    memory: "512Mi"
+  # -- resource limit for pods
+  limits:
+    cpu: 1000
+    memory: "1024Mi"
+
 testApp:
   ## -- create a test deployment to verify db connectivity.
   # Populates user and DB from the Initdb owner and database values