From mboxrd@z Thu Jan  1 00:00:00 1970
From: Yolkfull Chow <yzhou@redhat.com>
Subject: Re: [AUTOTEST PATCH 2/2] KVM-test: Move two
 'remote_login' out of try block in kvm_vm.py
Date: Wed, 13 Jan 2010 10:24:12 +0800
Message-ID: <20100113022412.GC2333@aFu.nay.redhat.com>
References: <278734081.2246011262253625627.JavaMail.root@zmail05.collab.prod.int.phx2.redhat.com>
	<1360006704.2246041262253801462.JavaMail.root@zmail05.collab.prod.int.phx2.redhat.com>
	<6ac58f4f1001121437j240b86b4x748e4d0d0c1d5408@mail.gmail.com>
Reply-To: Yolkfull Chow <yzhou@redhat.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Cc: autotest@test.kernel.org, kvm@vger.kernel.org
To: Lucas Meneghel Rodrigues <lmr@redhat.com>
Return-path: <autotest-bounces@test.kernel.org>
Content-Disposition: inline
In-Reply-To: <6ac58f4f1001121437j240b86b4x748e4d0d0c1d5408@mail.gmail.com>
List-Unsubscribe: <http://test.kernel.org/cgi-bin/mailman/listinfo/autotest>,
	<mailto:autotest-request@test.kernel.org?subject=unsubscribe>
List-Archive: <http://test.kernel.org/pipermail/autotest>
List-Post: <mailto:autotest@test.kernel.org>
List-Help: <mailto:autotest-request@test.kernel.org?subject=help>
List-Subscribe: <http://test.kernel.org/cgi-bin/mailman/listinfo/autotest>,
	<mailto:autotest-request@test.kernel.org?subject=subscribe>
Sender: autotest-bounces@test.kernel.org
Errors-To: autotest-bounces@test.kernel.org
List-Id: kvm.vger.kernel.org

On Tue, Jan 12, 2010 at 08:37:25PM -0200, Lucas Meneghel Rodrigues wrote:
> On Thu, Dec 31, 2009 at 8:03 AM, Michael Goldish <mgoldish@redhat.com> wrote:
> >
> > ----- "Yolkfull Chow" <yzhou@redhat.com> wrote:
> >
> >> If vm.remote_login failed 'session.close()' can result in exception
> >> in finally clause. This patch fix the problem.
> 
> Hi Yolkfull, please implement Michael's comments and re-submit.

Sorry, hadn't seen the reply from Michael. I will post the updated patch soon.

Thanks, Lucas. :)

> 
> Cheers,
> 
> -- 
> Lucas
> --
> 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