Rakshith R

> +1 As discussed, I think we can close this issue and let admin/rook at higher level handle pinning

Closing this epic as per above comments. subvolumegroup pinning support will be added Rook in upcoming releases. In standalone ceph clusters, storage admins needs to do it.

hey @riya-singhal31 , This is part of 3.10 milestone. Can you prioritize this issue to get it done within this week or early next week ? Thanks,

> https://github.com/ceph/ceph-csi/blob/cd2e25c290a642154c25c4bf42e739f39c1d51bd/e2e/rbd.go#L88 > >

Release notes regarding read affinity and removal of option to run cephcsi being run as both controller and node servers when either of the option is provided. https://github.com/ceph/ceph-csi/pull/3639#discussion_r1095685719

> @Rakshith-R I guess all the task here is done already. Can you confirm and close the issue, if done? No, 3 out of 4 TODOs are still not done....

> Closes with - #3608 you mean updates, Make sure to address this [comment](https://github.com/ceph/ceph-csi/pull/3608/files#r1092953730) too. three more tasks are remaining including the above one.

> > add minimal unit tests like internal/csi-addons/rbd/reclaimspace_test.go > There is already a test file for replication.go, moved it to csi-addons. Do we need to add some more tests to...

> > > remove replication service from controller server > > @Rakshith-R can you please elaborate on it. My understanding is that we want to remove the replication import from...

> > You are referring to metrics port 9080? @Rakshith-R @Madhu-1 thoughts? > > yes,9081 and 9080 xxxx:9081/debug/pprof That should be disabled by default https://github.com/ceph/ceph-csi/blob/6cab5bfd42906615143d432dcc5d1b3c91161974/cmd/cephcsi.go#L143