✨ feat: worker and webhook autosacling

closes 8gears#80, closes 8gears#87 Signed-off-by: Muhammed Hussein Karimi <[email protected]>
mhkarimi1383 · Aug 5, 2024 · a5ca708 · a5ca708
1 parent d10683b
commit a5ca708
Show file tree

Hide file tree

Showing 5 changed files with 119 additions and 2 deletions.
diff --git a/charts/n8n/templates/deployment.webhooks.yaml b/charts/n8n/templates/deployment.webhooks.yaml
@@ -6,7 +6,7 @@ metadata:
   labels:
     {{- include "n8n.labels" . | nindent 4 }}
 spec:
-  {{- if not .Values.autoscaling.enabled }}
+  {{- if not .Values.webhookAutoscaling.enabled }}
   replicas: {{ .Values.scaling.webhook.count }}
   {{- end }}
   strategy:

diff --git a/charts/n8n/templates/deployment.worker.yaml b/charts/n8n/templates/deployment.worker.yaml
@@ -6,7 +6,7 @@ metadata:
   labels:
     {{- include "n8n.labels" . | nindent 4 }}
 spec:
-  {{- if not .Values.autoscaling.enabled }}
+  {{- if not .Values.workerAutoscaling.enabled }}
   replicas: {{ .Values.scaling.worker.count }}
   {{- end }}
   strategy:

diff --git a/charts/n8n/templates/hpa.webhook.yaml b/charts/n8n/templates/hpa.webhook.yaml
@@ -0,0 +1,51 @@
+{{- if .Values.autoscaling.enabled }}
+{{- if semverCompare ">=1.25-0" .Capabilities.KubeVersion.GitVersion -}}
+apiVersion: autoscaling/v2
+{{- else -}}
+apiVersion: autoscaling/v2beta1
+{{- end }}
+kind: HorizontalPodAutoscaler
+metadata:
+  name: {{ include "n8n.fullname" . }}-webhook
+  labels:
+    {{- include "n8n.labels" . | nindent 4 }}
+spec:
+  scaleTargetRef:
+    apiVersion: apps/v1
+    kind: Deployment
+    name: {{ include "n8n.fullname" . }}-webhook
+  minReplicas: {{ .Values.webhookAutoscaling.minReplicas }}
+  maxReplicas: {{ .Values.webhookAutoscaling.maxReplicas }}
+  metrics:
+{{- if semverCompare ">=1.25-0" .Capabilities.KubeVersion.GitVersion -}}
+    {{- if .Values.webhookAutoscaling.targetCPUUtilizationPercentage }}
+    - type: Resource
+      resource:
+        name: cpu
+        target:
+          type: Utilization
+          averageUtilization: {{ .Values.webhookAutoscaling.targetCPUUtilizationPercentage }}
+    {{- end }}
+    {{- if .Values.webhookAutoscaling.targetMemoryUtilizationPercentage }}
+    - type: Resource
+      resource:
+        name: memory
+        target:
+          type: Utilization
+          averageUtilization: {{ .Values.webhookAutoscaling.targetMemoryUtilizationPercentage }}
+    {{- end }}
+{{- else -}}
+    {{- if .Values.webhookAutoscaling.targetCPUUtilizationPercentage }}
+    - type: Resource
+      resource:
+        name: cpu
+        targetAverageUtilization: {{ .Values.webhookAutoscaling.targetCPUUtilizationPercentage }}
+    {{- end }}
+    {{- if .Values.webhookAutoscaling.targetMemoryUtilizationPercentage }}
+    - type: Resource
+      resource:
+        name: memory
+        targetAverageUtilization: {{ .Values.webhookAutoscaling.targetMemoryUtilizationPercentage }}
+    {{- end }}
+{{- end }}
+{{- end }}
diff --git a/charts/n8n/templates/hpa.worker.yaml b/charts/n8n/templates/hpa.worker.yaml
@@ -0,0 +1,51 @@
+{{- if .Values.autoscaling.enabled }}
+{{- if semverCompare ">=1.25-0" .Capabilities.KubeVersion.GitVersion -}}
+apiVersion: autoscaling/v2
+{{- else -}}
+apiVersion: autoscaling/v2beta1
+{{- end }}
+kind: HorizontalPodAutoscaler
+metadata:
+  name: {{ include "n8n.fullname" . }}-worker
+  labels:
+    {{- include "n8n.labels" . | nindent 4 }}
+spec:
+  scaleTargetRef:
+    apiVersion: apps/v1
+    kind: Deployment
+    name: {{ include "n8n.fullname" . }}-worker
+  minReplicas: {{ .Values.workerAutoscaling.minReplicas }}
+  maxReplicas: {{ .Values.workerAutoscaling.maxReplicas }}
+  metrics:
+{{- if semverCompare ">=1.25-0" .Capabilities.KubeVersion.GitVersion -}}
+    {{- if .Values.workerAutoscaling.targetCPUUtilizationPercentage }}
+    - type: Resource
+      resource:
+        name: cpu
+        target:
+          type: Utilization
+          averageUtilization: {{ .Values.workerAutoscaling.targetCPUUtilizationPercentage }}
+    {{- end }}
+    {{- if .Values.workerAutoscaling.targetMemoryUtilizationPercentage }}
+    - type: Resource
+      resource:
+        name: memory
+        target:
+          type: Utilization
+          averageUtilization: {{ .Values.workerAutoscaling.targetMemoryUtilizationPercentage }}
+    {{- end }}
+{{- else -}}
+    {{- if .Values.workerAutoscaling.targetCPUUtilizationPercentage }}
+    - type: Resource
+      resource:
+        name: cpu
+        targetAverageUtilization: {{ .Values.workerAutoscaling.targetCPUUtilizationPercentage }}
+    {{- end }}
+    {{- if .Values.workerAutoscaling.targetMemoryUtilizationPercentage }}
+    - type: Resource
+      resource:
+        name: memory
+        targetAverageUtilization: {{ .Values.workerAutoscaling.targetMemoryUtilizationPercentage }}
+    {{- end }}
+{{- end }}
+{{- end }}
diff --git a/charts/n8n/values.yaml b/charts/n8n/values.yaml
@@ -277,9 +277,24 @@ ingress:
 workerResources:
   {}
 
+workerAutoscaling:
+  enabled: false
+  minReplicas: 1
+  maxReplicas: 100
+  targetCPUUtilizationPercentage: 80
+  # targetMemoryUtilizationPercentage: 80
+
 webhookResources:
   {}
 
+webhookAutoscaling:
+  enabled: false
+  minReplicas: 1
+  maxReplicas: 100
+  targetCPUUtilizationPercentage: 80
+  # targetMemoryUtilizationPercentage: 80
+
+
 resources:
   {}
   # We usually recommend not to specify default resources and to leave this as a conscious