mirror of
https://github.com/ceph/ceph-csi.git
synced 2024-11-09 16:00:22 +00:00
rbd: flatten image in CreateVolumeGroup
This commit adds support for flattenMode option for volumegroup. If the flattenMode is set to "force" in volumegroupreplicationclass parameters, cephcsi will add a task to flatten the image if it has parent before adding it to the group. This enable cephcsi to then mirror such images after flattening them. Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
This commit is contained in:
parent
b222b773aa
commit
c773c98408
@ -18,6 +18,7 @@ package rbd
|
|||||||
|
|
||||||
import (
|
import (
|
||||||
"context"
|
"context"
|
||||||
|
"fmt"
|
||||||
"slices"
|
"slices"
|
||||||
|
|
||||||
"github.com/ceph/ceph-csi/internal/rbd"
|
"github.com/ceph/ceph-csi/internal/rbd"
|
||||||
@ -87,6 +88,11 @@ func (vs *VolumeGroupServer) CreateVolumeGroup(
|
|||||||
|
|
||||||
// resolve all volumes
|
// resolve all volumes
|
||||||
volumes := make([]types.Volume, len(req.GetVolumeIds()))
|
volumes := make([]types.Volume, len(req.GetVolumeIds()))
|
||||||
|
defer func() {
|
||||||
|
for _, vol := range volumes {
|
||||||
|
vol.Destroy(ctx)
|
||||||
|
}
|
||||||
|
}()
|
||||||
for i, id := range req.GetVolumeIds() {
|
for i, id := range req.GetVolumeIds() {
|
||||||
vol, err := mgr.GetVolumeByID(ctx, id)
|
vol, err := mgr.GetVolumeByID(ctx, id)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
@ -97,9 +103,6 @@ func (vs *VolumeGroupServer) CreateVolumeGroup(
|
|||||||
req.GetName(),
|
req.GetName(),
|
||||||
err.Error())
|
err.Error())
|
||||||
}
|
}
|
||||||
|
|
||||||
//nolint:gocritic // need to call .Destroy() for all volumes
|
|
||||||
defer vol.Destroy(ctx)
|
|
||||||
volumes[i] = vol
|
volumes[i] = vol
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -117,6 +120,21 @@ func (vs *VolumeGroupServer) CreateVolumeGroup(
|
|||||||
|
|
||||||
log.DebugLog(ctx, "VolumeGroup %q has been created: %+v", req.GetName(), vg)
|
log.DebugLog(ctx, "VolumeGroup %q has been created: %+v", req.GetName(), vg)
|
||||||
|
|
||||||
|
// extract the flatten mode
|
||||||
|
flattenMode, err := getFlattenMode(ctx, req.GetParameters())
|
||||||
|
if err != nil {
|
||||||
|
return nil, err
|
||||||
|
}
|
||||||
|
// Flatten the image if the flatten mode is set to FlattenModeForce
|
||||||
|
// before adding it to the volume group.
|
||||||
|
for _, v := range volumes {
|
||||||
|
err = v.HandleParentImageExistence(ctx, flattenMode)
|
||||||
|
if err != nil {
|
||||||
|
err = fmt.Errorf("failed to handle parent image for volume group %q: %w", vg, err)
|
||||||
|
|
||||||
|
return nil, getGRPCError(err)
|
||||||
|
}
|
||||||
|
}
|
||||||
// add each rbd-image to the RBDVolumeGroup
|
// add each rbd-image to the RBDVolumeGroup
|
||||||
for _, vol := range volumes {
|
for _, vol := range volumes {
|
||||||
err = vg.AddVolume(ctx, vol)
|
err = vg.AddVolume(ctx, vol)
|
||||||
|
Loading…
Reference in New Issue
Block a user