ClientProfile reconcile

[nb-ohad]

Describe what this PR does

Initial implications for clientprofiles.csi.ceph.io controller

• Controller validation against a cephconnection.csi.ceph.io resource
• Shared owner logic for attached cephconnection.csi.ceph.io logic
• Shared owner logic for Ceph CSI Config, config map
• Reconcile a Ceph CSI Config record into the shared config map
• Handle clean-up upon resource deletion (removal of owners, removal of configmap record, etc.)

Is there anything that requires special attention

• PR does not include test code. Test code will be added in a new PR
• Controller name might need to change as of pending discussion on config.csi.ceph.io name change

[Madhu-1]

@nb-ohad can you please resolve merge conflicts?

[leelavg]

can below be included in this PR?

diff --git a/config/rbac/kustomization.yaml b/config/rbac/kustomization.yaml
index c26ae59..3d0a21b 100644
--- a/config/rbac/kustomization.yaml
+++ b/config/rbac/kustomization.yaml
@@ -28,3 +28,21 @@ resources:
 - operatorconfig_viewer_role.yaml
 - driver_editor_role.yaml
 - driver_viewer_role.yaml
+- csi_cephfs_ctrlplugin_clusterrole.yaml
+- csi_cephfs_ctrlplugin_clusterrole_binding.yaml
+- csi_cephfs_ctrlplugin_role.yaml
+- csi_cephfs_ctrlplugin_role_binding.yaml
+- csi_cephfs_ctrlplugin_service_account.yaml
+- csi_cephfs_nodeplugin_clusterrole.yaml
+- csi_cephfs_nodeplugin_clusterrole_binding.yaml
+- csi_cephfs_nodeplugin_service_account.yaml
+- csi_rbd_ctrlplugin_clusterrole.yaml
+- csi_rbd_ctrlplugin_clusterrole_binding.yaml
+- csi_rbd_ctrlplugin_role.yaml
+- csi_rbd_ctrlplugin_role_binding.yaml
+- csi_rbd_ctrlplugin_service_account.yaml
+- csi_rbd_nodeplugin_clusterrole.yaml
+- csi_rbd_nodeplugin_clusterrole_binding.yaml
+- csi_rbd_nodeplugin_role.yaml
+- csi_rbd_nodeplugin_role_binding.yaml
+- csi_rbd_nodeplugin_service_account.yaml

[nb-ohad]

@Madhu-1 I am adding the hold label on this one to make sure that #65 is merged first. That PR contains a small fix for map-to-string serialization needed in the code supporting Kernal and Fuse mount options.

I cannot just rebase this PR on top of the fix PR because this PR is already rebased on top of the API changes which is not yet merged

[nb-ohad]

can below be included in this PR?

diff --git a/config/rbac/kustomization.yaml b/config/rbac/kustomization.yaml
index c26ae59..3d0a21b 100644
--- a/config/rbac/kustomization.yaml
+++ b/config/rbac/kustomization.yaml
@@ -28,3 +28,21 @@ resources:
 - operatorconfig_viewer_role.yaml
 - driver_editor_role.yaml
 - driver_viewer_role.yaml
+- csi_cephfs_ctrlplugin_clusterrole.yaml
+- csi_cephfs_ctrlplugin_clusterrole_binding.yaml
+- csi_cephfs_ctrlplugin_role.yaml
+- csi_cephfs_ctrlplugin_role_binding.yaml
+- csi_cephfs_ctrlplugin_service_account.yaml
+- csi_cephfs_nodeplugin_clusterrole.yaml
+- csi_cephfs_nodeplugin_clusterrole_binding.yaml
+- csi_cephfs_nodeplugin_service_account.yaml
+- csi_rbd_ctrlplugin_clusterrole.yaml
+- csi_rbd_ctrlplugin_clusterrole_binding.yaml
+- csi_rbd_ctrlplugin_role.yaml
+- csi_rbd_ctrlplugin_role_binding.yaml
+- csi_rbd_ctrlplugin_service_account.yaml
+- csi_rbd_nodeplugin_clusterrole.yaml
+- csi_rbd_nodeplugin_clusterrole_binding.yaml
+- csi_rbd_nodeplugin_role.yaml
+- csi_rbd_nodeplugin_role_binding.yaml
+- csi_rbd_nodeplugin_service_account.yaml

@leelavg is this request still relevant? Aren't these changes already in main?

[leelavg]

 is this request still relevant? Aren't these changes already in main?

• Yep, should have been part of fixes PR that was merged. In main the manifests are defined but will not be included in the bundle unless referenced in Kustomization.yml

[nb-ohad]

is this request still relevant? Aren't these changes already in main?

• Yep, should have been part of fixes PR that was merged. In main the manifests are defined but will not be included in the bundle unless referenced in Kustomization.yml

This request is out of scope of this PR. If you have the solution please provide a PR. If not please open a gitissue to track

[nb-ohad]

@Madhu-1
PR refreshed, hold removed.