On Wed, Jun 12, 2019 at 05:59:02PM +0200, Dumitru Ceara wrote: > Encap tunnel-ids are of the form: > <chassis-id><OVN_MVTEP_CHASSISID_DELIM><encap-ip>. > In physical_run we were checking if a tunnel-id corresponds > to the local chassis-id by searching if the chassis-id string > is included in the tunnel-id (strstr). This can break quite > easily, for example, if the local chassis-id is a substring > of a remote chassis-id. In that case we were wrongfully > skipping the tunnel creation. > > To fix that new tunnel-id creation and parsing functions are added in > encaps.[ch]. These functions are now used everywhere where applicable. > > Acked-by: Venu Iyer <[email protected]> > Reported-at: https://bugzilla.redhat.com/1708131 > Reported-by: Haidong Li <[email protected]> > Fixes: b520ca7 ("Support for multiple VTEP in OVN") > Signed-off-by: Dumitru Ceara <[email protected]> > Signed-off-by: Ben Pfaff <[email protected]>
Thanks! Applied to master. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
