[ovs-dev] [oftest v3 5/5] tests: Add support for running OFTest.
Ben Pfaff
blp at nicira.com
Tue May 28 22:42:23 UTC 2013
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
> >>>
> >>
> >>
> >
More information about the dev
mailing list