On 2016/10/26 13:58, Amit Shah wrote:
On (Tue) 18 Oct 2016 [20:10:10], zhanghailiang wrote:
For primary side, if COLO gets failover request from users.
To be exact, gets 'x_colo_lost_heartbeat' command.
COLO thread will exit the loop while the failover BH does the
cleanup work and resumes VM.
Signed-off-by: zhanghailiang <zhang.zhanghaili...@huawei.com>
Signed-off-by: Li Zhijian <lizhij...@cn.fujitsu.com>
Reviewed-by: Dr. David Alan Gilbert <dgilb...@redhat.com>
@@ -162,9 +196,20 @@ static int colo_do_checkpoint_transaction(MigrationState
*s,
@@ -280,8 +330,6 @@ out:
if (local_err) {
error_report_err(local_err);
}
- migrate_set_state(&s->state, MIGRATION_STATUS_COLO,
- MIGRATION_STATUS_COMPLETED);
Yea, I guess this should not have been done in the previous patch to
begin with?
Yes, I should not add this in patch 3, i will fix this in next version.
Thanks,
Hailiang
Amit
.