Thank you for letting me know your investigation.
I agree the delete_volume have to handle both source and destination volume properly.
I think using "if volume_ref['migration_status'] != 'migrating'" has a little bit problem
since source volume deletion calls delete_volume with 'completing' status.
In this case, I think we also want to skip quota and other database updates in delete_volume.
I posted new patch. Please continue to discuss based on the new patch.
Hi Avishay,
Thank you for letting me know your investigation.
I agree the delete_volume have to handle both source and destination volume properly.
I think using "if volume_ ref['migration_ status' ] != 'migrating'" has a little bit problem
since source volume deletion calls delete_volume with 'completing' status.
In this case, I think we also want to skip quota and other database updates in delete_volume.
I posted new patch. Please continue to discuss based on the new patch.
Regards,
Mitsuhiro Tanino