From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mike Burns Subject: Re: [PATCH][KVM-AUTOTEST] Add custom install option for kvm_install Date: Tue, 12 May 2009 11:28:32 -0400 Message-ID: <4A0995A0.8040302@redhat.com> References: <471764781.379441242067968821.JavaMail.root@zmail05.collab.prod.int.phx2.redhat.com> <1295446409.380601242068429782.JavaMail.root@zmail05.collab.prod.int.phx2.redhat.com> <1242068979-sup-1532@blackpad> Reply-To: mburns@redhat.com Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: Michael Goldish , "kvm@vger.kernel.org" , Uri Lublin To: Eduardo Pereira Habkost Return-path: Received: from mx2.redhat.com ([66.187.237.31]:60088 "EHLO mx2.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751318AbZELP2e (ORCPT ); Tue, 12 May 2009 11:28:34 -0400 Received: from int-mx2.corp.redhat.com (int-mx2.corp.redhat.com [172.16.27.26]) by mx2.redhat.com (8.13.8/8.13.8) with ESMTP id n4CFSZWb014088 for ; Tue, 12 May 2009 11:28:35 -0400 In-Reply-To: <1242068979-sup-1532@blackpad> Sender: kvm-owner@vger.kernel.org List-ID: Eduardo Pereira Habkost wrote: > Excerpts from Michael Goldish's message of Mon May 11 16:00:29 -0300 2009: > >> ----- "Eduardo Habkost" wrote: >> >> > > >>> Also, can't the install rule be used on the cartesian configuration >>> file? In this case, the install parameters may be specified on a >>> different config rule. >>> >> It's certainly possible. We chose to put kvm_install in the control >> file because currently it's the only test we run only once per job. >> There is no natural place for kvm_install in kvm_tests.cfg because >> everything else there gets multiplied. >> > > True. I haven't thought of that. > > >>> For example, the Fedora project could use it like this: >>> >>> cvs_server = "cvs.fedoraproject.org:/..." >>> variants: >>> - CVSF10: >>> cvs_branch = F10 >>> - CVSF11: >>> cvs_branch = F11 >>> - CVSRawhide: >>> cvs_branch = devel >>> >>> >>> variants: >>> - install: >>> type = kvm_install >>> mode = custom >>> install_command = "install_from_fedora_cvs.sh" >>> - ... >>> >>> I used Fedora CVS as an example, but the user may use anything we can >>> imagine, to store KVM code (or pointer to its), possibly having >>> different branches. >>> >> This can be done, though the order and syntax of the 'variants' blocks >> should be different, more like: >> >> variants: >> - kvm_install: >> cvs_server = "cvs.fedoraproject.org:/..." >> type = kvm_install >> mode = custom >> install_command = "install_from_fedora_cvs.sh" >> - everything_else_in_kvm_tests.cfg: >> >> ... >> >> ... >> >> ... >> >> ... >> > > We could have something to make it easier to build test runs like , > without having to nest the whole original file. But I can't think of a > good solution. > > > >> variants: >> - CVSF10: >> kvm_install: >> cvs_branch = F10 >> - CVSF11: >> kvm_install: >> cvs_branch = F11 >> - CVSRawhide: >> kvm_install: >> cvs_branch = devel >> >> # test sets >> variants: >> - nightly: >> ... >> >> That is, assuming you want to >> - install KVM from F10 branch >> - run all tests >> - install KVM from F11 branch >> - run all tests >> - install KVM from devel branch >> - run all tests >> >> If you meant something different please correct me. >> > > Yeah. I am not really used to the configuration file format, but that > was the idea. :) > I've reworked the patch to use test.bindir and force it to run from kvm_runtest_2. I also doubled checked that both parameters and variables defined in the params object are available to the script. The updated past should go out shortly. Right now, nothing from the kvm_tests.cfg file is available. I don't see an obvious way to do that, but we can put that into a separate patch if needed later. Mike