<br><tt><font size=2>libvirt-cim-bounces@redhat.com wrote on 2009-02-12
03:13:46:<br>
<br>
> <br>
> > diff -r 1bb347c22e6e -r dd59a8d6a123 suites/libvirt-<br>
> cim/lib/XenKvmLib/const.py<br>
> > --- a/suites/libvirt-cim/lib/XenKvmLib/const.py   Mon Feb
09 21:<br>
> 47:35 2009 -0800<br>
> > +++ b/suites/libvirt-cim/lib/XenKvmLib/const.py   Wed Feb
11 00:<br>
> 10:44 2009 -0800<br>
> > @@ -140,8 +140,8 @@<br>
> >                  
   destroy_and_undefine_all(options.ip, options.virt)<br>
> >                  
   rc = f()<br>
> >                  except
Exception, e:<br>
> > -                  
 logger.error('%s : %s' % (e.__class__.__name__, e))<br>
> > -                  
 logger.error("%s" % traceback.print_exc())<br>
> > +                  
 logger.error('%s : %s', e.__class__.__name__, e)<br>
> > +                  
 logger.error("%s", traceback.print_exc())<br>
> >                  
   rc = FAIL<br>
> >                  return
rc<br>
> >              setattr(do_try,
'options', options)<br>
> > diff -r 1bb347c22e6e -r dd59a8d6a123 suites/libvirt-<br>
> cim/lib/XenKvmLib/test_xml.py<br>
> > --- a/suites/libvirt-cim/lib/XenKvmLib/test_xml.py   Mon
Feb 09 <br>
> 21:47:35 2009 -0800<br>
> > +++ b/suites/libvirt-cim/lib/XenKvmLib/test_xml.py   Wed
Feb 11 <br>
> 00:10:44 2009 -0800<br>
> > @@ -48,7 +48,8 @@<br>
> >  def testxml(test_dom="domU1", mem = 128, vcpus
= 1, mac = default_mac, \<br>
> >   disk_file_path = disk_path, disk = "xvda"):<br>
> >      if not (os.path.exists(kernel_path) and os.path.exists(init_path))
:<br>
> > -        logger.error("ERROR: Either
the kernel image or the <br>
> init_path does not exist")<br>
> > +        logger.error("ERROR: Either
the kernel image or the \<br>
> > +                  
  init_path does not exist")<br>
> <br>
> Can you do the following instead (otherwise extra whitespace will
be <br>
> displayed when the line is printed):<br>
> <br>
> logger.error("ERROR: Either the kernel image or the"<br>
>               "init_path does
not exist")<br>
> <br>
> >          sys.exit(SKIP)<br>
> >      test_xml = """<br>
> >      <domain type='xen' id='23'><br>
> > @@ -76,8 +77,8 @@<br>
> >          </disk><br>
> >        </devices><br>
> >      </domain><br>
> > -    """ % ( test_dom, set_uuid(), kernel_path,
init_path, <br>
> mem*1024, vcpus, mac, \<br>
> > -                  
                     
       disk_file_path, disk )<br>
> > +    """ % ( test_dom, set_uuid(), kernel_path,
init_path, <br>
> mem*1024, vcpus, \<br>
> <br>
> No need for the \ here.<br>
> <br>
> > +            mac, disk_file_path,
disk )<br>
> >      return test_xml<br>
> > <br>
> >  def testxml_bl(test_dom="domU1", mem = 128, vcpus
= 1, mac = default_mac, \<br>
> > @@ -88,7 +89,8 @@<br>
> >  specify either localhost or remote machine ip/name "
)<br>
> >          sys.exit(SKIP)<br>
> >      if not (os.path.exists(kernel_path) and os.path.exists(init_path))
:<br>
> > -        logger.error("ERROR: Either
the kernel image or the <br>
> init_path does not exist")<br>
> > +        logger.error("ERROR: Either
the kernel image or the \<br>
> > +                  
  init_path does not exist")<br>
> <br>
> <br>
> Can you do the following instead (otherwise extra whitespace will
be <br>
> displayed when the line is printed):<br>
> <br>
> logger.error("ERROR: Either the kernel image or the"<br>
>               "init_path does
not exist")<br>
> <br>
> >          sys.exit(SKIP)<br>
> >      test_xml = """<br>
> >      <domain type='xen' id='23'><br>
> > @@ -125,7 +127,8 @@<br>
> >              mac = default_mac,
disk_file_path = disk_path,<br>
> >              disk = "xvda",
server = ""): <br>
> >      if not (os.path.exists(kernel_path) and os.path.exists(init_path))
:<br>
> > -        logger.error("ERROR: Either
the kernel image or the <br>
> init_path does not exist")<br>
> > +        logger.error("ERROR: Either
the kernel image or the \<br>
> > +                  
  init_path does not exist")<br>
> <br>
> Same here.<br>
> <br>
> >          sys.exit(SKIP)<br>
> >      vir_network = net_list(server)<br>
> >      if len(vir_network) > 0 :<br>
> > @@ -140,7 +143,8 @@<br>
> >          net_xml, bridge = netxml(server,
bridgename, networkname)<br>
> >          ret = create_vnet(server, net_xml)<br>
> >          if not ret:<br>
> > -            logger.error("Failed
to create the Virtual Network '%<br>
> s'", networkname)<br>
> > +            logger.error("Failed
to create the Virtual Network '%s'", <br>
> > +                  
      networkname)<br>
> >              sys.exit(SKIP)<br>
> > <br>
> >      test_xml = """<br>
> > @@ -179,7 +183,8 @@<br>
> >      if bridgename in bridges:<br>
> >          bridge_name = bridgename +
str(random.randint(1, 100))<br>
> >          if bridge_name in bridges:<br>
> > -            logger.error("Need
to give different bridge name <br>
> since it alreay exists")<br>
> > +            logger.error("Need
to give different bridge name since \<br>
> > +                  
      it alreay exists")<br>
> <br>
> Same here.<br>
> </font></tt>
<br>
<br><tt><font size=2>   Fixed all of them on #2 patch.</font></tt>
<br><tt><font size=2>   Thanks!</font></tt>
<br><tt><font size=2><br>
> -- <br>
> Kaitlin Rupert<br>
> IBM Linux Technology Center<br>
> kaitlin@linux.vnet.ibm.com<br>
> <br>
> _______________________________________________<br>
> Libvirt-cim mailing list<br>
> Libvirt-cim@redhat.com<br>
> https://www.redhat.com/mailman/listinfo/libvirt-cim<br>
</font></tt>