Re: [ovs-dev] [PATCH] ovs-macros: An option to suspend test execution on error

2019-07-15 Thread 0-day Robot
Bleep bloop. Greetings Vasu Dasari, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: WARNING: Line has trailing whitespace #58 FILE: Documentation/topics/testing.rst:78: state, foll

Re: [ovs-dev] [PATCH] ovs-macros: An option to suspend test execution on error

2019-07-15 Thread Vasu Dasari
Please ignore this. *Vasu Dasari* On Mon, Jul 15, 2019 at 3:22 PM Vasu Dasari wrote: > Origins for this patch are captured at > https://mail.openvswitch.org/pipermail/ovs-discuss/2019-June/048923.html. > > Summarizing here, when a test fails, it would be good to pause test > execution > and le

[ovs-dev] [PATCH] ovs-macros: An option to suspend test execution on error

2019-07-15 Thread Vasu Dasari
Origins for this patch are captured at https://mail.openvswitch.org/pipermail/ovs-discuss/2019-June/048923.html. Summarizing here, when a test fails, it would be good to pause test execution and let the developer poke around the system to see current status of system. As part of this patch, made

Re: [ovs-dev] [PATCH] ovs-macros: An option to suspend test execution on error

2019-07-12 Thread Vasu Dasari
Sure. Will do. *Vasu Dasari* On Fri, Jul 12, 2019 at 12:08 PM Ben Pfaff wrote: > On Fri, Jul 12, 2019 at 08:42:51AM -0700, Ben Pfaff wrote: > > On Thu, Jul 11, 2019 at 11:46:04AM -0400, Vasu Dasari wrote: > > > Origins for this patch are captured at > > > > https://mail.openvswitch.org/piperma

Re: [ovs-dev] [PATCH] ovs-macros: An option to suspend test execution on error

2019-07-12 Thread Ben Pfaff
On Fri, Jul 12, 2019 at 08:42:51AM -0700, Ben Pfaff wrote: > On Thu, Jul 11, 2019 at 11:46:04AM -0400, Vasu Dasari wrote: > > Origins for this patch are captured at > > https://mail.openvswitch.org/pipermail/ovs-discuss/2019-June/048923.html. > > > > Summarizing here, when a test fails, it would b

Re: [ovs-dev] [PATCH] ovs-macros: An option to suspend test execution on error

2019-07-12 Thread Ben Pfaff
Thanks. I see I wrote "replies" below when I meant "relies". (Maybe that was obvious.) On Fri, Jul 12, 2019 at 11:45:11AM -0400, Vasu Dasari wrote: > Thanks. Will do. > > *Vasu Dasari* > > > On Fri, Jul 12, 2019 at 11:43 AM Ben Pfaff wrote: > > > On Thu, Jul 11, 2019 at 11:46:04AM -0400, Va

Re: [ovs-dev] [PATCH] ovs-macros: An option to suspend test execution on error

2019-07-12 Thread Vasu Dasari
Thanks. Will do. *Vasu Dasari* On Fri, Jul 12, 2019 at 11:43 AM Ben Pfaff wrote: > On Thu, Jul 11, 2019 at 11:46:04AM -0400, Vasu Dasari wrote: > > Origins for this patch are captured at > > https://mail.openvswitch.org/pipermail/ovs-discuss/2019-June/048923.html > . > > > > Summarizing here,

Re: [ovs-dev] [PATCH] ovs-macros: An option to suspend test execution on error

2019-07-12 Thread Ben Pfaff
On Thu, Jul 11, 2019 at 11:46:04AM -0400, Vasu Dasari wrote: > Origins for this patch are captured at > https://mail.openvswitch.org/pipermail/ovs-discuss/2019-June/048923.html. > > Summarizing here, when a test fails, it would be good to pause test execution > and let the developer poke around th

[ovs-dev] [PATCH] ovs-macros: An option to suspend test execution on error

2019-07-11 Thread Vasu Dasari
Origins for this patch are captured at https://mail.openvswitch.org/pipermail/ovs-discuss/2019-June/048923.html. Summarizing here, when a test fails, it would be good to pause test execution and let the developer poke around the system to see current status of system. As part of this patch, made