Comment 9 for bug 1403916

Revision history for this message
Mitsuhiro Tanino (mitsuhiro-tanino) wrote :

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