On 6 Mar 2023, at 3:49, Nobuhiro MIKI wrote:

> v7:
> - Fix error message
> v6:
> - Fix notation quirks between man page and in-app help
> - Fix error message in ovs/route/add command
> - Fix coding style
> v5:
> - Add patch to fix man page
> v4:
> - Add cleanup patch for ovs/route/add
> - Remove unrelated code
> v3:
> - Fix netdev-dummy to support multiple IP addresses
> - Add validation and unit tests for ovs/route/add
> - Refactor parsing for optional parameters in ovs/route/add command
> v2:
> - Add NEWS
>
> Nobuhiro MIKI (5):
>   netdev-dummy: Support multiple IP addresses.
>   ovs-router: Cleanup parser for ovs/route/add command.
>   ofproto: Fix mam page for tunnel related commands.
>   ovs-router: Introduce src option in ovs/route/add command.
>   route-table: Retrieving the preferred source address from Netlink.
>
>  NEWS                            |   3 +
>  lib/netdev-dummy.c              |  70 +++++++++------
>  lib/ovs-router.c                | 145 ++++++++++++++++++++++++--------
>  lib/ovs-router.h                |   3 +-
>  lib/route-table.c               |  16 +++-
>  ofproto/ofproto-tnl-unixctl.man |   9 +-
>  tests/ovs-router.at             | 105 ++++++++++++++++++++++-
>  tests/system-route.at           |  39 +++++++++
>  8 files changed, 317 insertions(+), 73 deletions(-)

Thanks for this revision, it fixes all my remaining comments.

So for clarity, I’ll ack the series explicitly.

Acked-by: Eelco Chaudron <[email protected]>

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to