[ovs-dev] [PATCH 2/2] tests: Test ofproto-dpif set_tunnel translation.

Ben Pfaff blp at nicira.com
Tue Sep 13 02:10:55 UTC 2011


It's fine.

On Mon, Sep 12, 2011 at 06:47:36PM -0700, Ethan Jackson wrote:
> I just thought it was easier to read.  One line per output.  If you
> disagree I can squash it.
> 
> Ethan
> 
> On Mon, Sep 12, 2011 at 18:47, Ben Pfaff <blp at nicira.com> wrote:
> > On Mon, Sep 12, 2011 at 05:43:26PM -0700, Ethan Jackson wrote:
> >> This patch adds a unit test which would have caught the issue fixed
> >> in Commit 2446268e "ofproto-dpif: set_tunnel when required to."
> >
> > Did you copy this from a test of resubmit? ?Otherwise why all the
> > resubmits?
> >
> > I don't see any actual problems though. ?Thanks for adding a test.
> >



More information about the dev mailing list