Skip to content

[Backport release-1.3] fix(seaweedfs): pass filer.replicas and s3.replicas to HelmRelease - #2551

Merged
myasnikovdaniil merged 1 commit into
release-1.3from
backport-2533-to-release-1.3
May 6, 2026
Merged

myasnikovdaniil merged 1 commit into
release-1.3from
backport-2533-to-release-1.3

Conversation

@github-actions

@github-actions github-actions Bot commented May 1, 2026

Copy link
Copy Markdown

Description

Backport of #2533 to release-1.3.

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]>
(cherry picked from commit 400da5e)
@dosubot dosubot Bot added size/XS This PR changes 0-9 lines, ignoring generated files area/storage Issues or PRs related to storage (linstor, seaweedfs, bucket, velero, harbor) kind/backport-previous Categorizes issue or PR as requiring a backport to the previous release line labels May 1, 2026
@myasnikovdaniil
myasnikovdaniil merged commit 313a1ac into release-1.3 May 6, 2026
6 of 7 checks passed
@myasnikovdaniil
myasnikovdaniil deleted the backport-2533-to-release-1.3 branch May 6, 2026 05:31
@github-actions

github-actions Bot commented May 6, 2026

Copy link
Copy Markdown
Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/storage Issues or PRs related to storage (linstor, seaweedfs, bucket, velero, harbor) kind/backport-previous Categorizes issue or PR as requiring a backport to the previous release line size/XS This PR changes 0-9 lines, ignoring generated files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants