July 2009 - Devel - libvirt List Archives

[libvirt] Switch from CVS to GIT is done
by Daniel Veillard 07 Jul '09

07 Jul '09

Thanks to Jim Meyering we now have a new git repository, I deprecated the CVS repository, it's read only, you should still be able to keep it around to make patches for a few weeks if needed. The new repo is at: http://libvirt.org/git/?p=libvirt.git;a=summary Unfortunately Jim had to clean up the conversion from CVS that was used previously at git://git.et.redhat.com/libvirt.git so the two are not compatible and the best is to clone a new tree using git clone git://libvirt.org/libvirt.git and then copying over your set of patches with "git format-patch" in the old dir and "git am" to paste them into the new cloned-dir (Jim can certainly provide more details if needed). Let us know if you see something weird in the new tree or on the web site, I have updated the download informations at http://libvirt.org/downloads.html A few things may change but I guess the switch is done at this point, Many thanks to Jim for maintaining the old tree and the migration ! Daniel -- Daniel Veillard | libxml Gnome XML XSLT toolkit http://xmlsoft.org/ daniel(a)veillard.com | Rpmfind RPM search engine http://rpmfind.net/ http://veillard.com/ | virtualization library http://libvirt.org/

6 16

[libvirt] [PATCH 1/2] Added new filesystem glusterfs
by Harshavardhana 07 Jul '09

07 Jul '09

Added new enum for glusterfs, a new netfs based filesystem --- src/storage_conf.c | 2 +- src/storage_conf.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/storage_conf.c b/src/storage_conf.c index 5f724dc..16ed906 100644 --- a/src/storage_conf.c +++ b/src/storage_conf.c @@ -68,7 +68,7 @@ VIR_ENUM_IMPL(virStoragePoolFormatFileSystem, VIR_ENUM_IMPL(virStoragePoolFormatFileSystemNet, VIR_STORAGE_POOL_NETFS_LAST, - "auto", "nfs") + "auto", "nfs", "glusterfs") VIR_ENUM_IMPL(virStoragePoolFormatDisk, VIR_STORAGE_POOL_DISK_LAST, diff --git a/src/storage_conf.h b/src/storage_conf.h index 8a4fed2..27dc4a7 100644 --- a/src/storage_conf.h +++ b/src/storage_conf.h @@ -151,7 +151,6 @@ struct _virStoragePoolSourceHost { int protocol; }; - /* * Available extents on the underlying storage */ @@ -366,6 +365,7 @@ VIR_ENUM_DECL(virStoragePoolFormatFileSystem) enum virStoragePoolFormatFileSystemNet { VIR_STORAGE_POOL_NETFS_AUTO = 0, VIR_STORAGE_POOL_NETFS_NFS, + VIR_STORAGE_POOL_NETFS_GLUSTERFS, VIR_STORAGE_POOL_NETFS_LAST, }; VIR_ENUM_DECL(virStoragePoolFormatFileSystemNet) -- 1.6.0.6

1 0

[libvirt] make distcheck failure
by Guido Günther 07 Jul '09

07 Jul '09

Hi, in current git make distcheck currently fails on me with: make[5]: Entering directory `/var/scratch/debian/libvirt/cvs/libvirt/libvirt-0.6.5/_build/gnulib/lib' /bin/sh ../../../mylibtool --tag=CC --mode=compile gcc -DHAVE_CONFIG_H -I. -I../../../gnulib/lib -I../.. -I../../intl -g -O2 -MT c-ctype.lo -MD -MP -MF .deps/c-ctype.Tpo -c -o c-ctype.lo ../../../gnulib/lib/c-ctype.c (CC) -o c-ctype.lo ../../../gnulib/lib/c-ctype.c ../../../mylibtool: line 58: /var/scratch/debian/libvirt/cvs/libvirt/libvirt-0.6.5/_build/gnulib/lib/../../../libtool: No such file or directory ../../../mylibtool: line 58: exec: /var/scratch/debian/libvirt/cvs/libvirt/libvirt-0.6.5/_build/gnulib/lib/../../../libtool: cannot execute: No such file or directory make[5]: *** [c-ctype.lo] Error 126 Any ideas? Cheers, -- Guido

2 1

[libvirt] Save and restore with KVM!
by Erkan Unal 07 Jul '09

07 Jul '09

Hi, First of all, I want to ask: is there a libvirt user mailing list? I generally see people exchanging ideas on development of libvirt in this mailing list. So feel free to direct me to the right place if this is not the main mailing list for these kind of questions. I am having problems with save and restore commands in virsh when I try to save and restore KVM virtual machines. After I initiate restore operation, instead of starting where I left during the save operation, it is restarting the VM. Therefore, I need to ask what should I check for these operations to execute correctly? What kind of vm disk types: qcow or vmdk? Which versions of qemu, kvm and libvirt combinations? currently I have libvirt 0.6.5, qemu 0.9.1, kvm-82 on fedora core 8 with kernel version of 2.6.26.8. Thanks, Erkan Unal

2 1

[libvirt] [PATCH] createLinux requires full-access connection
by Dan Kenigsberg 06 Jul '09

06 Jul '09

--- docs/examples/python/domstart.py | 2 +- python/tests/create.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/docs/examples/python/domstart.py b/docs/examples/python/domstart.py index 64434c4..52fb79e 100755 --- a/docs/examples/python/domstart.py +++ b/docs/examples/python/domstart.py @@ -32,7 +32,7 @@ if len(sys.argv) != 2: (name, xmldesc) = read_domain(sys.argv[1]) -conn = libvirt.openReadOnly(None) +conn = libvirt.open(None) if conn == None: print 'Failed to open connection to the hypervisor' sys.exit(1) diff --git a/python/tests/create.py b/python/tests/create.py index 0da89a9..d2c434e 100755 --- a/python/tests/create.py +++ b/python/tests/create.py @@ -52,7 +52,7 @@ else: initrdU = "<initrd>" + initrdU + "</initrd>" -conn = libvirt.openReadOnly(None) +conn = libvirt.open(None) if conn == None: print 'Failed to open connection to the hypervisor' sys.exit(1) -- 1.6.2.5

3 2

[libvirt] [PATCH] Do a correct format mapping of partitions 2
by Henrik Persson 06 Jul '09

06 Jul '09

I make a new try from a different mail client now. /Henrik

4 5

Re: [libvirt] Patch to include arm variants in virsh capabilities
by C.J. Adams-Collier 05 Jul '09

05 Jul '09

> On Wed, Jul 01, 2009 at 11:02:16AM -0700, C.J. Adams-Collier wrote: > > Hey folks, > > > > I got an android and wanted to set up an armel debian vm to test out > > some code. I had to launch it directly with qemu-system-arm, though, > > since virsh didn't have an arm option to select. > > > > This is a patch against the debian version of the source after applying > > debian-specific changes. If you'd like a patch against mainline, I'd be > > willing to do that. > > > > http://karma.colliertech.org/~cjac/tmp/add-arm.patch > > Well, this list is really about upstream patches not distro specific > ones, I just verified that it applies cleanly against CVS. > and I could not access said patch [1]. That is weird. I hope it was just a fluke! Attached. > Send a patch against the > latest version or CVS/git as an mail attachemnt and we will try to add > it. Alrighty. I see that the mips machines are separated into 'mips' and 'mipsel'. I don't know anything about the endian-ness of the various machines (aside from versatilepb, which is little), so I just left them all in the generic qemu_arch_info.arch="arm" table entry. > Daniel Cheers, C.J. > [1] the server seems down I got an HTTP request timeout. hurm... I'm getting hits at least hourly from a site revproxied by the same host ( http://wp.colliertech.org/cj/ ) - perhaps it was a coincidental outage (*shakes fist at comcast*)? > -- > Daniel Veillard | libxml Gnome XML XSLT toolkit http://xmlsoft.org/ > daniel <at> veillard.com | Rpmfind RPM search engine http://rpmfind.net/ > http://veillard.com/ | virtualization library http://libvirt.org/

3 5

[libvirt] Release of libvirt-0.6.5
by Daniel Veillard 03 Jul '09

03 Jul '09

As scheduled, the new version is available on the server: ftp://libvirt.org/libvirt/ This is mostly a bug fix release, though it includes some serious improvements for storage/NPIV and on the OpenNebula driver: * New features - create storage columes on disk backend (Henrik Persson) - drop of capabilities based on libcap-ng when possible (Daniel Berrange) * Portability - fix build on non-Linux targets (Daniel Berrange) * Documentation - typo and english fixes (Runa Bhattacharjee and Garry Dolley) - Docs on extending APIs (Dave Allan) - cleanup of debug and logging documentation (Amy Griffis) - add HACKING doc to the website (Daniel Berrange) - documentation for OpenNebula driver (Abel Miguez Rodriguez) * Bug fixes - forbid autostart on transcient networks - xen device removal crash (Daniel Berrange) - re-detection of transient VMs after libvirtd restart(Daniel Berrange) - bug in virFindFileInPath (Daniel Berrange) - handle new availheap sysctl in Xen (Daniel Berrange) - allow USB hostdev product 0 (Cole Robinson) - cleanup when creating a storage pool fails (Henrik Persson) - domain id fix on redefinition in test driver (Cole Robinson) - fix raw storage allocation (Cole Robinson) - memory reporting for inactive qemu drivers (Cole Robinson) - segfault if storage pool has no type attribute (Cole Robinson) - OpenNebula compilation issues (Javier Fontan) - dominfo command without security driver (Daniel Berrange) - domain state problems after migration or destroy (Federico Simoncelli) - leak in node device parsing (Dave Allan) - storage pool definitions reading at startup (Cole Robinson) - bogus WWN in NPIV support (David Allan) - avoid a segfault with recent Xen (Sascha) - cope with libnuma failures on weird topologies (Dan Berrange) - crash in QEMU driver with bad capabilities data (Dan Berrange) - trying to re-create a pool should not destroy it (Dave Allan) - endless loop in node device XML dump (Cole Robinson) - Re-label shared and readonly images (Dan Berrange) * Improvements - create and destroy NPIV support (David Allan) - networking in UML driver (Daniel Berrange) - HAL driver restart thread safety (Daniel Berrange) - capabilities and nodeinfo APIs for LXC (Daniel Berrange) - iNUMA API for VBox (Daniel Berrange) - dynamically search and use kvm-img qemu-img or qcow-create (Doug Goldstein) - fix qemu and kvm version parsing (Mark McLoughlin) - serial number for HAL storage (Dave Allan) - improve error reporting for virConnectOpen URIs (Daniel Berrange) - include OS driver name in device XML (Daniel Berrange) - fix qemu command flags fetching (Cole Robinson) - check that qemu support -drive format= (Cole Robinson) - improve emulator detection (Cole Robinson) - changes to config parser to accomodate VMX syntax (Matthias Bolte) - update network schemas and driver for missing elements (Satoru SATOH) - avoid changing file context if not needed (Tim Waugh) - skip labelling if no src path (Cole Robinson) - add arm emulation if qemu-system-arm is present (C.J. Adams-Collier) * Cleanups - daemon check logging env variables (Daniel Berrange) - User Mode Linux start and stop cleanups (Daniel Berrange) - share the NUMA api implementations (Daniel Berrange) - storage module dependancies (Dave Allan) - refactor storage XML parsing (Cole Robinson) - big cleanup of logging code (Amy Griffis) - superfluous % on format (Matthias Bolte) - cleanups and updates on OpenNebula driver (Daniel Berrange and Abel Miguez Rodriguez) thanks everybody for report, patches, documentation, etc ! Daniel -- Daniel Veillard | libxml Gnome XML XSLT toolkit http://xmlsoft.org/ daniel(a)veillard.com | Rpmfind RPM search engine http://rpmfind.net/ http://veillard.com/ | virtualization library http://libvirt.org/

1 0

[libvirt] gitignore & cscope
by Dave Allan 03 Jul '09

03 Jul '09

Could we add cscope's files to .gitignore? Dave diff --git a/.gitignore b/.gitignore index 1b36694..ca73a98 100644 --- a/.gitignore +++ b/.gitignore @@ -19,6 +19,8 @@ config.status config.sub configure coverage +cscope.files +cscope.out libtool libvirt-*.tar.gz libvirt.pc

4 3

[libvirt] [PATCH 1/2] Re-label shared and readonly images
by Mark McLoughlin 03 Jul '09

03 Jul '09

From: Daniel P. Berrange <berrange(a)redhat.com> This patch was posted ages ago here: https://bugzilla.redhat.com/493692 But was never posted upstream AFAICT. Signed-off-by: Mark McLoughlin <markmc(a)redhat.com> --- src/security_selinux.c | 27 +++++++++++++++++---------- 1 files changed, 17 insertions(+), 10 deletions(-) diff --git a/src/security_selinux.c b/src/security_selinux.c index 4fb7c86..87073d2 100644 --- a/src/security_selinux.c +++ b/src/security_selinux.c @@ -24,11 +24,12 @@ #include "virterror_internal.h" #include "util.h" #include "memory.h" - +#include "logging.h" #define VIR_FROM_THIS VIR_FROM_SECURITY static char default_domain_context[1024]; +static char default_content_context[1024]; static char default_image_context[1024]; #define SECURITY_SELINUX_VOID_DOI "0" #define SECURITY_SELINUX_NAME "selinux" @@ -148,8 +149,13 @@ SELinuxInitialize(virConnectPtr conn) close(fd); ptr = strchrnul(default_image_context, '\n'); - *ptr = '\0'; - + if (*ptr == '\n') { + *ptr = '\0'; + strcpy(default_content_context, ptr+1); + ptr = strchrnul(default_content_context, '\n'); + if (*ptr == '\n') + *ptr = '\0'; + } return 0; } @@ -313,6 +319,8 @@ SELinuxSetFilecon(virConnectPtr conn, const char *path, char *tcon) { char ebuf[1024]; + VIR_INFO("Setting SELinux context on '%s' to '%s'", path, tcon); + if(setfilecon(path, tcon) < 0) { virSecurityReportError(conn, VIR_ERR_ERROR, _("%s: unable to set security context " @@ -337,9 +345,6 @@ SELinuxRestoreSecurityImageLabel(virConnectPtr conn, char *newpath = NULL; const char *path = disk->src; - if (disk->readonly || disk->shared) - return 0; - if ((err = virFileResolveLink(path, &newpath)) < 0) { virReportSystemError(conn, err, _("cannot resolve symlink %s"), path); @@ -366,8 +371,13 @@ SELinuxSetSecurityImageLabel(virConnectPtr conn, { const virSecurityLabelDefPtr secdef = &vm->def->seclabel; - if (secdef->imagelabel) + if (disk->shared) { + return SELinuxSetFilecon(conn, disk->src, default_image_context); + } else if (disk->readonly) { + return SELinuxSetFilecon(conn, disk->src, default_content_context); + } else if (secdef->imagelabel) { return SELinuxSetFilecon(conn, disk->src, secdef->imagelabel); + } return 0; } @@ -441,9 +451,6 @@ SELinuxSetSecurityLabel(virConnectPtr conn, if (secdef->imagelabel) { for (i = 0 ; i < vm->def->ndisks ; i++) { - if (vm->def->disks[i]->readonly || - vm->def->disks[i]->shared) continue; - if (SELinuxSetSecurityImageLabel(conn, vm, vm->def->disks[i]) < 0) return -1; } -- 1.6.2.5

2 5