Skip to content

Commit 400da5e

Browse files
lexfreilexfrei
committed
fix(seaweedfs): pass filer.replicas and s3.replicas to HelmRelease
The filer and s3 blocks in the seaweedfs HelmRelease values omitted the replicas: field, so user-supplied .Values.filer.replicas and .Values.s3.replicas were silently ignored and the upstream chart defaults took over. This also caused the WorkloadMonitor resources for filer and s3 (which read the same values) to diverge from the actual replica count and trigger spurious alerts when users set non-default values. Mirror the master/volume/db pattern and interpolate replicas: into both blocks. Assisted-By: Claude <[email protected]> Signed-off-by: Aleksei Sviridkin <[email protected]>
1 parent 6e045e6 commit 400da5e

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

‎packages/extra/seaweedfs/templates/seaweedfs.yaml‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -232,10 +232,12 @@ spec:
232232
{{- else if eq .Values.topology "MultiZone" }}
233233
defaultReplicaPlacement: "{{ sub .Values.replicationFactor 1 }}00"
234234
{{- end }}
235+
replicas: {{ .Values.filer.replicas }}
235236
s3:
236237
domainName: {{ .Values.host | default (printf "s3.%s" $host) }}
237238
resources: {{- include "cozy-lib.resources.defaultingSanitize" (list .Values.filer.resourcesPreset .Values.filer.resources $) | nindent 10 }}
238239
s3:
240+
replicas: {{ .Values.s3.replicas }}
239241
ingress:
240242
className: {{ $ingress }}
241243
host: {{ .Values.host | default (printf "s3.%s" $host) }}

0 commit comments

Comments
 (0)