Re: [ovs-dev] [patch v1] tests: Include ovn-macros for system testsuites.

2019-06-04 Thread Darrell Ball
On Tue, Jun 4, 2019 at 8:44 AM Ilya Maximets wrote: > On 04.06.2019 7:37, Darrell Ball wrote: > > > > > > On Mon, Jun 3, 2019 at 10:27 AM Ilya Maximets > wrote: > > > > On 03.06.2019 18:55, Darrell Ball wrote: > > > > > > > > > On Mon, Jun 3, 2019

Re: [ovs-dev] [patch v1] tests: Include ovn-macros for system testsuites.

2019-06-04 Thread Ilya Maximets
On 04.06.2019 7:37, Darrell Ball wrote: > > > On Mon, Jun 3, 2019 at 10:27 AM Ilya Maximets > wrote: > > On 03.06.2019 18:55, Darrell Ball wrote: > > > > > > On Mon, Jun 3, 2019 at 12:48 AM Ilya Maximets

Re: [ovs-dev] [patch v1] tests: Include ovn-macros for system testsuites.

2019-06-03 Thread Darrell Ball
On Mon, Jun 3, 2019 at 10:27 AM Ilya Maximets wrote: > On 03.06.2019 18:55, Darrell Ball wrote: > > > > > > On Mon, Jun 3, 2019 at 12:48 AM Ilya Maximets > wrote: > > > > On 25.05.2019 23:49, Darrell Ball wrote: > > > ovn-macros are needed to run the OVN

Re: [ovs-dev] [patch v1] tests: Include ovn-macros for system testsuites.

2019-06-03 Thread Ilya Maximets
On 25.05.2019 23:49, Darrell Ball wrote: > ovn-macros are needed to run the OVN system tests. > > Signed-off-by: Darrell Ball > Tested-by: Greg Rose > Reviewed-by: Greg Rose > --- > tests/system-kmod-testsuite.at | 1 + > tests/system-userspace-testsuite.at | 1 + > 2 files changed, 2

Re: [ovs-dev] [patch v1] tests: Include ovn-macros for system testsuites.

2019-06-03 Thread Ilya Maximets
On 03.06.2019 18:55, Darrell Ball wrote: > > > On Mon, Jun 3, 2019 at 12:48 AM Ilya Maximets > wrote: > > On 25.05.2019 23:49, Darrell Ball wrote: > > ovn-macros are needed to run the OVN system tests. > > > > Signed-off-by: Darrell Ball

Re: [ovs-dev] [patch v1] tests: Include ovn-macros for system testsuites.

2019-06-03 Thread Darrell Ball
On Mon, Jun 3, 2019 at 12:48 AM Ilya Maximets wrote: > On 25.05.2019 23:49, Darrell Ball wrote: > > ovn-macros are needed to run the OVN system tests. > > > > Signed-off-by: Darrell Ball > > Tested-by: Greg Rose > > Reviewed-by: Greg Rose > > --- > > Hi Darrell. > Thanks for the fix. It looks

Re: [ovs-dev] [patch v1] tests: Include ovn-macros for system testsuites.

2019-06-03 Thread Ilya Maximets
On 25.05.2019 23:49, Darrell Ball wrote: > ovn-macros are needed to run the OVN system tests. > > Signed-off-by: Darrell Ball > Tested-by: Greg Rose > Reviewed-by: Greg Rose > --- Hi Darrell. Thanks for the fix. It looks good. BTW, I have a question. Are you able to run 'make

Re: [ovs-dev] [patch v1] tests: Include ovn-macros for system testsuites.

2019-05-30 Thread Gregory Rose
On 5/25/2019 1:49 PM, Darrell Ball wrote: ovn-macros are needed to run the OVN system tests. Signed-off-by: Darrell Ball --- tests/system-kmod-testsuite.at | 1 + tests/system-userspace-testsuite.at | 1 + 2 files changed, 2 insertions(+) diff --git a/tests/system-kmod-testsuite.at

Re: [ovs-dev] [patch v1] tests: Include ovn-macros for system testsuites.

2019-05-28 Thread Aaron Conole
Darrell Ball writes: > On Sat, May 25, 2019 at 2:19 PM 0-day Robot wrote: > > Bleep bloop. Greetings Darrell Ball, 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. > > build: >

Re: [ovs-dev] [patch v1] tests: Include ovn-macros for system testsuites.

2019-05-28 Thread Darrell Ball
On Sat, May 25, 2019 at 2:19 PM 0-day Robot wrote: > Bleep bloop. Greetings Darrell Ball, 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. > > > build: > sysconfdir='/usr/local/etc' \

Re: [ovs-dev] [patch v1] tests: Include ovn-macros for system testsuites.

2019-05-25 Thread 0-day Robot
Bleep bloop. Greetings Darrell Ball, 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. build: sysconfdir='/usr/local/etc' \ bindir='/usr/local/bin' \ sbindir='/usr/local/sbin' mv

[ovs-dev] [patch v1] tests: Include ovn-macros for system testsuites.

2019-05-25 Thread Darrell Ball
ovn-macros are needed to run the OVN system tests. Signed-off-by: Darrell Ball --- tests/system-kmod-testsuite.at | 1 + tests/system-userspace-testsuite.at | 1 + 2 files changed, 2 insertions(+) diff --git a/tests/system-kmod-testsuite.at b/tests/system-kmod-testsuite.at index