test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts] [PATCH V1] TIMEOUT on eal_flags_autotest
@ 2015-11-17  4:13 Shijie Dong
  2015-11-18  1:22 ` Liu, Yong
  0 siblings, 1 reply; 6+ messages in thread
From: Shijie Dong @ 2015-11-17  4:13 UTC (permalink / raw)
  To: dts

add time for exec eal_flags_autotest
Signed-off-by: Shijie Dong <shijiex.dong@intel.com>
---
 tests/TestSuite_unit_tests_eal.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/TestSuite_unit_tests_eal.py b/tests/TestSuite_unit_tests_eal.py
index d0b9696..d6d21e4 100644
--- a/tests/TestSuite_unit_tests_eal.py
+++ b/tests/TestSuite_unit_tests_eal.py
@@ -34,7 +34,7 @@ class TestUnitTestsEal(TestCase):
         self.verify('make: Leaving directory' in out, "Compilation failed")
         [arch, machine, self.env, toolchain] = self.target.split('-')
         self.start_test_time = 60
-        self.run_cmd_time = 240
+        self.run_cmd_time = 300
 
     def set_up(self):
         """
-- 
2.5.0

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [dts] [PATCH V1] TIMEOUT on eal_flags_autotest
  2015-11-17  4:13 [dts] [PATCH V1] TIMEOUT on eal_flags_autotest Shijie Dong
@ 2015-11-18  1:22 ` Liu, Yong
  2015-11-18  1:42   ` Dong, ShijieX
  0 siblings, 1 reply; 6+ messages in thread
From: Liu, Yong @ 2015-11-18  1:22 UTC (permalink / raw)
  To: Shijie Dong, dts

Hi Shijie,
As i known that timeout value has been extended. Could you give more 
information why four minutes is still not enough for this test?

On 11/17/2015 12:13 PM, Shijie Dong wrote:
> add time for exec eal_flags_autotest
> Signed-off-by: Shijie Dong <shijiex.dong@intel.com>
> ---
>   tests/TestSuite_unit_tests_eal.py | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/TestSuite_unit_tests_eal.py b/tests/TestSuite_unit_tests_eal.py
> index d0b9696..d6d21e4 100644
> --- a/tests/TestSuite_unit_tests_eal.py
> +++ b/tests/TestSuite_unit_tests_eal.py
> @@ -34,7 +34,7 @@ class TestUnitTestsEal(TestCase):
>           self.verify('make: Leaving directory' in out, "Compilation failed")
>           [arch, machine, self.env, toolchain] = self.target.split('-')
>           self.start_test_time = 60
> -        self.run_cmd_time = 240
> +        self.run_cmd_time = 300
>   
>       def set_up(self):
>           """

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [dts] [PATCH V1] TIMEOUT on eal_flags_autotest
  2015-11-18  1:22 ` Liu, Yong
@ 2015-11-18  1:42   ` Dong, ShijieX
  2015-11-18  1:48     ` Liu, Yong
  0 siblings, 1 reply; 6+ messages in thread
From: Dong, ShijieX @ 2015-11-18  1:42 UTC (permalink / raw)
  To: Liu, Yong, dts

Hi Liuyong
The original value is 60 seconds, I test is 120 seconds. OK, because of the "fialed several OS, in order to prevent the other systems but.

-----Original Message-----
From: Liu, Yong 
Sent: Wednesday, November 18, 2015 9:23 AM
To: Dong, ShijieX; dts@dpdk.org
Subject: Re: [dts] [PATCH V1] TIMEOUT on eal_flags_autotest

Hi Shijie,
As i known that timeout value has been extended. Could you give more information why four minutes is still not enough for this test?

On 11/17/2015 12:13 PM, Shijie Dong wrote:
> add time for exec eal_flags_autotest
> Signed-off-by: Shijie Dong <shijiex.dong@intel.com>
> ---
>   tests/TestSuite_unit_tests_eal.py | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/TestSuite_unit_tests_eal.py 
> b/tests/TestSuite_unit_tests_eal.py
> index d0b9696..d6d21e4 100644
> --- a/tests/TestSuite_unit_tests_eal.py
> +++ b/tests/TestSuite_unit_tests_eal.py
> @@ -34,7 +34,7 @@ class TestUnitTestsEal(TestCase):
>           self.verify('make: Leaving directory' in out, "Compilation failed")
>           [arch, machine, self.env, toolchain] = self.target.split('-')
>           self.start_test_time = 60
> -        self.run_cmd_time = 240
> +        self.run_cmd_time = 300
>   
>       def set_up(self):
>           """

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [dts] [PATCH V1] TIMEOUT on eal_flags_autotest
  2015-11-18  1:42   ` Dong, ShijieX
@ 2015-11-18  1:48     ` Liu, Yong
  2015-11-18  1:50       ` Dong, ShijieX
  0 siblings, 1 reply; 6+ messages in thread
From: Liu, Yong @ 2015-11-18  1:48 UTC (permalink / raw)
  To: Dong, ShijieX, dts

But from the log, the timeout value extended from 240 to 300, so i asked 
the question.
One remind, please create your patch based on latest DTS code, not your 
local code.

On 11/18/2015 09:42 AM, Dong, ShijieX wrote:
> Hi Liuyong
> The original value is 60 seconds, I test is 120 seconds. OK, because of the "fialed several OS, in order to prevent the other systems but.
>
> -----Original Message-----
> From: Liu, Yong
> Sent: Wednesday, November 18, 2015 9:23 AM
> To: Dong, ShijieX; dts@dpdk.org
> Subject: Re: [dts] [PATCH V1] TIMEOUT on eal_flags_autotest
>
> Hi Shijie,
> As i known that timeout value has been extended. Could you give more information why four minutes is still not enough for this test?
>
> On 11/17/2015 12:13 PM, Shijie Dong wrote:
>> add time for exec eal_flags_autotest
>> Signed-off-by: Shijie Dong <shijiex.dong@intel.com>
>> ---
>>    tests/TestSuite_unit_tests_eal.py | 2 +-
>>    1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/tests/TestSuite_unit_tests_eal.py
>> b/tests/TestSuite_unit_tests_eal.py
>> index d0b9696..d6d21e4 100644
>> --- a/tests/TestSuite_unit_tests_eal.py
>> +++ b/tests/TestSuite_unit_tests_eal.py
>> @@ -34,7 +34,7 @@ class TestUnitTestsEal(TestCase):
>>            self.verify('make: Leaving directory' in out, "Compilation failed")
>>            [arch, machine, self.env, toolchain] = self.target.split('-')
>>            self.start_test_time = 60
>> -        self.run_cmd_time = 240
>> +        self.run_cmd_time = 300
>>    
>>        def set_up(self):
>>            """

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [dts] [PATCH V1] TIMEOUT on eal_flags_autotest
  2015-11-18  1:48     ` Liu, Yong
@ 2015-11-18  1:50       ` Dong, ShijieX
  2015-11-18  7:22         ` Liu, Yong
  0 siblings, 1 reply; 6+ messages in thread
From: Dong, ShijieX @ 2015-11-18  1:50 UTC (permalink / raw)
  To: Liu, Yong, dts

I am this is DTS, this value I made two changes, the next time I pay attention, this need to re do a patch?

-----Original Message-----
From: Liu, Yong 
Sent: Wednesday, November 18, 2015 9:49 AM
To: Dong, ShijieX; dts@dpdk.org
Subject: Re: [dts] [PATCH V1] TIMEOUT on eal_flags_autotest

But from the log, the timeout value extended from 240 to 300, so i asked the question.
One remind, please create your patch based on latest DTS code, not your local code.

On 11/18/2015 09:42 AM, Dong, ShijieX wrote:
> Hi Liuyong
> The original value is 60 seconds, I test is 120 seconds. OK, because of the "fialed several OS, in order to prevent the other systems but.
>
> -----Original Message-----
> From: Liu, Yong
> Sent: Wednesday, November 18, 2015 9:23 AM
> To: Dong, ShijieX; dts@dpdk.org
> Subject: Re: [dts] [PATCH V1] TIMEOUT on eal_flags_autotest
>
> Hi Shijie,
> As i known that timeout value has been extended. Could you give more information why four minutes is still not enough for this test?
>
> On 11/17/2015 12:13 PM, Shijie Dong wrote:
>> add time for exec eal_flags_autotest
>> Signed-off-by: Shijie Dong <shijiex.dong@intel.com>
>> ---
>>    tests/TestSuite_unit_tests_eal.py | 2 +-
>>    1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/tests/TestSuite_unit_tests_eal.py
>> b/tests/TestSuite_unit_tests_eal.py
>> index d0b9696..d6d21e4 100644
>> --- a/tests/TestSuite_unit_tests_eal.py
>> +++ b/tests/TestSuite_unit_tests_eal.py
>> @@ -34,7 +34,7 @@ class TestUnitTestsEal(TestCase):
>>            self.verify('make: Leaving directory' in out, "Compilation failed")
>>            [arch, machine, self.env, toolchain] = self.target.split('-')
>>            self.start_test_time = 60
>> -        self.run_cmd_time = 240
>> +        self.run_cmd_time = 300
>>    
>>        def set_up(self):
>>            """

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [dts] [PATCH V1] TIMEOUT on eal_flags_autotest
  2015-11-18  1:50       ` Dong, ShijieX
@ 2015-11-18  7:22         ` Liu, Yong
  0 siblings, 0 replies; 6+ messages in thread
From: Liu, Yong @ 2015-11-18  7:22 UTC (permalink / raw)
  To: Dong, ShijieX, dts

It's better to send out new patch:)

> -----Original Message-----
> From: Dong, ShijieX
> Sent: Wednesday, November 18, 2015 9:51 AM
> To: Liu, Yong; dts@dpdk.org
> Subject: RE: [dts] [PATCH V1] TIMEOUT on eal_flags_autotest
> 
> I am this is DTS, this value I made two changes, the next time I pay
> attention, this need to re do a patch?
> 
> -----Original Message-----
> From: Liu, Yong
> Sent: Wednesday, November 18, 2015 9:49 AM
> To: Dong, ShijieX; dts@dpdk.org
> Subject: Re: [dts] [PATCH V1] TIMEOUT on eal_flags_autotest
> 
> But from the log, the timeout value extended from 240 to 300, so i asked
> the question.
> One remind, please create your patch based on latest DTS code, not your
> local code.
> 
> On 11/18/2015 09:42 AM, Dong, ShijieX wrote:
> > Hi Liuyong
> > The original value is 60 seconds, I test is 120 seconds. OK, because of
> the "fialed several OS, in order to prevent the other systems but.
> >
> > -----Original Message-----
> > From: Liu, Yong
> > Sent: Wednesday, November 18, 2015 9:23 AM
> > To: Dong, ShijieX; dts@dpdk.org
> > Subject: Re: [dts] [PATCH V1] TIMEOUT on eal_flags_autotest
> >
> > Hi Shijie,
> > As i known that timeout value has been extended. Could you give more
> information why four minutes is still not enough for this test?
> >
> > On 11/17/2015 12:13 PM, Shijie Dong wrote:
> >> add time for exec eal_flags_autotest
> >> Signed-off-by: Shijie Dong <shijiex.dong@intel.com>
> >> ---
> >>    tests/TestSuite_unit_tests_eal.py | 2 +-
> >>    1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git a/tests/TestSuite_unit_tests_eal.py
> >> b/tests/TestSuite_unit_tests_eal.py
> >> index d0b9696..d6d21e4 100644
> >> --- a/tests/TestSuite_unit_tests_eal.py
> >> +++ b/tests/TestSuite_unit_tests_eal.py
> >> @@ -34,7 +34,7 @@ class TestUnitTestsEal(TestCase):
> >>            self.verify('make: Leaving directory' in out, "Compilation
> failed")
> >>            [arch, machine, self.env, toolchain] = self.target.split('-')
> >>            self.start_test_time = 60
> >> -        self.run_cmd_time = 240
> >> +        self.run_cmd_time = 300
> >>
> >>        def set_up(self):
> >>            """

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2015-11-18  7:22 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-11-17  4:13 [dts] [PATCH V1] TIMEOUT on eal_flags_autotest Shijie Dong
2015-11-18  1:22 ` Liu, Yong
2015-11-18  1:42   ` Dong, ShijieX
2015-11-18  1:48     ` Liu, Yong
2015-11-18  1:50       ` Dong, ShijieX
2015-11-18  7:22         ` Liu, Yong

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).