Remove all CpuLimit per https://home.robusta.dev/blog/stop-using-cpu-…

…limits
NREL · Apr 1, 2023 · dd718b6 · dd718b6
1 parent e369470
commit dd718b6
Show file tree

Hide file tree

Showing 4 changed files with 1 addition and 7 deletions.
diff --git a/.helm/templates/julia-deployment.yaml b/.helm/templates/julia-deployment.yaml
@@ -52,5 +52,4 @@ spec:
               cpu: {{ .Values.juliaCpuRequest | quote }}
               memory: {{ .Values.juliaMemoryRequest | quote }}
             limits:
-              cpu: {{ .Values.juliaCpuLimit | quote }}
               memory: {{ .Values.juliaMemoryLimit | quote }}
diff --git a/.helm/values.production.yaml b/.helm/values.production.yaml
@@ -8,6 +8,5 @@ celeryMemoryRequest: "900Mi"
 celeryMemoryLimit: "900Mi"
 juliaReplicas: 20
 juliaCpuRequest: "300m"
-juliaCpuLimit: "4000m"
 juliaMemoryRequest: "8000Mi"
 juliaMemoryLimit: "8000Mi"
diff --git a/.helm/values.staging.yaml b/.helm/values.staging.yaml
@@ -2,6 +2,5 @@ appEnv: staging
 djangoSettingsModule: reopt_api.staging_settings
 juliaReplicas: 2
 juliaCpuRequest: "2000m"
-juliaCpuLimit: "3000m"
-juliaMemoryRequest: "8000Mi"
+juliaMemoryRequest: "12000Mi"
 juliaMemoryLimit: "12000Mi"
diff --git a/.helm/values.yaml b/.helm/values.yaml
@@ -10,16 +10,13 @@ redisHost: "{{ .Chart.Name }}-redis-service"
 redisPort: 6379
 djangoReplicas: 2
 djangoCpuRequest: "100m"
-djangoCpuLimit: "4000m"
 djangoMemoryRequest: "1600Mi"
 djangoMemoryLimit: "1600Mi"
 celeryReplicas: 2
 celeryCpuRequest: "100m"
-celeryCpuLimit: "800m"
 celeryMemoryRequest: "700Mi"
 celeryMemoryLimit: "700Mi"
 juliaReplicas: 2
 juliaCpuRequest: "300m"
-juliaCpuLimit: "4000m"
 juliaMemoryRequest: "3000Mi"
 juliaMemoryLimit: "3000Mi"