Merge pull request #1383 from porter-dev/ym/gpu_autoscaling_fix

fix: remove division by 100 for gpu scaling threshold in umbrella
porter-dev · Aug 19, 2024 · 1345be6 · 1345be6
2 parents 129fcca + 1a26875
commit 1345be6
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 5 deletions.
diff --git a/applications/web/templates/hpa-blue-green.yaml b/applications/web/templates/hpa-blue-green.yaml
@@ -50,7 +50,7 @@ spec:
           name: gpu_memory_utilization_ratio
         target:
           type: AverageValue
-          averageValue: {{ . | div 100.0 }}
+          averageValue: {{ . }}
     {{- end }}
 {{- end }}
 {{- end }}

diff --git a/applications/web/templates/hpa.yaml b/applications/web/templates/hpa.yaml
@@ -69,7 +69,7 @@ spec:
           name: gpu_memory_utilization_ratio
         target:
           type: AverageValue
-          averageValue: {{ . | div 100.0 }}
+          averageValue: {{ . }}
     {{- end }}
 {{- end }}
 {{- end }}

diff --git a/applications/web/values.yaml b/applications/web/values.yaml
@@ -129,8 +129,8 @@ autoscaling:
   maxReplicas: 10
   targetCPUUtilizationPercentage: 50
   targetMemoryUtilizationPercentage: 50
-  targetGPUUtilizationPercentage: 50
-  targetGPUMemoryUtilizationRatio: 50
+  targetGPUUtilizationPercentage: 100
+  targetGPUMemoryUtilizationRatio: 1
   behavior:
     enabled: false
     scaleDown:

diff --git a/applications/worker/templates/hpa.yaml b/applications/worker/templates/hpa.yaml
@@ -66,7 +66,7 @@ spec:
           name: gpu_memory_utilization_ratio
         target:
           type: AverageValue
-          averageValue: {{ . | div 100.0 }}
+          averageValue: {{ . }}
     {{- end }}
 {{- end }}
 {{- end }}
diff --git a/applications/worker/values.yaml b/applications/worker/values.yaml
@@ -57,6 +57,8 @@ autoscaling:
   maxReplicas: 10
   targetCPUUtilizationPercentage: 50
   targetMemoryUtilizationPercentage: 50
+  targetGPUUtilizationPercentage: 100
+  targetGPUMemoryUtilizationRatio: 1
   behavior:
     enabled: false
     scaleDown: