[ovs-dev] [oftest v3 5/5] tests: Add support for running OFTest.

[Ben Pfaff]

Thank you for the reviews.  I pushed this to master.

On Tue, May 28, 2013 at 03:18:09PM -0700, Alex Wang wrote:
> And the previous email marks the completion of review of this series of
> patches. Thanks.
> 
> 
> On Tue, May 28, 2013 at 3:16 PM, Alex Wang <alexw at nicira.com> wrote:
> 
> > This patch looks good to me.
> >
> >
> > On Tue, May 28, 2013 at 11:50 AM, Alex Wang <alexw at nicira.com> wrote:
> >
> >> Thanks Ben, the README is clear to me.
> >>
> >> I'll play with the oftest more and finish all review.
> >>
> >>
> >> On Tue, May 28, 2013 at 11:44 AM, Ben Pfaff <blp at nicira.com> wrote:
> >>
> >>> On Fri, May 17, 2013 at 03:23:50PM -0700, Ben Pfaff wrote:
> >>> > +Second, obtain a copy of OFTest and install its prerequisites.  You
> >>> > +need a copy of OFTest that includes this commit available at
> >>> > +https://github.com/blp/oftest/commits/master:
> >>> > +
> >>> > +    commit 406614846c5eae01f0eb460a9f107e7ed604924f
> >>> > +    Author: Ben Pfaff <blp at nicira.com>
> >>> > +
> >>> > +    make ovs-dummy platform work again
> >>> > +
> >>> > +    Commit e1b8da9 (dataplane: single-threaded rewrite) changed the
> >>> > +    DataPlanePort required interface but it did not update the
> >>> ovs-dummy
> >>> > +    implementation to match.  This commit makes the platform work
> >>> again.
> >>>
> >>> This makes it sound as though it's necessary to get that commit
> >>> specifically.  It's not (anymore), because it has been merged into the
> >>> upstream repo.  Here's some better text that I've substituted locally:
> >>>
> >>> Second, obtain a copy of OFTest and install its prerequisites.  You
> >>> need a copy of OFTest that includes commit 406614846c5 (make ovs-dummy
> >>> platform work again).  This commit was merged into the OFTest
> >>> repository on Feb 1, 2013, so any copy of OFTest more recent than that
> >>> should work.
> >>> _______________________________________________
> >>> dev mailing list
> >>> dev at openvswitch.org
> >>> http://openvswitch.org/mailman/listinfo/dev
> >>>
> >>
> >>
> >