| Message ID | a50cf5ab0907052243g77bc2a82wee3ffb0be3d037c@mail.gmail.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Mon, 2009-07-06 at 11:13 +0530, sudhir kumar wrote: > This patch creates a link to the results html file generated by the test > under autotest. This is specific to the kvm part only. The assumption made is > that the file name is <test_name>_results.html and it is located under > <test_name>/results/ directory. This helps in quickly viewing the test results. > > The attached tar file contains the full results directory. The results.html file > points to ltp_results.html which looks quite fancy. > > Please have a look at the results and the patch and provide your comments. > > Signed-off-by: Sudhir Kumar <skumar@linux.vnet.ibm.com> > > Index: autotest/client/tests/kvm/kvm_tests.py > =================================================================== > --- autotest.orig/client/tests/kvm/kvm_tests.py > +++ autotest/client/tests/kvm/kvm_tests.py > @@ -391,6 +391,15 @@ def run_autotest(test, params, env): > if not vm.scp_from_remote("autotest/results/default/*", guest_results_dir): > logging.error("Could not copy results back from guest") > > + # Some tests create html file as a result, link it to be viewed under > + # the results statistics. We assume this file is located under > + # test_name/results/ directory and named as test_name_results.html, > + # e.g. ltp_result.html, vmmstress_results.html > + html_file = test_name + "_results.html" > + html_path = os.path.join(guest_results_dir, test_name, "results", > html_file) > + if os.path.exists(html_path): > + os.symlink(html_path, os.path.join(test.debugdir, html_file)) > + > # Fail the test if necessary > if status_fail: > raise error.TestFail(message_fail) Looks good to me -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Index: autotest/client/tests/kvm/kvm_tests.py =================================================================== --- autotest.orig/client/tests/kvm/kvm_tests.py +++ autotest/client/tests/kvm/kvm_tests.py @@ -391,6 +391,15 @@ def run_autotest(test, params, env): if not vm.scp_from_remote("autotest/results/default/*", guest_results_dir): logging.error("Could not copy results back from guest") + # Some tests create html file as a result, link it to be viewed under + # the results statistics. We assume this file is located under + # test_name/results/ directory and named as test_name_results.html, + # e.g. ltp_result.html, vmmstress_results.html + html_file = test_name + "_results.html" + html_path = os.path.join(guest_results_dir, test_name, "results", html_file) + if os.path.exists(html_path): + os.symlink(html_path, os.path.join(test.debugdir, html_file)) + # Fail the test if necessary if status_fail:
This patch creates a link to the results html file generated by the test under autotest. This is specific to the kvm part only. The assumption made is that the file name is <test_name>_results.html and it is located under <test_name>/results/ directory. This helps in quickly viewing the test results. The attached tar file contains the full results directory. The results.html file points to ltp_results.html which looks quite fancy. Please have a look at the results and the patch and provide your comments. Signed-off-by: Sudhir Kumar <skumar@linux.vnet.ibm.com> raise error.TestFail(message_fail)