Skip to content

Fix CD/RO OpenShift examples: missing boundAgent.volumePermissions + analytics sysctl override - #101

Open
Muxamedin wants to merge 4 commits into
cloudbees:masterfrom
Muxamedin:CDRO-17580-openshift-examples-fix
Open

Muxamedin wants to merge 4 commits into
cloudbees:masterfrom
Muxamedin:CDRO-17580-openshift-examples-fix

Conversation

@Muxamedin

Copy link
Copy Markdown
Collaborator

Summary

  • Both cloudbees-cd-{demo,prod}-openshift.yaml were missing boundAgent.volumePermissions.enabled: false — the chart's _helpers.tpl hard-fails Helm render on any OpenShift cluster without it ("Please make boundAgent.volumePermissions.enabled=false to install/upgrade on OpenShift platform").
  • The existing openshiftNodeTuning/sysctlInitContainer override in both files was set under dois:, a component that's disabled by default (dois.enabled: false) and never renders. analytics is the component that actually always deploys, and its sysctlInitContainer is a privileged init container — confirmed via helm template that it would be rejected by OpenShift's default restricted SCC without this override.

Found and verified while validating CD/RO on OpenShift 4.22 (CDRO-17580) — same two issues were also fixed in electric-cloud/flow-on-kubernetes/examples/.

Test plan

  • helm template with --api-versions route.openshift.io/v1: fails without boundAgent.volumePermissions.enabled: false, passes with it
  • helm template -s templates/analytics-stateful-set.yaml: privileged sysctl init container present with chart defaults, absent with analytics.sysctlInitContainer.enabled: false
  • YAML syntax validated for both files

@Muxamedin
Muxamedin requested a review from a team as a code owner October 6, 2026 11:04
…xamples

Mirrors the fix applied to flow-on-kubernetes/examples/ (CDRO-17580). Both
OpenShift reference values files were missing this required setting, which
causes a hard Helm render failure on any OpenShift cluster
(_helpers.tpl fails with "Please make boundAgent.volumePermissions.enabled=false
to install/upgrade on OpenShift platform").
…t that actually runs

The existing dois openshiftNodeTuning/sysctlInitContainer override in both OpenShift
example files was set on the wrong key: dois.enabled is false by default, so that
statefulset never renders. analytics is the component that actually always deploys,
and its sysctlInitContainer is a *privileged* init container (sets vm.max_map_count) --
confirmed via `helm template` that OpenShift's default restricted SCC would reject it
without this override. Left the existing (harmless but inert) dois block in place.
@Muxamedin
Muxamedin force-pushed the CDRO-17580-openshift-examples-fix branch from d99e056 to 46550f1 Compare October 6, 2026 15:55
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.

1 participant