From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lucas Meneghel Rodrigues Subject: Re: [Autotest] [AUTOTEST] [PATCH 2/2] View LTP execution results under kvm's results.html file Date: Mon, 06 Jul 2009 04:15:31 -0300 Message-ID: <1246864531.2865.56.camel@localhost.localdomain> References: Mime-Version: 1.0 Content-Type: text/plain Content-Transfer-Encoding: 7bit Cc: Autotest mailing list , Lucas Meneghel Rodrigues , Uri Lublin , kvm-devel To: sudhir kumar Return-path: Received: from mx2.redhat.com ([66.187.237.31]:55813 "EHLO mx2.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752638AbZGFHPd (ORCPT ); Mon, 6 Jul 2009 03:15:33 -0400 In-Reply-To: Sender: kvm-owner@vger.kernel.org List-ID: 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 _results.html and it is located under > /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 > > 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