Re: [ovs-dev] [unixctl 2/2] unixctl: Timeout unit tests instead of hanging.

2012-03-07 Thread Ethan Jackson
Thanks, I've removed the redundant double [] and merged this to master. Ethan On Wed, Mar 7, 2012 at 12:45, Ben Pfaff wrote: > On Tue, Mar 06, 2012 at 05:53:08PM -0800, Ethan Jackson wrote: >> We've seen some unixctl tests hang indefinitely which makes them >> difficult to debug.  ovs-appctl and

Re: [ovs-dev] [unixctl 2/2] unixctl: Timeout unit tests instead of hanging.

2012-03-07 Thread Ben Pfaff
On Tue, Mar 06, 2012 at 05:53:08PM -0800, Ethan Jackson wrote: > We've seen some unixctl tests hang indefinitely which makes them > difficult to debug. ovs-appctl and appctl.py calls to timeout > instead. > > Signed-off-by: Ethan Jackson I don't see any reason to double the [] in the definition

[ovs-dev] [unixctl 2/2] unixctl: Timeout unit tests instead of hanging.

2012-03-06 Thread Ethan Jackson
We've seen some unixctl tests hang indefinitely which makes them difficult to debug. ovs-appctl and appctl.py calls to timeout instead. Signed-off-by: Ethan Jackson --- tests/appctl.py|6 + tests/test-unixctl.py |5 tests/unixctl-py.at| 66 +++