From fe677bdf664600beacee3de57223a20a76f6570b Mon Sep 17 00:00:00 2001 From: Om Mishra <32200996+mishraomp@users.noreply.github.com> Date: Tue, 24 Dec 2024 12:28:21 -0800 Subject: [PATCH] chore: remove limits (#2218) --- .../backend/templates/deployment.yaml | 6 ----- .../frontend/templates/deployment.yaml | 3 --- charts/app/values.yaml | 6 ----- charts/crunchy/templates/PostgresCluster.yaml | 24 ------------------- charts/crunchy/values.yaml | 24 ------------------- 5 files changed, 63 deletions(-) diff --git a/charts/app/templates/backend/templates/deployment.yaml b/charts/app/templates/backend/templates/deployment.yaml index 76ca5d11f..8331dde1b 100644 --- a/charts/app/templates/backend/templates/deployment.yaml +++ b/charts/app/templates/backend/templates/deployment.yaml @@ -45,9 +45,6 @@ spec: - name: FLYWAY_GROUP value: "true" resources: - limits: - cpu: 200m - memory: 200Mi requests: cpu: 50m memory: 75Mi @@ -90,9 +87,6 @@ spec: periodSeconds: 30 timeoutSeconds: 5 resources: # this is optional - limits: - cpu: 150m - memory: 150Mi requests: cpu: 50m memory: 75Mi diff --git a/charts/app/templates/frontend/templates/deployment.yaml b/charts/app/templates/frontend/templates/deployment.yaml index d50ba540b..154fd07e4 100644 --- a/charts/app/templates/frontend/templates/deployment.yaml +++ b/charts/app/templates/frontend/templates/deployment.yaml @@ -66,9 +66,6 @@ spec: periodSeconds: 30 timeoutSeconds: 5 resources: - limits: - cpu: 100m - memory: 150Mi requests: cpu: 30m memory: 50Mi diff --git a/charts/app/values.yaml b/charts/app/values.yaml index 5802db28f..180151cd5 100644 --- a/charts/app/values.yaml +++ b/charts/app/values.yaml @@ -56,9 +56,6 @@ backend: - prod/api-2 #-- resources specific to vault initContainer. it is optional and is an object. resources: - limits: - cpu: 50m - memory: 50Mi requests: cpu: 50m memory: 25Mi @@ -157,7 +154,4 @@ bitnamipg: requests: cpu: 50m memory: 150Mi - limits: - cpu: 150m - memory: 250Mi diff --git a/charts/crunchy/templates/PostgresCluster.yaml b/charts/crunchy/templates/PostgresCluster.yaml index 2fc074f27..70f643a74 100644 --- a/charts/crunchy/templates/PostgresCluster.yaml +++ b/charts/crunchy/templates/PostgresCluster.yaml @@ -52,9 +52,6 @@ spec: requests: cpu: {{ .Values.crunchy.pgmonitor.exporter.requests.cpu }} memory: {{ .Values.crunchy.pgmonitor.exporter.requests.memory }} - limits: - cpu: {{ .Values.crunchy.pgmonitor.exporter.limits.cpu }} - memory: {{ .Values.crunchy.pgmonitor.exporter.limits.memory }} {{ end }} @@ -69,9 +66,6 @@ spec: requests: cpu: {{ .Values.crunchy.instances.requests.cpu }} memory: {{ .Values.crunchy.instances.requests.memory }} - limits: - cpu: {{ .Values.crunchy.instances.limits.cpu }} - memory: {{ .Values.crunchy.instances.limits.memory }} sidecars: replicaCertCopy: @@ -79,9 +73,6 @@ spec: requests: cpu: {{ .Values.crunchy.instances.replicaCertCopy.requests.cpu }} memory: {{ .Values.crunchy.instances.replicaCertCopy.requests.memory }} - limits: - cpu: {{ .Values.crunchy.instances.replicaCertCopy.limits.cpu }} - memory: {{ .Values.crunchy.instances.replicaCertCopy.limits.memory }} dataVolumeClaimSpec: accessModes: - "ReadWriteOnce" @@ -179,9 +170,6 @@ spec: requests: cpu: {{ .Values.crunchy.pgBackRest.repoHost.requests.cpu }} memory: {{ .Values.crunchy.pgBackRest.repoHost.requests.memory }} - limits: - cpu: {{ .Values.crunchy.pgBackRest.repoHost.limits.cpu }} - memory: {{ .Values.crunchy.pgBackRest.repoHost.limits.memory }} sidecars: # this stuff is for the "pgbackrest" container in the "postgres-crunchy-ha" set of pods pgbackrest: @@ -189,25 +177,16 @@ spec: requests: cpu: {{ .Values.crunchy.pgBackRest.sidecars.requests.cpu }} memory: {{ .Values.crunchy.pgBackRest.sidecars.requests.memory }} - limits: - cpu: {{ .Values.crunchy.pgBackRest.sidecars.limits.cpu }} - memory: {{ .Values.crunchy.pgBackRest.sidecars.limits.memory }} pgbackrestConfig: resources: requests: cpu: {{ .Values.crunchy.pgBackRest.sidecars.requests.cpu }} memory: {{ .Values.crunchy.pgBackRest.sidecars.requests.memory }} - limits: - cpu: {{ .Values.crunchy.pgBackRest.sidecars.limits.cpu }} - memory: {{ .Values.crunchy.pgBackRest.sidecars.limits.memory }} jobs: resources: requests: cpu: {{ .Values.crunchy.pgBackRest.jobs.requests.cpu }} memory: {{ .Values.crunchy.pgBackRest.jobs.requests.memory }} - limits: - cpu: {{ .Values.crunchy.pgBackRest.jobs.limits.cpu }} - memory: {{ .Values.crunchy.pgBackRest.jobs.limits.memory }} {{- end }} patroni: dynamicConfiguration: @@ -240,9 +219,6 @@ spec: requests: cpu: {{ .Values.crunchy.proxy.pgBouncer.requests.cpu }} memory: {{ .Values.crunchy.proxy.pgBouncer.requests.memory }} - limits: - cpu: {{ .Values.crunchy.proxy.pgBouncer.limits.cpu }} - memory: {{ .Values.crunchy.proxy.pgBouncer.limits.memory }} affinity: podAntiAffinity: preferredDuringSchedulingIgnoredDuringExecution: diff --git a/charts/crunchy/values.yaml b/charts/crunchy/values.yaml index d833f42d4..0a0980369 100644 --- a/charts/crunchy/values.yaml +++ b/charts/crunchy/values.yaml @@ -39,16 +39,10 @@ crunchy: # enable it for TEST and PROD, for PR based pipelines simply use single requests: cpu: 50m memory: 128Mi - limits: - cpu: 150m - memory: 256Mi replicaCertCopy: requests: cpu: 1m memory: 32Mi - limits: - cpu: 50m - memory: 64Mi pgBackRest: enabled: true @@ -81,30 +75,18 @@ crunchy: # enable it for TEST and PROD, for PR based pipelines simply use single requests: cpu: 5m memory: 32Mi - limits: - cpu: 20m - memory: 64Mi repoHost: requests: cpu: 20m memory: 128Mi - limits: - cpu: 50m - memory: 256Mi sidecars: requests: cpu: 5m memory: 16Mi - limits: - cpu: 20m - memory: 64Mi jobs: requests: cpu: 20m memory: 128Mi - limits: - cpu: 100m - memory: 256Mi patroni: postgresql: @@ -129,9 +111,6 @@ crunchy: # enable it for TEST and PROD, for PR based pipelines simply use single requests: cpu: 5m memory: 32Mi - limits: - cpu: 20m - memory: 64Mi maxConnections: 10 # make sure less than postgres max connections # Postgres Cluster resource values: @@ -142,6 +121,3 @@ crunchy: # enable it for TEST and PROD, for PR based pipelines simply use single requests: cpu: 1m memory: 16Mi - limits: - cpu: 35m - memory: 32Mi