From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:36352) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bzPYp-0003YI-Lr for qemu-devel@nongnu.org; Wed, 26 Oct 2016 10:52:15 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bzPYl-0005zR-J9 for qemu-devel@nongnu.org; Wed, 26 Oct 2016 10:52:11 -0400 Received: from szxga03-in.huawei.com ([119.145.14.66]:15742) by eggs.gnu.org with esmtps (TLS1.0:RSA_ARCFOUR_SHA1:16) (Exim 4.71) (envelope-from ) id 1bzPYd-0005Zj-15 for qemu-devel@nongnu.org; Wed, 26 Oct 2016 10:52:07 -0400 References: <1476792613-11712-1-git-send-email-zhang.zhanghailiang@huawei.com> <1476792613-11712-13-git-send-email-zhang.zhanghailiang@huawei.com> <20161026055157.GL1679@amit-lp.rh> From: Hailiang Zhang Message-ID: <5810C2CE.1080005@huawei.com> Date: Wed, 26 Oct 2016 22:50:54 +0800 MIME-Version: 1.0 In-Reply-To: <20161026055157.GL1679@amit-lp.rh> Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH COLO-Frame (Base) v21 12/17] COLO: Add 'x-colo-lost-heartbeat' command to trigger failover List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Amit Shah Cc: quintela@redhat.com, qemu-devel@nongnu.org, dgilbert@redhat.com, wency@cn.fujitsu.com, lizhijian@cn.fujitsu.com, xiecl.fnst@cn.fujitsu.com, Luiz Capitulino , Eric Blake , Markus Armbruster On 2016/10/26 13:51, Amit Shah wrote: > On (Tue) 18 Oct 2016 [20:10:08], zhanghailiang wrote: >> We leave users to choose whatever heartbeat solution they want, >> if the heartbeat is lost, or other errors they detect, they can use >> experimental command 'x_colo_lost_heartbeat' to tell COLO to do failover, >> COLO will do operations accordingly. >> >> For example, if the command is sent to the PVM, the Primary side will >> exit COLO mode and take over operation. > > Primary should already be in control, so there's nothing special > needed to 'take over operation'? At max, it should not do periodic Maybe the description here is not accurate which makes you confuse, It should be 'does cleanup work and PVM will take over the service work', I will fix it in next version. > syncs anymore till it hears from a (new) secondary. > >> If sent to the Secondary, the >> secondary will run failover work, then take over server operation to >> become the new Primary. >> >> Cc: Luiz Capitulino >> Cc: Eric Blake >> Cc: Markus Armbruster >> Signed-off-by: zhanghailiang >> Signed-off-by: Li Zhijian >> Reviewed-by: Dr. David Alan Gilbert > > Reviewed-by: Amit Shah > > > Amit > > . >