From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lucas Meneghel Rodrigues Subject: Re: [KVM-AUTOTEST PATCH] bugfix for changing kvm_log to logging Date: Wed, 01 Jul 2009 00:24:23 -0300 Message-ID: <1246418664.9596.138.camel@freedom> References: <1245187799-7757-1-git-send-email-lmr@redhat.com> <4A439DB1.9040701@redhat.com> Mime-Version: 1.0 Content-Type: text/plain Content-Transfer-Encoding: 7bit Cc: kvm@vger.kernel.org, autotest@test.kernel.org, Michael Goldish To: Yolkfull Chow Return-path: Received: from mx2.redhat.com ([66.187.237.31]:48504 "EHLO mx2.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753377AbZGADYY (ORCPT ); Tue, 30 Jun 2009 23:24:24 -0400 In-Reply-To: <4A439DB1.9040701@redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: Applied, with minimum changes On Thu, 2009-06-25 at 23:54 +0800, Yolkfull Chow wrote: > diff --git a/client/tests/kvm/kvm.py b/client/tests/kvm/kvm.py > index 4c7bae4..26ae46f 100644 > --- a/client/tests/kvm/kvm.py > +++ b/client/tests/kvm/kvm.py > @@ -73,7 +73,7 @@ class kvm(test.test): > resource.setrlimit(resource.RLIMIT_CORE, (-1, -1)) > > # Set the logging prefix > - #kvm_log.set_prefix(params.get("shortname")) > + #logging.set_prefix(params.get("shortname")) This line was commented, the logging module does not have a set_prefix function. Therefore, the line was just removed. > # Report the parameters we've received and write them as keyvals > logging.debug("Test parameters:") > diff --git a/client/tests/kvm/kvm_utils.py b/client/tests/kvm/kvm_utils.py > index 4bc8dc7..a37d322 100644 > --- a/client/tests/kvm/kvm_utils.py > +++ b/client/tests/kvm/kvm_utils.py > @@ -670,7 +670,7 @@ def remote_login(command, password, prompt, > linesep="\n", timeout=10): > sub.close() > return None > elif match == 4: # "Connection refused" > - kvm_log.debug("Got 'Connection refused'") > + logging.debug("Got 'Connection refused'") > sub.close() > return None > elif match == 5: # prompt