mirrors/ceph-csi
1
0
Fork 0
mirror of https://github.com/ceph/ceph-csi.git synced 2024-11-23 23:00:19 +00:00
Code Issues Packages Projects Releases Wiki Activity

rbd: return crypt error for the rpc return

Browse Source 
At present we return the volume connect error if the clone
from snapshot fails when rbdvolume is encrypted, which is incorrect.
This patch correctly return the failed copy encryption error to the
caller

Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
(cherry picked from commit 798437d0c4)
This commit is contained in:
Humble Chirammal 2021-04-21 18:01:17 +05:30 committed by Madhu Rajanna
parent 76fb7f6441
commit 1367cb445f
1 changed files with 3 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
internal/rbd/controllerserver.go
View File

@ -811,13 +811,9 @@ func cloneFromSnapshot(ctx context.Context, rbdVol *rbdVolume, rbdSnap *rbdSnaps
if rbdVol.isEncrypted() { if rbdVol.isEncrypted() {
// FIXME: vol.VolID should be different from rbdVol.VolID // FIXME: vol.VolID should be different from rbdVol.VolID
vol.VolID = rbdVol.VolID vol.VolID = rbdVol.VolID
cryptErr := rbdVol.copyEncryptionConfig(&vol.rbdImage) err = rbdVol.copyEncryptionConfig(&vol.rbdImage)
if cryptErr != nil { if err != nil {
util.WarningLog(ctx, "failed copy encryption "+ return nil, status.Error(codes.Internal, err.Error())
"config for %q: %v", vol.String(),
rbdSnap.RequestName, cryptErr)
return nil, status.Errorf(codes.Internal,
err.Error())
} }
} }