Skip to content

fix(helm): omit unset rbd mirror daemon count - #573

Open
tomatotomata wants to merge 1 commit into
ceph:mainfrom
tomatotomata:codex/cephconnection-optional-rbd-count
Open

fix(helm): omit unset rbd mirror daemon count#573
tomatotomata wants to merge 1 commit into
ceph:mainfrom
tomatotomata:codex/cephconnection-optional-rbd-count

Conversation

@tomatotomata

Copy link
Copy Markdown

Summary

When a cephConnections entry omits rbdMirrorDaemonCount, the chart currently renders rbdMirrorDaemonCount: null. Kubernetes rejects that value because the CRD field is an optional integer with a minimum of 1.

Render the field only when a value is supplied. Existing positive values are unchanged, while omitted values now let the CRD/operator default apply.

Testing

  • Rendered the chart with an unset value and confirmed the field is omitted.
  • Rendered the chart with rbdMirrorDaemonCount=2 and confirmed the field remains 2.
  • git diff --check

Fixes #551

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

helm chart CephConnection not defaulting rbdMirrorDaemonCount

1 participant