<div dir="ltr">Hi,<br><div><br>I compiled libvirt form the latest git right now and I still see the same problem.<br><br></div><div>Thanks,<br>Bogdan<br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Fri, Mar 1, 2013 at 1:26 PM, Daniel P. Berrange <span dir="ltr"><<a href="mailto:berrange@redhat.com" target="_blank">berrange@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5">On Fri, Mar 01, 2013 at 11:12:28AM +0000, Bogdan Rădulescu wrote:<br>
> [Vi, 01 mar 2013 13:07:34 virt-install 16102] ERROR (cli:438) global name<br>
> 'virGetLastError' is not defined<br>
> [Vi, 01 mar 2013 13:07:34 virt-install 16102] DEBUG (cli:441)<br>
> Traceback (most recent call last):<br>
>   File "/usr/bin/virt-install", line 1015, in <module><br>
>     sys.exit(main())<br>
>   File "/usr/bin/virt-install", line 1000, in main<br>
>     guest = build_guest_instance(conn, options)<br>
>   File "/usr/bin/virt-install", line 529, in build_guest_instance<br>
>     options.disksize, options.sparse, need_storage)<br>
>   File "/usr/bin/virt-install", line 164, in get_disks<br>
>     get_disk(disklist[idx], sizelist[idx], sparse, guest, is_file_path)<br>
>   File "/usr/bin/virt-install", line 137, in get_disk<br>
>     dev, size = cli.parse_disk(guest, diskopts)<br>
>   File "/usr/lib64/python2.7/site-packages/virtinst/cli.py", line 1610, in<br>
> parse_disk<br>
>     format=fmt)<br>
>   File "/usr/lib64/python2.7/site-packages/virtinst/VirtualDisk.py", line<br>
> 659, in __init__<br>
>     self.vol_install)<br>
>   File "/usr/lib64/python2.7/site-packages/virtinst/VirtualDisk.py", line<br>
> 974, in __change_storage<br>
>     path)<br>
>   File "/usr/lib64/python2.7/site-packages/virtinst/VirtualDisk.py", line<br>
> 182, in _check_if_path_managed<br>
>     vol = lookup_vol_by_path()[0]<br>
>   File "/usr/lib64/python2.7/site-packages/virtinst/VirtualDisk.py", line<br>
> 164, in lookup_vol_by_path<br>
>     vol = conn.storageVolLookupByPath(path)<br>
>   File "/usr/lib64/python2.7/site-packages/libvirt.py", line 3387, in<br>
> storageVolLookupByPath<br>
>     if ret is None:raise libvirtError('virStorageVolLookupByPath() failed',<br>
> conn=self)<br>
>   File "/usr/lib64/python2.7/site-packages/libvirt.py", line 35, in __init__<br>
>     err = virGetLastError()<br>
> NameError: global name 'virGetLastError' is not<br>
> defined<br>
><br>
> ERROR    global name 'virGetLastError' is not defined<br>
<br>
</div></div>This is a bug in the 1.0.2 version of libvirt. I guess the slackware<br>
maintainers deployed that version without backporting the bug fix for<br>
it :-( File a bug with slackware asking them to apply the following<br>
two patches from upstream<br>
<br>
commit 25ea8e47e74def560bf89cd94dd54b75ca5ff4d6<br>
Author: Daniel P. Berrange <<a href="mailto:berrange@redhat.com">berrange@redhat.com</a>><br>
Date:   Tue Feb 5 12:55:09 2013 +0000<br>
<br>
    Fix missing error constants in libvirt python module<br>
<br>
    The previous change to the generator, changed too much - only<br>
    the functions are in 'virerror.c', the constants remained in<br>
    'virerror.h' which could not be renamed for API compat reasons.<br>
<br>
    Add a test case to sanity check the generated python bindings<br>
<br>
    Signed-off-by: Daniel P. Berrange <<a href="mailto:berrange@redhat.com">berrange@redhat.com</a>><br>
<br>
commit a6b8bae5a6a4752926eba409202ec061d81c6c8a<br>
Author: Serge Hallyn <<a href="mailto:serge.hallyn@canonical.com">serge.hallyn@canonical.com</a>><br>
Date:   Wed Jan 30 21:05:45 2013 -0600<br>
<br>
    complete virterror->virerror name change<br>
<br>
    Without these two string changes in generator.py, the<br>
    virGetLastError wrapper does not get created in<br>
    /usr/share/pyshared/libvirt.py.  Noticed when running<br>
    tests with virt-install.<br>
<br>
    Signed-off-by: Serge Hallyn <<a href="mailto:serge.hallyn@ubuntu.com">serge.hallyn@ubuntu.com</a>><br>
<br>
Regards,<br>
Daniel<br>
<span class="HOEnZb"><font color="#888888">--<br>
|: <a href="http://berrange.com" target="_blank">http://berrange.com</a>      -o-    <a href="http://www.flickr.com/photos/dberrange/" target="_blank">http://www.flickr.com/photos/dberrange/</a> :|<br>
|: <a href="http://libvirt.org" target="_blank">http://libvirt.org</a>              -o-             <a href="http://virt-manager.org" target="_blank">http://virt-manager.org</a> :|<br>
|: <a href="http://autobuild.org" target="_blank">http://autobuild.org</a>       -o-         <a href="http://search.cpan.org/~danberr/" target="_blank">http://search.cpan.org/~danberr/</a> :|<br>
|: <a href="http://entangle-photo.org" target="_blank">http://entangle-photo.org</a>       -o-       <a href="http://live.gnome.org/gtk-vnc" target="_blank">http://live.gnome.org/gtk-vnc</a> :|<br>
</font></span></blockquote></div><br></div>