On Fri, Nov 01, 2019 at 01:05:15PM +0100, Dumitru Ceara wrote:
> Until now, when translating SB database contents to OpenFlow flows,
> ovn-controller stored (part of) the SB record UUID in the cookie field
> of the OpenFlow entry only when translating logical flows.
> 
> Extend this behavior to the following SB Database table records too:
> - Port_Binding
> - Mac_Binding
> - Multicast_Group
> - Chassis
> 
> This makes debugging easier by allowing the user to trace back the
> original SB entry that generated the OpenFlow entry. Also, the
> "ovn-sbctl lflow-list" command now supports an additional "--vflows"
> argument (virtual flows). When present this will instruct ovn-sbctl
> to also pretty print the contents of the above mentioned tables for
> the given datapath (or all datapaths if none is specified).

This is smart and I fully endorse the idea.

I haven't properly reviewed the patch, so the following is for the idea:
Acked-by: Ben Pfaff <[email protected]>
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to