Re: [PATCH v7 0/4] colo: Add support for continuous replication

[Zhang, Chen]

Hi Jason,

I noticed this series can't be merged or queued, do you met some problem 
about it?


Thanks

Zhang Chen


> > > > 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
> >
> > Will try to queue this series.
> Thank you Jason~
>
> Thanks
> Zhang Chen
>
> > Thanks
> >
> >
> > > > Regards,
> > > > Lukas Straub