Skip to content

Commit 87551e7

Browse files
Merge pull request #2 from yaswanthkumar1995/feat/1122-helm-mcp-servers-chart
fix(hpa): use deep merge for horizontal pod autoscaler config overrides
2 parents 4b5c52d + 64f379a commit 87551e7

2 files changed

Lines changed: 5 additions & 1 deletion

File tree

charts/mcp-servers/templates/hpa.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
{{- $root := . -}}
22
{{- range $i, $server := $root.Values.mcpServers }}
33
{{- if (default true $server.enabled) }}
4-
{{- $hpa := default $root.Values.global.hpa $server.hpa -}}
4+
{{- $hpa := merge (default (dict) $server.hpa) $root.Values.global.hpa -}}
55
{{- if $hpa.enabled }}
66
{{- $fullname := include "mcp-servers.serverFullname" (dict "root" $root "server" $server) -}}
77
apiVersion: autoscaling/v2

charts/mcp-servers/values.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -110,6 +110,10 @@ global:
110110
# image:
111111
# repository: mcp/git
112112
# tag: "latest"
113+
# hpa:
114+
# enabled: true
115+
# minReplicas: 2
116+
# maxReplicas: 5
113117
#
114118
# - name: memory
115119
# enabled: true

0 commit comments

Comments
 (0)