8:46 p.m.
on 2014/01/09 22:26, Sheldon wrote:
Just a minor comment below
I have encounter this problems several times. not always happen.
======================================================================
FAIL: test_debug_reports (test_model.ModelTests)
----------------------------------------------------------------------
Traceback (most recent call last):
File "/run/jenkins/workspace/kimchi_test/tests/test_model.py",
line 639, in test_debug_reports
raise e
File "/run/jenkins/workspace/kimchi_test/tests/test_model.py",
line 632, in test_debug_reports
"It is not necessary an error. "
AssertionError: It is not necessary an error. You may need to
increase the timeout number in _wait_task()
----------------------------------------------------------------------
Ran 114 tests in 130.198s
FAILED (failures=1)
We need this patch.
On 01/09/2014 06:40 PM, Shu Ming wrote:
> It is helpful when the timeout happens and the tester can tune it
you can give more information tell how to tune timeout
$ TEST_REPORT_TIMEOUT=120 ./run_tests.sh test_mode
Agree. You can update the timeout exception message bellow, and tell the
user to set TEST_REPORT_TIMEOUT environment variable to a larger value.
>
> Signed-off-by: Shu Ming <shuming(a)linux.vnet.ibm.com>
> ---
> tests/test_model.py | 7 ++++++-
> 1 file changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/tests/test_model.py b/tests/test_model.py
> index dad4670..41032f7 100644
> --- a/tests/test_model.py
> +++ b/tests/test_model.py
> @@ -614,6 +614,11 @@ class ModelTests(unittest.TestCase):
> if not inst.get_capabilities()['system_report_tool']:
> raise unittest.SkipTest("Without debug report tool")
>
> + try:
> + timeout = int(os.environ['TEST_REPORT_TIMEOUT'])
> + except (ValueError, KeyError):
> + timeout = 120
> +
> namePrefix = 'unitTestReport'
> # sosreport always deletes unsual letters like '-' and '_'
> in the
> # generated report file name.
> @@ -630,7 +635,7 @@ class ModelTests(unittest.TestCase):
> task = inst.debugreports_create({'name': reportName})
> rollback.prependDefer(inst.debugreport_delete,
> reportName)
> taskid = task['id']
> - self._wait_task(inst, taskid, 60)
> + self._wait_task(inst, taskid, timeout)
> self.assertEquals('finished',
> inst.task_lookup(taskid)['status'],
> "It is not necessary an error. "
_______________________________________________
Kimchi-devel mailing list
Kimchi-devel(a)ovirt.org
http://lists.ovirt.org/mailman/listinfo/kimchi-devel
--
Thanks and best regards!
Zhou Zheng Sheng / 周征晟
E-mail: zhshzhou(a)linux.vnet.ibm.com
Telephone: 86-10-82454397