[ovs-dev] [PATCH v2 2/3] OVN: Add IGMP SB definitions and ovn-controller support

Dumitru Ceara dceara at redhat.com
Thu Jul 11 08:21:45 UTC 2019


On Wed, Jul 10, 2019 at 10:44 PM Ben Pfaff <blp at ovn.org> wrote:
>
> On Tue, Jul 09, 2019 at 02:49:18PM +0200, Dumitru Ceara wrote:
> > A new action ("igmp") is added to punt IGMP packets on a specific logical
> > switch datapath to ovn-controller if IGMP snooping is enabled.
>
> I do not fully understand the new action.  It is defined to take a set
> of nested actions inside {}, but I don't see what it actually does with
> those actions.  I do not think that it executes them, and the
> documentation does not mention them.  Maybe this means that it is a
> mistake to have it take them at all?

You're right, it's a mistake. It should be "igmp;" with no nested
actions. It will be fixed in v3.

Thanks,
Dumitru


More information about the dev mailing list