On Mon, Nov 27, 2023 at 7:36 PM Ilya Maximets <[email protected]> wrote:

> On 11/16/23 16:08, Ales Musil wrote:
> > The CT flush is not capable of partial flush based
> > on the touples and zone combinations. Extend it
> > so it also accepts mark and labels. As part of this
> > series unify parsing of arguments in dpctl and ovs-ofctl
> > for the ct flush command.
>
> Hi, Ales.  Thanks for the new version!
>
> Are you planning to send a fix for unknown properties?
> I think, we need to get it before extending the functionality
> as we'll also need to backport it.
>
>
I was under the impression that we want to do it after, my bad. I'll send a
fix so that this extension can be rebased on top later on.


>
> Best regards, Ilya Maximets.
>
> >
> > Ales Musil (3):
> >   ofp-prop: Add helper for parsing and storing of ovs_u128.
> >   dpctl, ovs-ofctl: Unify parsing of ct-flush arguments.
> >   openflow: Allow CT flush to match on mark and labels.
> >
> >  NEWS                           |   4 +
> >  include/openflow/nicira-ext.h  |   4 +
> >  include/openvswitch/ofp-ct.h   |  14 ++-
> >  include/openvswitch/ofp-prop.h |   5 +
> >  lib/ct-dpif.c                  |  12 ++-
> >  lib/dpctl.c                    |  46 ++-------
> >  lib/ofp-ct.c                   | 182 ++++++++++++++++++++++++++++++++-
> >  lib/ofp-prop.c                 |  42 ++++++++
> >  tests/ofp-print.at             |  85 +++++++++++++++
> >  tests/ovs-ofctl.at             |  42 +++++++-
> >  tests/system-traffic.at        | 112 +++++++++++++-------
> >  utilities/ovs-ofctl.8.in       |  31 +++---
> >  utilities/ovs-ofctl.c          |  49 +++------
> >  13 files changed, 489 insertions(+), 139 deletions(-)
> >
>
>
Thanks,
Ales

-- 

Ales Musil

Senior Software Engineer - OVN Core

Red Hat EMEA <https://www.redhat.com>

[email protected]
<https://red.ht/sig>
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to