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@xxxxxxxxxxxxxxxxxx>
>
> 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@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
