Uninstall will wait untill all PVs using OCS storageClasses are removed #152
Conversation
| PersistentVolumeSource: corev1.PersistentVolumeSource{ | ||
| RBD: &corev1.RBDPersistentVolumeSource{ | ||
| CephMonitors: []string{ | ||
| "0.0.0.0:6789", | ||
| }, | ||
| RBDImage: "test", |
There was a problem hiding this comment.
the source should be CSI, not RBD nor Cephfs.
There was a problem hiding this comment.
But this is just for our unit tests, we just need to make sure that the PVs are using OCS storage classes
There was a problem hiding this comment.
I agree with Dhruv on this one, our unit tests are mocking the stoagecluster and as such we have no notion of CSI.
There was a problem hiding this comment.
But this is just for our unit tests, we just need to make sure that the PVs are using OCS storage classes
there is no relationship between the OCS storageclasses and the PersistentVolumeSource which is RBD and Cephfs mentioned here.
There was a problem hiding this comment.
Yes, I agree but we just want PVs using OCS storageClasses
It doesn't matter what PersistentVolumeSource we use
Signed-off-by: bindrad <dbindra@redhat.com>
Signed-off-by: bindrad dbindra@redhat.com