[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
RE: [PATCH v7 0/4] colo: Add support for continuous replication
From: |
Zhang, Chen |
Subject: |
RE: [PATCH v7 0/4] colo: Add support for continuous replication |
Date: |
Tue, 19 Nov 2019 12:28:32 +0000 |
> -----Original Message-----
> From: Lukas Straub <address@hidden>
> Sent: Thursday, November 14, 2019 12:36 AM
> To: qemu-devel <address@hidden>
> Cc: Zhang, Chen <address@hidden>; Jason Wang
> <address@hidden>; Wen Congyang <address@hidden>;
> Xie Changlong <address@hidden>; Kevin Wolf
> <address@hidden>; Max Reitz <address@hidden>; qemu-block
> <address@hidden>
> Subject: Re: [PATCH v7 0/4] colo: Add support for continuous replication
>
> On Fri, 25 Oct 2019 19:06:31 +0200
> Lukas Straub <address@hidden> wrote:
>
> > Hello Everyone,
> > These Patches add support for continuous replication to colo. This
> > means that after the Primary fails and the Secondary did a failover,
> > the Secondary can then become Primary and resume replication to a new
> Secondary.
> >
> > Regards,
> > Lukas Straub
> >
> > v7:
> > - clarify meaning of ip's in documentation and note that active and hidden
> > images just need to be created once
> > - reverted removal of bdrv_is_root_node(top_bs) in replication and
> adjusted
> > the top-id= parameter in documentation acordingly
> >
> > v6:
> > - documented the position= and insert= options
> > - renamed replication test
> > - clarified documentation by using different ip's for primary and
> > secondary
> > - added Reviewed-by tags
> >
> > v5:
> > - change syntax for the position= parameter
> > - fix spelling mistake
> >
> > v4:
> > - fix checkpatch.pl warnings
> >
> > v3:
> > - add test for replication changes
> > - check if the filter to be inserted before/behind belongs to the
> > same interface
> > - fix the error message for the position= parameter
> > - rename term "after" -> "behind" and variable "insert_before" ->
> "insert_before_flag"
> > - document the quorum node on the secondary side
> > - simplify quorum parameters in documentation
> > - remove trailing spaces in documentation
> > - clarify the testing procedure in documentation
> >
> > v2:
> > - fix email formating
> > - fix checkpatch.pl warnings
> > - fix patchew error
> > - clearer commit messages
> >
> >
> > Lukas Straub (4):
> > block/replication.c: Ignore requests after failover
> > tests/test-replication.c: Add test for for secondary node continuing
> > replication
> > net/filter.c: Add Options to insert filters anywhere in the filter
> > list
> > colo: Update Documentation for continuous replication
> >
> > block/replication.c | 35 +++++-
> > docs/COLO-FT.txt | 224 +++++++++++++++++++++++++++----------
> > docs/block-replication.txt | 28 +++--
> > include/net/filter.h | 2 +
> > net/filter.c | 92 ++++++++++++++-
> > qemu-options.hx | 31 ++++-
> > tests/test-replication.c | 52 +++++++++
> > 7 files changed, 389 insertions(+), 75 deletions(-)
> >
>
> Hello Everyone,
> So I guess this is ready for merging or will that have to wait until the 4.2
> release is done?
Due to Qemu 4.2 release schedule, after soft feature freeze(Oct29) the master
branch does not accept feature changes.
But I don't know if sub-maintainer(block or net) can queue this series first
then merge it after 4.2 release?
Thanks
Zhang Chen
>
> Regards,
> Lukas Straub