Skip to content

Explicitly set namespace for Ceph CSI operator/driver resources#391

Closed
degorenko wants to merge 1 commit intoceph:mainfrom
degorenko:set-namespace
Closed

Explicitly set namespace for Ceph CSI operator/driver resources#391
degorenko wants to merge 1 commit intoceph:mainfrom
degorenko:set-namespace

Conversation

@degorenko
Copy link

Add explicitly namespace name for CephCSI operator resources to have an ability specify it during helm install.

Fixes: #384

Resolves: ceph#384

Signed-Off-By: Denis Egorenko <degorenko@mirantis.com>
@Madhu-1
Copy link
Collaborator

Madhu-1 commented Jan 28, 2026

@degorenko these files are auto generated, we cannot manual change these files. we need something like below in makefile

$(KUSTOMIZE) build build | $(HELMIFY) -preserve-ns -image-pull-secrets deploy/charts/ceph-csi-operator

am yet to test the changes, feel free to test it out and see if it solves the problem.

@degorenko
Copy link
Author

@Madhu-1 this will add namespace like: namespace: ceph-csi-operator-system . How we can use release namespace?

@Madhu-1
Copy link
Collaborator

Madhu-1 commented Jan 28, 2026

@Madhu-1 this will add namespace like: namespace: ceph-csi-operator-system . How we can use release namespace?

we need to open a tracker with https://github.com/arttor/helmify if that field is not helpful

@degorenko
Copy link
Author

I guess issue that kustomize build with ceph-csi-operator-system namespace, which is preserved by helmify itself (so -preserve-ns flag is actually working fine)

@degorenko
Copy link
Author

There is related closed issue/pr - arttor/helmify#78 - so looks like it is by design to avoid setting {{ .Release.Namespace }} with helmify.

@Madhu-1
Copy link
Collaborator

Madhu-1 commented Jan 29, 2026

@degorenko i have sent PR #392 to fix it for the operator for now, will take care of the helm charts later when Rook adopts to it.

@Madhu-1
Copy link
Collaborator

Madhu-1 commented Feb 5, 2026

fixed in #392

@Madhu-1 Madhu-1 closed this Feb 5, 2026
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.

csi-controller-manager helm chart does not manage namespace

2 participants