From mboxrd@z Thu Jan  1 00:00:00 1970
From: Lucas Meneghel Rodrigues <lmr@redhat.com>
Subject: Re: [PATCH 18/18] KVM test: Add subtest of testing
 offload by ethtool
Date: Wed, 06 Oct 2010 13:57:52 -0300
Message-ID: <1286384273.30803.0.camel@freedom>
References: <1285627444-2732-1-git-send-email-lmr@redhat.com>
	<1285627444-2732-19-git-send-email-lmr@redhat.com>
	<20101006142646.09c45a03@skywalker> <20101006155545.GF23351@us.ibm.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Cc: kvm@vger.kernel.org, mst@redhat.com, autotest@test.kernel.org
To: Ryan Harper <ryanh@us.ibm.com>
Return-path: <autotest-bounces@test.kernel.org>
In-Reply-To: <20101006155545.GF23351@us.ibm.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 Wed, 2010-10-06 at 10:55 -0500, Ryan Harper wrote:
> * pradeep <psuriset@linux.vnet.ibm.com> [2010-10-06 03:57]:
> > On Mon, 27 Sep 2010 18:44:04 -0400
> > Lucas Meneghel Rodrigues <lmr@redhat.com> wrote:
> > 
> > 
> > > +
> > > +    vm = kvm_test_utils.get_living_vm(env, params.get("main_vm"))
> > > +    session = kvm_test_utils.wait_for_login(vm,
> > > +                  timeout=int(params.get("login_timeout", 360)))
> > > +    # Let's just error the test if we identify that there's no
> > > ethtool installed
> > > +    if session.get_command_status("ethtool -h"):
> > > +        raise error.TestError("Command ethtool not installed on
> > > guest")
> > > +    session2 = kvm_test_utils.wait_for_login(vm,
> > > +                  timeout=int(params.get("login_timeout", 360)))
> > > +    mtu = 1514
> > > +    feature_status = {}
> > > +    filename = "/tmp/ethtool.dd"
> > > +    guest_ip = vm.get_address()
> > > +    ethname = kvm_test_utils.get_linux_ifname(session,
> > > vm.get_mac_address(0))
> > > +    supported_features = params.get("supported_features").split()
> > 
> > I guess split this expects input.
> > 
> > 23:48:03 ERROR| Test failed: AttributeError: 'NoneType' object has no
> > attribute 'split'
> 
> That'll need an update to the tests_base.cfg file to ensure the test
> type has that config value set.
> 
> Did the patchset miss updating tests_base.cfg.sample with this one ?

I think pradeep forgot to update tests_base.cfg indeed. It's working
fine for me.