On 8 May 2017 at 11:34, Greg Rose <[email protected]> wrote: > On Mon, 2017-05-08 at 11:15 -0700, Joe Stringer wrote: >> This reverts commit f1dac5128ce6db2e493f0d1c7a8b53fb9f34476f. When this >> commit was introduced, it broke the 'make check-system-userspace' >> testsuite. It appears that the new translation fails to modify the flow >> in a way that would represent the flow as an encapsulated flow when the >> traffic is patched through to the second bridge. As such, rather than >> matching on, for example, "ip,proto=47" for gre, it would use the inner >> packet's flow headers. It also results in problems reporting statistics, >> as the tunnel's header is not reflected in subsequent statistics and >> truncation is not properly applied during translation. >> >> While a refreshed approach to solving the above problem is formed, >> revert this patch. >> >> Reported-at: >> https://mail.openvswitch.org/pipermail/ovs-dev/2017-May/331972.html >> Signed-off-by: Joe Stringer <[email protected]> > > Acked-by: Greg Rose <[email protected]>
Thanks for the review, applied to master. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
