On Wed, Aug 26, 2020 at 11:24:09 +0530, Prathamesh Chavan wrote: > `qemuMigrationJobPhase` was transformed into `virMigrationJobPhase` > and a common util file `virmigration` was created to store its > defination. > > This is one of the initial steps we are taking towards making > more and more code hypervisor agnostic. And this shall > be followed by transformation of similar structs in the future. I'm not really persuaded that this enum makes sense outside of the qemu hypervisor driver. If any new hypervisor implements migration it's not very likely to share the steps required in the qemu driver anyways. Aditionally making this too universal might lead to a false assumption that all the phases make sense for a new hypervisor driver implementing migration. If you have any more justification for this change please put it into the commit message.
