On Thu, 2011-01-13 at 10:27 +0800, Feng Yang wrote:
> On 01/13/2011 12:15 AM, Lucas Meneghel Rodrigues wrote:
> > In order to avoid namespace collisions, turn the kvm test dir into
> > a module and, whenever importing full tests, use the complete
> > namespace. The full namespace might not be pretty, but it is
> > certainly safer, specially executing from autoserv.
> >
> > Signed-off-by: Lucas Meneghel Rodrigues<[email protected]>
> 
> When test this patch I got following error:
>   ImportError: No module named kvm.tests
> So we need add __init__.py in kvm foler.

The patch does that, if you look closely:

diff --git a/client/tests/kvm/__init__.py b/client/tests/kvm/__init__.py
new file mode 100644
index 0000000..e69de29

> > ---
> >   client/tests/kvm/tests/nic_bonding.py |    2 +-
> >   1 files changed, 1 insertions(+), 1 deletions(-)
> >   create mode 100644 client/tests/kvm/__init__.py
> >
> > diff --git a/client/tests/kvm/__init__.py b/client/tests/kvm/__init__.py
> > new file mode 100644
> > index 0000000..e69de29
> > diff --git a/client/tests/kvm/tests/nic_bonding.py 
> > b/client/tests/kvm/tests/nic_bonding.py
> > index 692978c..ca9d70a 100644
> > --- a/client/tests/kvm/tests/nic_bonding.py
> > +++ b/client/tests/kvm/tests/nic_bonding.py
> > @@ -1,6 +1,6 @@
> >   import logging, time, threading
> >   from autotest_lib.client.common_lib import error
> > -from tests import file_transfer
> > +from autotest_lib.client.tests.kvm.tests import file_transfer
> >   import kvm_test_utils, kvm_utils
> >
> >   def run_nic_bonding(test, params, env):
> 
> _______________________________________________
> Autotest mailing list
> [email protected]
> http://test.kernel.org/cgi-bin/mailman/listinfo/autotest


_______________________________________________
Autotest mailing list
[email protected]
http://test.kernel.org/cgi-bin/mailman/listinfo/autotest

Reply via email to