From mboxrd@z Thu Jan 1 00:00:00 1970 From: Michael Goldish Subject: Re: [PATCH 1/3] KVM Test: Introduce a helper class to run a test in the background Date: Wed, 01 Dec 2010 10:06:37 +0200 Message-ID: <4CF6020D.3020002@redhat.com> References: <1290812474-21377-1-git-send-email-lmr@redhat.com> <1290813392.2578.60.camel@freedom> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Cc: autotest@test.kernel.org, kvm@vger.kernel.org, Jason Wang , mst@redhat.com To: Lucas Meneghel Rodrigues Return-path: Received: from mx1.redhat.com ([209.132.183.28]:52475 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752857Ab0LAIEx (ORCPT ); Wed, 1 Dec 2010 03:04:53 -0500 In-Reply-To: <1290813392.2578.60.camel@freedom> Sender: kvm-owner@vger.kernel.org List-ID: On 11/27/2010 01:16 AM, Lucas Meneghel Rodrigues wrote: > On Fri, 2010-11-26 at 21:01 -0200, Lucas Meneghel Rodrigues wrote: >> Sometimes, we need to run a test when doing operations on a running VM >> (such as migrate the vm during its rebooting ). So this patch introduces >> a simple warpper BackgroundTest to run a specified test in the background >> through a dedicated thread, it also records the exception raised in the >> thead and raise it when master call join(). > > Since we've spent some time looking into this and noone came up with a > better solution, I'm applying Jason's series. In order to run the tests > in parallel, on your test set you can put either: > > only migrate.with_file_transfer > or > only migrate.with_reboot > > Provided, of course, that you already had a vm installed with autotest. > If you don't you probably want to include the unattended_install.cdrom > test before. > > Cheers! > > Lucas I was planning to post a modified version of the wrapper. I guess I'll rebase it against this version. Also, migrate.with_reboot doesn't work, or at least shouldn't work, because reboot() receives a VM object that is later destroyed.