chore: switch HPA resource metric order to put CPU first (#349)

* chore: switch HPA resource metric order to put CPU first * oops
PrefectHQ · Jun 13, 2024 · 2b2d557 · 2b2d557
1 parent 18815e8
commit 2b2d557
Show file tree

Hide file tree

Showing 6 changed files with 20 additions and 12 deletions.
diff --git a/charts/prefect-agent/README.md b/charts/prefect-agent/README.md
@@ -80,3 +80,5 @@ Prefect Agent application bundle
 | serviceAccount.create | bool | `true` | specifies whether a ServiceAccount should be created |
 | serviceAccount.name | string | `""` | the name of the ServiceAccount to use. if not set and create is true, a name is generated using the common.names.fullname template |
 
+----------------------------------------------
+Autogenerated from chart metadata using [helm-docs v1.13.1](https://github.com/norwoodj/helm-docs/releases/v1.13.1)
diff --git a/charts/prefect-server/README.md b/charts/prefect-server/README.md
@@ -207,3 +207,5 @@ No secrets are created when providing an existing secret.
 | serviceAccount.create | bool | `true` | specifies whether a ServiceAccount should be created |
 | serviceAccount.name | string | `""` | the name of the ServiceAccount to use. if not set and create is true, a name is generated using the common.names.fullname template |
 
+----------------------------------------------
+Autogenerated from chart metadata using [helm-docs v1.13.1](https://github.com/norwoodj/helm-docs/releases/v1.13.1)
diff --git a/charts/prefect-server/templates/hpa.yaml b/charts/prefect-server/templates/hpa.yaml
@@ -20,20 +20,20 @@ spec:
   minReplicas: {{ .Values.server.autoscaling.minReplicas }}
   maxReplicas: {{ .Values.server.autoscaling.maxReplicas }}
   metrics:
-    {{- if .Values.server.autoscaling.targetMemory }}
+    {{- if .Values.server.autoscaling.targetCPU }}
     - type: Resource
       resource:
-        name: memory
+        name: cpu
         target:
           type: Utilization
-          averageUtilization: {{ .Values.server.autoscaling.targetMemory }}
+          averageUtilization: {{ .Values.server.autoscaling.targetCPU }}
     {{- end }}
-    {{- if .Values.server.autoscaling.targetCPU }}
+    {{- if .Values.server.autoscaling.targetMemory }}
     - type: Resource
       resource:
-        name: cpu
+        name: memory
         target:
           type: Utilization
-          averageUtilization: {{ .Values.server.autoscaling.targetCPU }}
+          averageUtilization: {{ .Values.server.autoscaling.targetMemory }}
     {{- end }}
 {{- end }}
diff --git a/charts/prefect-worker/README.md b/charts/prefect-worker/README.md
@@ -308,3 +308,5 @@ helm install prefect-worker prefect/prefect-worker -f values.yaml --set-file wor
 | worker.serverApiConfig.uiUrl | string | `"http://localhost:4200"` | prefect UI url |
 | worker.tolerations | list | `[]` | tolerations for worker pods assignment |
 
+----------------------------------------------
+Autogenerated from chart metadata using [helm-docs v1.13.1](https://github.com/norwoodj/helm-docs/releases/v1.13.1)
diff --git a/charts/prefect-worker/templates/hpa.yaml b/charts/prefect-worker/templates/hpa.yaml
@@ -21,20 +21,20 @@ spec:
   minReplicas: {{ .Values.worker.autoscaling.minReplicas }}
   maxReplicas: {{ .Values.worker.autoscaling.maxReplicas }}
   metrics:
-    {{- if .Values.worker.autoscaling.targetMemoryUtilizationPercentage }}
+    {{- if .Values.worker.autoscaling.targetCPUUtilizationPercentage }}
     - type: Resource
       resource:
-        name: memory
+        name: cpu
         target:
           type: Utilization
-          averageUtilization: {{ .Values.worker.autoscaling.targetMemoryUtilizationPercentage }}
+          averageUtilization: {{ .Values.worker.autoscaling.targetCPUUtilizationPercentage }}
     {{- end }}
-    {{- if .Values.worker.autoscaling.targetCPUUtilizationPercentage }}
+    {{- if .Values.worker.autoscaling.targetMemoryUtilizationPercentage }}
     - type: Resource
       resource:
-        name: cpu
+        name: memory
         target:
           type: Utilization
-          averageUtilization: {{ .Values.worker.autoscaling.targetCPUUtilizationPercentage }}
+          averageUtilization: {{ .Values.worker.autoscaling.targetMemoryUtilizationPercentage }}
     {{- end }}
 {{- end }}
diff --git a/charts/prometheus-prefect-exporter/README.md b/charts/prometheus-prefect-exporter/README.md
@@ -89,3 +89,5 @@ Shoutout to @ialejandro for the original work on this chart!
 | serviceMonitor.enabled | bool | `false` | Enable or disable |
 | tolerations | list | `[]` | Tolerations for pod assignment |
 
+----------------------------------------------
+Autogenerated from chart metadata using [helm-docs v1.13.1](https://github.com/norwoodj/helm-docs/releases/v1.13.1)