December 2012 - Devel - libvirt List Archives

[libvirt] [PATCH] build: Fix AUTHORS generation
by Jiri Denemark 17 Dec '12

17 Dec '12

Using s/#authorslist#/$$out/ makes perl eat @domain part of all email addresses from $out since it tries to interpret them as array variables. I'm not sure if we can escape those in s/// but I know we can use print: s/#authorslist#// and print '$$out' to tell perl not to even inside $out. This patch also fixes gen-AUTHORS so that it works in VPATH. --- Makefile.am | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Makefile.am b/Makefile.am index 9556b42..05301cd 100644 --- a/Makefile.am +++ b/Makefile.am @@ -95,9 +95,9 @@ gen-ChangeLog: .PHONY: gen-AUTHORS gen-AUTHORS: - $(AM_V_GEN)if test -d .git; then \ - out="`git log --pretty=format:'%aN <%aE>' | sort -u`" && \ - cat $(srcdir)/AUTHORS.in | perl -p -e "s/#authorslist#/$$out/" > \ + $(AM_V_GEN)if test -d $(srcdir)/.git; then \ + out="`cd $(srcdir) && git log --pretty=format:'%aN <%aE>' | sort -u`" && \ + cat $(srcdir)/AUTHORS.in | perl -p -e "s/#authorslist#// and print '$$out'" > \ $(distdir)/AUTHORS-tmp && \ mv -f $(distdir)/AUTHORS-tmp $(distdir)/AUTHORS ; \ fi -- 1.8.0.2

2 2

[libvirt] [PATCH] apparmor: Fix build
by Jiri Denemark 17 Dec '12

17 Dec '12

--- Pushed as a trivial build-breaker. src/security/virt-aa-helper.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/security/virt-aa-helper.c b/src/security/virt-aa-helper.c index 888c416..86fd5c4 100644 --- a/src/security/virt-aa-helper.c +++ b/src/security/virt-aa-helper.c @@ -1022,7 +1022,8 @@ get_files(vahControl * ctl) switch (dev->source.subsys.type) { case VIR_DOMAIN_HOSTDEV_SUBSYS_TYPE_USB: { usbDevice *usb = usbGetDevice(dev->source.subsys.u.usb.bus, - dev->source.subsys.u.usb.device); + dev->source.subsys.u.usb.device, + NULL); if (usb == NULL) continue; -- 1.8.0.2

1 0

[libvirt] [PATCH] .gitignore: add tests/sysinfotest
by Cole Robinson 17 Dec '12

17 Dec '12

--- Pushed since it's trivial .gitignore | 1 + 1 file changed, 1 insertion(+) diff --git a/.gitignore b/.gitignore index 32ae8a7..0557d59 100644 --- a/.gitignore +++ b/.gitignore @@ -164,6 +164,7 @@ /tests/ssh /tests/statstest /tests/storagebackendsheepdogtest +/tests/sysinfotest /tests/utiltest /tests/viratomictest /tests/virauthconfigtest -- 1.8.0.2

1 0

[libvirt] [PATCH] daemon: Preface polkit error output with 'polkit:'
by Cole Robinson 17 Dec '12

17 Dec '12

There's been a few bugs about an expected error from polkit: https://bugzilla.redhat.com/show_bug.cgi?id=873799 https://bugzilla.redhat.com/show_bug.cgi?id=872166 The error is: Authorization requires authentication but no agent is available. The error means that polkit needs a password, but there is no polkit agent registered in your session. Polkit agents are the bit of UI that pop up and actually ask for your password. Preface the error with the string 'polkit:' so folks can hopefully make more sense of it. --- daemon/remote.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/daemon/remote.c b/daemon/remote.c index 41b8ea8..f85dae0 100644 --- a/daemon/remote.c +++ b/daemon/remote.c @@ -2887,9 +2887,10 @@ error: if (authdismissed) { virReportError(VIR_ERR_AUTH_CANCELLED, "%s", _("authentication cancelled by user")); + } else if (pkout && *pkout) { + virReportError(VIR_ERR_AUTH_FAILED, "polkit: %s", pkout); } else { - virReportError(VIR_ERR_AUTH_FAILED, "%s", - pkout && *pkout ? pkout : _("authentication failed")); + virReportError(VIR_ERR_AUTH_FAILED, "%s", _("authentication failed")); } VIR_FREE(pkout); -- 1.8.0.2

2 2

[libvirt] Network XML Description
by Fawad Khaliq 17 Dec '12

17 Dec '12

Hi all, The XML description of a network does not contain information of the Domains (VMs) plugged into a network. For example, there is a network named 'default' and there are two domains named: domainA and domainB. The Network XML description of 'default' network cannot tell what domains are plugged into it (in this scenario: domainA and domainB). What reasons could be behind this Network XML format convention? Thanks, Fawad

3 3

[libvirt] [PATCH 00/23] Improve device support for LXC
by Daniel P. Berrange 17 Dec '12

17 Dec '12

This series makes some major improvements to the device support for LXC - USB, block and char device passthrough - Hotplug/unplug support for disk, nic, usb, block and char devices

4 58

[libvirt] [PATCH] build: libvirt-guests files misplaced in specfile
by Viktor Mihajlovski 17 Dec '12

17 Dec '12

In a non-systemd environment the post and preun scripts of libvirt-client fail, since the required files are in libvirt-daemon. Moved them to client. One curious thing: %{_unitdir}/libvirt-guests.service was contained in both libvirt-client and libvirt-daemon. I removed the extra copy from daemon. Note: this should be checked by someone building RPMs for a systemd machine. I don't have one at the moment. Signed-off-by: Viktor Mihajlovski <mihajlov(a)linux.vnet.ibm.com> --- libvirt.spec.in | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libvirt.spec.in b/libvirt.spec.in index 7457a48..b826a8f 100644 --- a/libvirt.spec.in +++ b/libvirt.spec.in @@ -1662,15 +1662,12 @@ fi %dir %attr(0700, root, root) %{_sysconfdir}/libvirt/nwfilter/ -%attr(0755, root, root) %{_libexecdir}/libvirt-guests.sh %if %{with_systemd} %{_unitdir}/libvirtd.service -%{_unitdir}/libvirt-guests.service %{_unitdir}/virtlockd.service %{_unitdir}/virtlockd.socket %else %{_sysconfdir}/rc.d/init.d/libvirtd -%{_sysconfdir}/rc.d/init.d/libvirt-guests %{_sysconfdir}/rc.d/init.d/virtlockd %endif %doc daemon/libvirtd.upstart @@ -1948,8 +1945,11 @@ rm -f $RPM_BUILD_ROOT%{_sysconfdir}/sysctl.d/libvirtd %if %{with_systemd} %{_unitdir}/libvirt-guests.service +%else +%{_sysconfdir}/rc.d/init.d/libvirt-guests %endif %config(noreplace) %{_sysconfdir}/sysconfig/libvirt-guests +%attr(0755, root, root) %{_libexecdir}/libvirt-guests.sh %dir %attr(0755, root, root) %{_localstatedir}/lib/libvirt/ %if %{with_sasl} -- 1.7.9.5

1 0

[libvirt] No VirtualBox Driver
by Arslan Waqar 17 Dec '12

17 Dec '12

Hi all, I am new to the group and hence there might be a chance I might be asking a simple question... The thing is I compiled and installed libvirt from git. Then in my terminal window I started virsh. It started. Then since I had Virtual Box installed on my machine I ran "connect vbox:///session". Once I did this I got two errors which said: error: Failed to connect to the hypervisor error: internal error unable to initialize VirtualBox driver API I am unable to resolve this issue...Any input is welcome! Thanks! -- Regards, Arslan Waqar.

2 9

[libvirt] [PATCH 0/4] Cleanup and fix parsing of vendor_id in the XML cpu definition
by Peter Krempa 17 Dec '12

17 Dec '12

This patchset contains Ken ICHIKAWA's patch to fix parsing of vendor_id (with tweaked commit message) and my cleanups to pre-existing problems in the code which I found while reviewing Ken's patch. Ken ICHIKAWA (1): conf: cpu: Fix parsing of vendor_id Peter Krempa (3): conf: cpu: Fix memory leak when specifying cpu vendor_id manually conf: cpu: Refactor parsing of vendor_id and fallback attributes conf: cpu: Break some long lines src/conf/cpu_conf.c | 82 +++++++++++++++++++++++++---------------------------- 1 file changed, 38 insertions(+), 44 deletions(-) -- 1.8.0.2

2 8

[libvirt] ANNOUNCE: New libosinfo project mailing list
by Daniel P. Berrange 17 Dec '12

17 Dec '12

Historically the libosinfo project has used virt-tools-list(a)redhat.com for all its discussions. Since the project has grown and matured, it is time for a dedicated mailing list. If you are interested/involved with libosinfo, please subscribe to the new list: https://www.redhat.com/mailman/listinfo/libosinfo The list only accepts postings from members. Once subscribed you can post to libosinfo(a)redhat.com Regards, Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|

1 0