From mboxrd@z Thu Jan 1 00:00:00 1970 From: Yolkfull Chow Subject: Re: [KVM-AUTOTEST PATCH] A test patch - Boot VMs until one of them becomes unresponsive Date: Wed, 10 Jun 2009 16:12:34 +0800 Message-ID: <4A2F6AF2.3090609@redhat.com> References: <1244433717-3391-1-git-send-email-lmr@redhat.com> <4A2E2052.7050304@redhat.com> <4A2E5950.900@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: kvm@vger.kernel.org To: Uri Lublin Return-path: Received: from mx2.redhat.com ([66.187.237.31]:34960 "EHLO mx2.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756165AbZFJIMp (ORCPT ); Wed, 10 Jun 2009 04:12:45 -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 n5A8Cm6w011633 for ; Wed, 10 Jun 2009 04:12:48 -0400 In-Reply-To: <4A2E5950.900@redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: On 06/09/2009 08:45 PM, Uri Lublin wrote: > On 06/09/2009 11:41 AM, Yolkfull Chow wrote: >> >> Hi, >> >> This test will boot VMs until one of them becomes unresponsive, and >> records the maximum number of VMs successfully started. >> >> > > Hello, > > Some more comments (in addition to previous comments by others) > 1. Do not just send monitor command "quit" but use vm.destroy > * This was mentioned by Michael, but in a different context. > 2. Do not destroy main_vm (or vm1). We may want to run other tests on it. > 3. You can use enumerate(vms) instead of looking for vm with index. > 4. It would be nice to close all ssh sessions too. OK, I will do modification according to your comments, thank you so much. :) Best Regards, Yolkfull > > Regards, > Uri. > -- > 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