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

2013-05-28 Thread 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 wrote: > > > This patch looks good to me

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

2013-05-28 Thread Alex Wang
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 wrote: > This patch looks good to me. > > > On Tue, May 28, 2013 at 11:50 AM, Alex Wang wrote: > >> Thanks Ben, the README is clear to me. >> >> I'll play with th

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

2013-05-28 Thread Alex Wang
This patch looks good to me. On Tue, May 28, 2013 at 11:50 AM, Alex Wang 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 wrote: > >> On Fri, May 17, 2013 at 03:23:50PM -0700, Ben Pfaff w

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

2013-05-28 Thread Alex Wang
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 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

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

2013-05-28 Thread Ben Pfaff
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 406614846c5eae01f0eb460a9f107e7ed60492

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

2013-05-17 Thread Ben Pfaff
Signed-off-by: Ben Pfaff --- Makefile.am |1 + NEWS |2 + README-OFTest | 77 +++ tests/automake.mk |6 +++ tests/run-oftest | 94 + 5 files changed, 180 insertions(