ceph / ceph-csi

CSI driver for Ceph
Apache License 2.0
1.29k stars 548 forks source link

Reaserch on VolumeGroup snapshotting and taking advantages of it #2226

Closed humblec closed 1 year ago

humblec commented 3 years ago

Describe the feature you'd like to have

Kubernetes KEP and implementation of Volume Group snapshot is in progress and we have to do some research on how this can be made use of in Ceph CSI project for both RBD and CephFS volumes we provision and control.

Ceph implementation is in progress and we have to list down the consumption of this feature by correlating between upstream API and core Ceph side.

This issue tracks the research and discussion points on the same

[1] https://github.com/ceph/ceph/pull/41572 [2] https://github.com/kubernetes/enhancements/blob/9ab31c34cded378a8ad722c481162ed8999d0cbb/keps/sig-storage/20200212-volume-group.md

humblec commented 3 years ago

Here is the summary:

Type VolumeGroupSnapshotStatus struct {
...
        // List of volume snapshots
    // +optional
        SnapshotList []VolumeSnapshot
}

cc @kotreshhr

Additional thoughts ( mainly for ceph-csi team):

Ceph CSI also has to take some decisions on how can we support volumegroup and subvolume layout to support this feature. against various models like [1] considering below

-- Maintance overhead of many groups ( good/bad) -- Limits of subvolumegroups we can have -- Limit of subvolumes in a group -- snap creation and restore operation time in different models...etc

[1]

humblec commented 3 years ago

[update] more discussions are happening on the KEP about individual snapshot list ..etc Ref:

https://github.com/kubernetes/enhancements/pull/1551#discussion_r687683605

https://github.com/kubernetes/enhancements/pull/1551#issuecomment-878805651

github-actions[bot] commented 3 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in a week if no further activity occurs. Thank you for your contributions.

Madhu-1 commented 1 year ago

Closing this due to inactivity