[PATCH] [TEST] Try to fix VSSD-02_bootldr.py by adding a 10s delay at the beginning of test

# HG changeset patch # User Guolian Yun <yunguol@cn.ibm.com> # Date 1220945607 25200 # Node ID c69e0d28bed68808d5326d84f6d3b9d70058ee0c # Parent 8abcd820b6b37e5fbe8ccc30734cefa908dfab78 [TEST] Try to fix VSSD-02_bootldr.py by adding a 10s delay at the beginning of test Signed-off-by: Guolian Yun <yunguol@cn.ibm.com> diff -r 8abcd820b6b3 -r c69e0d28bed6 suites/libvirt-cim/cimtest/VSSD/02_bootldr.py --- a/suites/libvirt-cim/cimtest/VSSD/02_bootldr.py Fri Sep 05 02:47:24 2008 -0700 +++ b/suites/libvirt-cim/cimtest/VSSD/02_bootldr.py Tue Sep 09 00:33:27 2008 -0700 @@ -27,6 +27,7 @@ # Date : 25-10-2007 import sys +from time import sleep from XenKvmLib import enumclass from VirtLib import utils from VirtLib.live import bootloader @@ -44,7 +45,7 @@ def main(): options = main.options status = 1 destroy_and_undefine_all(options.ip) - + sleep(10) xmlfile = testxml_bl(test_dom, server = options.ip, gtype = 0) ret = test_domain_function(xmlfile, options.ip, "define")

yunguol@cn.ibm.com wrote:
# HG changeset patch # User Guolian Yun <yunguol@cn.ibm.com> # Date 1220945607 25200 # Node ID c69e0d28bed68808d5326d84f6d3b9d70058ee0c # Parent 8abcd820b6b37e5fbe8ccc30734cefa908dfab78 [TEST] Try to fix VSSD-02_bootldr.py by adding a 10s delay at the beginning of test
Signed-off-by: Guolian Yun <yunguol@cn.ibm.com>
diff -r 8abcd820b6b3 -r c69e0d28bed6 suites/libvirt-cim/cimtest/VSSD/02_bootldr.py
I wouldn't use a sleep delay here. Instead, I'd modify this test so that it uses cim_define(). Also, I'd change BaseException to Exception. When this test fails, it returns the following error: ERROR - NameError : global name 'BaseException' is not defined Traceback (most recent call last): File "/data/users/kaitlin/sandbox/cimtest/suites/libvirt-cim/lib/XenKvmLib/const.py", line 115, in do_try rc = f() File "02_bootldr.py", line 73, in main except BaseException, detail : NameError: global name 'BaseException' is not defined ERROR - None This should be fixed so that we don't see the "global name 'BaseException' is not defined" error message when the test fails. Thanks! -- Kaitlin Rupert IBM Linux Technology Center kaitlin@linux.vnet.ibm.com
participants (2)
-
Kaitlin Rupert
-
yunguol@cn.ibm.com