October 2009 - Devel - libvirt List Archives

[libvirt] Release of libvirt-0.7.2
by Daniel Veillard 14 Oct '09

14 Oct '09

This is an intermediate release to flush out one month worth of development and patches, as we have another release scheduled for the 30th October. It is available on ftp://libvirt.org/libvirt as usual and tagged in GIT The main new features are AppArmor security driver and the possibility to define new domain in the ESX driver. The code had a lot of cleanup, including a big tree refactoring and a fairly long list of bug fixes: Features: - sVirt AppArmor security driver (Jamie Strandboge) - Add public API definition for data stream handling (Daniel P. Berrange) - ESX add esxDomainDefineXML() (Matthias Bolte) - LXC: suspend/resume support (Ryota Ozaki) - Big code tree cleanup (Daniel P. Berrange) Documentation: - Documentation and examples for SVirt Apparmor driver (Jamie Strandboge) - Fix documentation and comment typos (Paolo Bonzini) - Fix up a few typos in the tree. (Chris Lalancette) - Fix a typo in virNetHasValidPciAddr() too (Mark McLoughlin) - Fix a typo in virDiskHasValidPciAddr() (Jiri Denemark) - Fix a number of small typos (Dan Kenigsberg) - add doc for graphic and video elements (Florian Vichot) - Fix up 'neccessary -> necessary' in a comment. (Chris Lalancette) - Fix up comments for domainXML{To,From}Native. (Chris Lalancette) - Simple fix of a comment in qemuStringToArgvEnv. (Chris Lalancette) - Add a README file to src/ explaining the directory structure (Daniel P. Berrange) - don't emit trailing blanks into generated and VC'd NEWS file (Jim Meyering) Portability: - Misc win32 build fixes (Daniel P. Berrange) - Don't require full daemon install for libvirt python bindings (Daniel P. Berrange) - Tweak specfile to fix RHEL6 rules & ESX/PHYP enablement (Daniel P. Berrange) - Bug Fixes: - network: Fix printing XML 'delay' attribute (Cole Robinson) - Fix virFileReadLimFD/virFileReadAll to handle EINTR (Daniel P. Berrange) - storage: Fix generating iscsi 'auth' xml (Cole Robinson) - Fix QEMU restore from file in raw format (Daniel P. Berrange) - Take domain type into account when looking up default machine (Mark McLoughlin) - Fix schema to allow missing machine type (Mark McLoughlin) - Fix stream abort upon I/O failure during migration (Daniel P. Berrange) - Create /var/log/libvirt/{lxc,uml} dirs (Mark McLoughlin) - nodedev: Add locking in nodeNumOfDevices (Cole Robinson) - test: Throw a proper error in GetBridgeName (Cole Robinson) - 526769 change logrotate config default to weekly (Daniel Veillard) - Fix emission of domain events messages (Daniel P. Berrange) - unbreak `make rpcgen' (Paolo Bonzini) - unbreak migration (Paolo Bonzini) - Fix USB device re-labelling (Mark McLoughlin) - Avoid a libvirtd crash on broken input 523418 (Daniel Veillard) - Re-label image file backing stores (Mark McLoughlin) - Fix memory leaks in libvirtd's message processing (Matthias Bolte) - Fix QEMU test suite with new VNC env variable (Daniel P. Berrange) - VBox vboxDomainDestroy forgot to wait for completion (Pritesh Kothari) - Vbox call OpenHardDisk with "" instead of NULL (Pritesh Kothari) - Avoid double free in errors in virsh (Jim Fehlig) - Fix crash in device hotplug cleanup code (Daniel P. Berrange) - Maintain value of ctxt->node in virInterfaceDefParseDhcp (Laine Stump) - Fix some XPath relative node resets (Daniel Veillard) - Fix unitialized variable in qemudDomainDetachHostPciDevice() (Charles Duffy) - ESX: Check if a datastore is accessible first (Matthias Bolte) - Fix handling of Xen(ner) detection (Daniel P. Berrange) - Fix xen driver refcounting. (Matthias Bolte) - prevent attempt to call cat -c during virDomainSave to raw (Charles Duffy) - Don't do virSetConnError when virDrvSupportsFeature is successful. (Chris Lalancette) - Fix a double-free in qemudRunLoop() (Chris Lalancette) - Fix leak in PCI hostdev hot-unplug (Mark McLoughlin) - Fix net/disk hot-unplug segfault (Mark McLoughlin) Improvements: - schema: Update storage pool schema. (Cole Robinson) - test: Activate interfaces specified through driver config file. (Cole Robinson) - Rewrite example domain events programm for python (Daniel P. Berrange) - Support a new peer-to-peer migration mode & public API (Daniel P. Berrange) - LXC add augeas support for config file (Amy Griffis) - LXC add driver config file lxc.conf (Amy Griffis) - LXC do not truncate container log files on restart (Amy Griffis) - LXC initialize logging configuration (Amy Griffis) - Add debug for envp[] in virExecWithHook() (Amy Griffis) - Add accessors for logging filters and outputs (Amy Griffis) - Add virFileAbsPath() utility (Amy Griffis) - LXC implement memory control APIs (Ryota Ozaki) - Add a domain argument to SVirt *RestoreImageLabel (Jamie Strandboge) - test: Support loading node device info from file/XML (Cole Robinson) - test: Implement node device driver. (Cole Robinson) - configure: Add explict --with-python option. (Cole Robinson) - Tunnelled migration. (Chris Lalancette) - Various monitor improvements for migration. (Chris Lalancette) - 523639 Allows a <description> tag for domains (Daniel Veillard) - Add src/util/storage_file.c to the POTFILES.in. (Chris Lalancette) - Add a qemu feature flag for unix socket migration. (Chris Lalancette) - Let remoteClientStream only do RX if requested. (Chris Lalancette) - Introduce virStorageFileMetadata structure (Mark McLoughlin) - Allow control over QEMU audio backend (Daniel P. Berrange) - Handle data streams in remote client (Daniel P. Berrange) - Handle outgoing data streams in libvirtd (Daniel P. Berrange) - Handle incoming data streams in libvirtd (Daniel P. Berrange) - Lots of cleanups and improvement on QEmu monitor code (Daniel P. Berrange) - ESX add esxVI_Occurence enum to for occurences (Matthias Bolte) - ESX add x86_64 detection based on the CPUID (Matthias Bolte) - ESX add tests for the VMX to/from domain XML mapping (Matthias Bolte) - ESX Add esxDomainXMLToNative() (Matthias Bolte) - ESX Set challenge for auth callback to hostname (Matthias Bolte) - ESX Add esxNodeGetFreeMemory() (Matthias Bolte) - network: add 'bootp' and 'tftp' config (Paolo Bonzini) - OpenVZ Fix a restriction about domain names (Yuji NISHIDA) - Make pki_check.sh into an installed & supported tool (Daniel P. Berrange) - ESX add support for vmxnet3 virtual device (Shahar Klein) Cleanups: - remote: Don't print a warning every time a remote call fails (Cole Robinson) - storage: Report errors in FindPoolSources (Cole Robinson) - LXC fix return code handling in lxcVmStart (Ryota Ozaki) - Add a target for libvirt.devhelp (Daniel Veillard) - Remove some auto-generated files (Daniel P. Berrange) - Re-arrange doTunnelMigrate to simplify cleanup code (Daniel P. Berrange) - Separate out code for sending tunnelled data (Daniel P. Berrange) - Pull connection handling code out of doTunnelMigrate (Daniel P. Berrange) - Refactor native QEMU migration code (Daniel P. Berrange) - Don't force dconn to be NULL in virDomainMigrate (Daniel P. Berrange) - Remove unneccessary uri_in parameter from virMigratePrepareTunnel (Daniel P. Berrange) - Move the VIR_DRV_FEATURE* constants (Daniel P. Berrange) - Fix configure.ac message vertical alignment (Daniel P. Berrange) - cgroup: Fix -Werror breakage (Cole Robinson) - Fix handling return value of qemuMonitorSetBalloon (Ryota Ozaki) - Fix up "make check" (Chris Lalancette) - Fix rebuilding of devhelp files (Daniel P. Berrange) - Fix ordering of <exports> in API description file (Daniel P. Berrange) - node conf: Make parsing routines consistent with other drivers (Cole Robinson) - nodedev: Break out virNodeDeviceHasCap to node_conf (Cole Robinson) - python: Add a newline after custom classes (Cole Robinson) - python: Fix generated virInterface method names (Cole Robinson) - python: Use a pure python implementation of 'vir*GetConnect' (Cole Robinson) - python: Don't generate bindings for vir*Ref (Cole Robinson) - python: Don't generate conflicting conn.createXML functions. (Cole Robinson) - python: Remove use of xmllib in generator.py (Cole Robinson) - python: Remove FastParser from generator. (Cole Robinson) - Fix typo in Makefile.am breaking NEWS file generation (Daniel P. Berrange) - Fix build in separate build directory (Jiri Denemark) - Incorrect error message in virDomainNetDefParseXML (Florian Vichot) - Fix a few 'make rpm' breakages (Daniel Veillard) - Pass remote_message_header to the dispatch functions. (Chris Lalancette) - Fix up some warnings from stream DEBUG statements. (Chris Lalancette) - Fix apibuild.py warnings (Matthias Bolte) - Change signature of remoteSendStreamData() to fix compile warning (Matthias Bolte) - Add virStorageFileGetMetadata() helper (Mark McLoughlin) - Move virStorageGetMetadataFromFD() to libvirt_util (Mark McLoughlin) - Split virStorageGetMetadataFromFD() from virStorageBackendProbeTarget() (Mark McLoughlin) - Move file format enum to libvirt_util (Mark McLoughlin) - Remove hand-crafted UUID parsers (Daniel P. Berrange) - Helper functions for processing data streams in libvirtd (Daniel P. Berrange) - Standardize debugging messages in QEMU monitor code (Daniel P. Berrange) - Remove low level monitor APIs from header file (Daniel P. Berrange) - Rename qemudMonitorSendCont to qemuMonitorStartCPUs (Daniel P. Berrange) - Pull QEMU monitor interaction out to separate file (Daniel P. Berrange) - util.h needs libvirt.h for virConnectPtr (Mark McLoughlin) - Fix API doc extractor to stop munging comment formatting (Daniel P. Berrange) - Fix secret_driver compile warning, bug. (Charles Duffy) - ESX remove phantom mode (Matthias Bolte) - ESX replace esxUtil_EqualSuffix() with virFileHasSuffix() (Matthias Bolte) - ESX Whitespace cleanup (Matthias Bolte) - Fix up "make syntax-check" after the tree restructuring. (Chris Lalancette) - Introduce virStrncpy. (Chris Lalancette) - Ignore auto-generated header file (Daniel P. Berrange) - Remove an unnecessary variable from remoteIOReadMessage(). (Chris Lalancette) - Remove auto-generated header file from repo (Daniel P. Berrange) - Move example XML files into examples/xml (Daniel P. Berrange) - Remove all generated docs from source control (Daniel P. Berrange) - Fix missing data file in qemuhelpdata (Daniel P. Berrange) - Misc syntax-check fixes (Daniel P. Berrange) - Move remote protocol definition into src/remote/ (Daniel P. Berrange) - Move all shared utility files to src/util/ (Daniel P. Berrange) - Move all XML configuration handling to src/conf/ (Daniel P. Berrange) - Re-arrange python generator to make it clear what's auto-generated (Daniel P. Berrange) - Remove obsolete files (Daniel P. Berrange) - Move docs/examples into examples/ (Daniel P. Berrange) - Remove unused images from docs/ directory (Daniel P. Berrange) - Rename daemon main code (Daniel P. Berrange) - Move config files to align with driver sources (Daniel P. Berrange) - Move virsh into tools/ directory (Daniel P. Berrange) - Move security drivers to src/security/ (Daniel P. Berrange) - Move secret driver into src/secret/ (Daniel P. Berrange) - Move netcf interface driver into src/interface/ (Daniel P. Berrange) - Move network driver into src/network (Daniel P. Berrange) - Move remote driver to src/remote/ (Daniel P. Berrange) - Move test driver into src/test/ (Daniel P. Berrange) - Move node device drivers to src/node_device/ (Daniel P. Berrange) - Move storage drivers into src/storage/ (Daniel P. Berrange) - Move OpenVZ driver to src/openvz/ (Daniel P. Berrange) - Move UML driver to src/uml/ (Daniel P. Berrange) - Move QEMU driver to src/qemu/ (Daniel P. Berrange) - Move LXC driver into src/lxc/ (Daniel P. Berrange) - Move xen driver code into src/xen/ directory (Daniel P. Berrange) - Rename qemud/ directory to daemon/ (Daniel P. Berrange) - Refactor libvirt.spec to allow client-only builds (Daniel P. Berrange) Please give it a try, and thanks everybody for reports, patches or documentation ! 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]: Not able to login to container when cgroup is enabled
by Rishikesh 14 Oct '09

14 Oct '09

Hi All, On F12 i was playing with libvirt support for container. I am seeing two issue over here: 1> Not able to create a container when start cgconfig services. 2> When i stop cgconfig, then able to create a container, but seeing few error messages to libvirtd daemon. 3> After creation of container through libvirt, i am not able to touch any file or directory. Attached: vm1.xml , through which i created container. I referred following link: http://libvirt.org/drvlxc.html http://openvz.org/pipermail/devel/2008-September/014314.html Detailed Bug Report: When i create a container without starting "cgconfig" service, i am able to create but getting some error message in libvirtd daemon: [root@mx3950 ~]# libvirtd 14:47:24.556: warning : qemudStartup:565 : Unable to create cgroup for driver: No such device or address 14:47:24.647: warning : lxcStartup:1462 : Unable to create cgroup for driver: No such device or address libvir: Linux Container error : failed to connect to client socket: No such file or directory libvir: Linux Container error : failed to connect to client socket: No such file or directory libvir: Linux Container error : failed to connect to client socket: No such file or directory But when i start cgconfig service, then i am not able to open console for my container. Below is the detailed steps which i executed. [root@mx3950 ~]# /etc/init.d/cgconfig stop Stopping cgconfig service: [ OK ] [root@mx3950 ~]# virsh -c lxc:/// define vm1.xml error: Failed to define domain from vm1.xml error: operation failed: domain 'vm1' is already defined with uuid ef558ab3-54ea-5b95-205d-8c0096ae9a4f [root@mx3950 ~]# virsh -c lxc:/// start vm1 Domain vm1 started [root@mx3950 ~]# virsh -c lxc:/// console vm1 Connected to domain vm1 Escape character is ^] sh-4.0# ps -ef UID PID PPID C STIME TTY TIME CMD root 1 0 0 15:23 pts/4 00:00:00 /bin/sh root 4 1 0 15:23 pts/4 00:00:00 ps -ef sh-4.0# touch rishi touch: cannot touch `rishi': Permission denied sh-4.0# exit exit [root@mx3950 ~]# /etc/init.d/cgconfig start Starting cgconfig service: [ OK ] [root@mx3950 ~]# virsh -c lxc:/// start vm1 Domain vm1 started [root@mx3950 ~]# virsh -c lxc:/// console vm1 Connected to domain vm1 Escape character is ^] 15:24:12.869: error : vshRunConsole:77 : unable to open tty /dev/pts/3: No such file or directory [root@mx3950 ~]# Package List: [root@mx3950 ~]# rpm -qa | grep -e libcgroup -e libvirt libcgroup-0.34-2.fc12.x86_64 libvirt-0.7.1-1.fc12.x86_64 libvirt-client-0.7.1-1.fc12.x86_64 libvirt-python-0.7.1-1.fc12.x86_64 [root@mx3950 ~]# uname -a Linux mx3950.in.ibm.com 2.6.31-14.fc12.x86_64 #1 SMP Tue Sep 15 03:48:57 EDT 2009 x86_64 x86_64 x86_64 GNU/Linux

5 17

[libvirt] Execute script on migration
by Radek Hladík 14 Oct '09

14 Oct '09

Short question: What is the best way to run some script on VM migration? Detailed description: We use own system for volume management. Its based on md and we found it very flexible. Basic idea is that we have md mirror everywhere we can. This suits great for libvirt: We have a disk on iSCSI server, its "partioned" using device mapper, the resulting "partition" is member of md raid1, that raid is exported via iSCSI to the libvirt node, it is on the node included in another md raid1 and that "final" md raid is used as storage by the VM. This provides high level of abstraction that allows us to migrate the storage to another disk, another iSCSI server, etc... The only glitch is on live migration of the VM. It is possible to assemble and run the array on two nodes. Also all superblock modifications are done on first write to the array. However it would be more secure to stop the array after the source vm stops and before destination vm starts. What is the best way to run some script in this particular moment? I know we could create a new storage driver but I am not sure that it is the best option. -- Open WebMail Project (http://openwebmail.org)

3 2

[libvirt] [PATCH] test: Activate interfaces specified through driver config file.
by Cole Robinson 14 Oct '09

14 Oct '09

This matches the behavior of domains, storage, etc. Also, fix up some whitespace issues in the area. Signed-off-by: Cole Robinson <crobinso(a)redhat.com> --- src/test/test_driver.c | 5 ++++- 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/src/test/test_driver.c b/src/test/test_driver.c index 0b866d2..0541a73 100644 --- a/src/test/test_driver.c +++ b/src/test/test_driver.c @@ -779,13 +779,16 @@ static int testOpenFromFile(virConnectPtr conn, if (!def) goto error; } else { - if ((def = virInterfaceDefParseNode(conn, xml, ifaces[i])) == NULL) + if ((def = virInterfaceDefParseNode(conn, xml, ifaces[i])) == NULL) goto error; } + if (!(iface = virInterfaceAssignDef(conn, &privconn->ifaces, def))) { virInterfaceDefFree(def); goto error; } + + iface->active = 1; virInterfaceObjUnlock(iface); } VIR_FREE(ifaces); -- 1.6.5.rc2

3 3

[libvirt] [PATCH] storage: Report errors in FindPoolSources
by Cole Robinson 14 Oct '09

14 Oct '09

Signed-off-by: Cole Robinson <crobinso(a)redhat.com> --- src/storage/storage_driver.c | 15 ++++++++++++--- 1 files changed, 12 insertions(+), 3 deletions(-) diff --git a/src/storage/storage_driver.c b/src/storage/storage_driver.c index 9ab53e1..4f8949b 100644 --- a/src/storage/storage_driver.c +++ b/src/storage/storage_driver.c @@ -440,15 +440,24 @@ storageFindPoolSources(virConnectPtr conn, char *ret = NULL; backend_type = virStoragePoolTypeFromString(type); - if (backend_type < 0) + if (backend_type < 0) { + virStorageReportError(conn, VIR_ERR_INTERNAL_ERROR, + _("unknown storage pool type %s"), type); goto cleanup; + } backend = virStorageBackendForType(backend_type); if (backend == NULL) goto cleanup; - if (backend->findPoolSources) - ret = backend->findPoolSources(conn, srcSpec, flags); + if (!backend->findPoolSources) { + virStorageReportError(conn, VIR_ERR_NO_SUPPORT, + _("pool type '%s' does not support source " + "discovery"), type); + goto cleanup; + } + + ret = backend->findPoolSources(conn, srcSpec, flags); cleanup: return ret; -- 1.6.5.rc2

3 3

[libvirt] [PATCH] remote: Don't print a warning every time a remote call fails
by Cole Robinson 14 Oct '09

14 Oct '09

$ sudo virsh pool-start idontexist 10:58:18.716: warning : processCallDispatchReply:7612 : Method call error error: failed to get pool 'idontexist' error: Storage pool not found: no pool with matching name 'idontexist' That warning doesn't server much purpose being printed via a virsh call. Change the warning to a debug message. Signed-off-by: Cole Robinson <crobinso(a)redhat.com> --- src/remote/remote_driver.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/remote/remote_driver.c b/src/remote/remote_driver.c index 284593a..162d682 100644 --- a/src/remote/remote_driver.c +++ b/src/remote/remote_driver.c @@ -7606,7 +7606,7 @@ processCallDispatchReply(virConnectPtr conn, struct private_data *priv, return 0; case REMOTE_ERROR: - VIR_WARN0("Method call error"); + VIR_DEBUG0("Method call error"); memset (&thecall->err, 0, sizeof thecall->err); if (!xdr_remote_error (xdr, &thecall->err)) { error (in_open ? NULL : conn, -- 1.6.5.rc2

2 2

[libvirt] [PATCH] LXC fix rc handling in lxcVmStart
by Ryota Ozaki 13 Oct '09

13 Oct '09

Hi, In lxcVmStart we have to set rc = -1 before jumping to the cleanup code when a local procedure call fails and the rc is overwritten, however, some codes don't observe the rule and so invalid cleanups are likely to happen. A simple solution for the defect is to add rc = -1 before every jumping if needed, however, IMHO, we first should not reuse rc for another purpose rather than the return value of lxcVmStart. Instead the patch introduces yet another variable for storing only the return value of the local procedure calls. By doing so, we don't need to care about resetting rc anymore. Thanks, ozaki-r >From c3eb2cfa1280b220d5b92ad2ad8bbf6cf6ca0a04 Mon Sep 17 00:00:00 2001 From: Ryota Ozaki <ozaki.ryota(a)gmail.com> Date: Tue, 13 Oct 2009 00:36:41 +0900 Subject: [PATCH] LXC fix rc handling in lxcVmStart * src/lxc/lxc_driver.c: don't reuse rc for local procedure calls --- src/lxc/lxc_driver.c | 15 ++++++--------- 1 files changed, 6 insertions(+), 9 deletions(-) diff --git a/src/lxc/lxc_driver.c b/src/lxc/lxc_driver.c index ef0d368..0b614e3 100644 --- a/src/lxc/lxc_driver.c +++ b/src/lxc/lxc_driver.c @@ -1117,7 +1117,7 @@ static int lxcVmStart(virConnectPtr conn, lxc_driver_t * driver, virDomainObjPtr vm) { - int rc = -1; + int rc = -1, r; unsigned int i; int parentTty; char *parentTtyPath = NULL; @@ -1126,8 +1126,8 @@ static int lxcVmStart(virConnectPtr conn, unsigned int nveths = 0; char **veths = NULL; - if ((rc = virFileMakePath(driver->logDir)) < 0) { - virReportSystemError(conn, rc, + if ((r = virFileMakePath(driver->logDir)) < 0) { + virReportSystemError(conn, r, _("cannot create log directory '%s'"), driver->logDir); return -1; @@ -1157,10 +1157,8 @@ static int lxcVmStart(virConnectPtr conn, goto cleanup; /* Persist the live configuration now we have veth & tty info */ - if (virDomainSaveConfig(conn, driver->stateDir, vm->def) < 0) { - rc = -1; + if (virDomainSaveConfig(conn, driver->stateDir, vm->def) < 0) goto cleanup; - } if ((logfd = open(logfile, O_WRONLY | O_APPEND | O_CREAT, S_IRUSR|S_IWUSR)) < 0) { @@ -1183,11 +1181,10 @@ static int lxcVmStart(virConnectPtr conn, goto cleanup; /* And get its pid */ - if ((rc = virFileReadPid(driver->stateDir, vm->def->name, &vm->pid)) != 0) { - virReportSystemError(conn, rc, + if ((r = virFileReadPid(driver->stateDir, vm->def->name, &vm->pid)) != 0) { + virReportSystemError(conn, r, _("Failed to read pid file %s/%s.pid"), driver->stateDir, vm->def->name); - rc = -1; goto cleanup; } -- 1.6.2.5

4 3

[libvirt] unable to start guest under kvm (revisited)
by Anton Protopopov 13 Oct '09

13 Oct '09

The problem, that I post here http://www.mail-archive.com/libvir-list@redhat.com/msg16154.html and that was solved, comes back in a different way. Consider a working kvm machine with the following description: ... <os> <type arch='x86_64' machine='pc'>hvm</type> <boot dev='hd'/> </os> ... If I delete "machine='pc'" attribute from <type>, so description becomes ... <os> <type arch='x86_64'>hvm</type> <boot dev='hd'/> </os> ... then libvirt chooses a machine='pc-0.11' value (again: qemu API version instead of kvm version) and I get virsh # start DD error: Failed to start domain DD error: internal error unable to start guest: Supported machines are: pc Standard PC (default) isapc ISA-only PC --- This problem actually occurs when using virt-manager, see https://www.redhat.com/archives/virt-tools-list/2009-October/msg00037.html

4 6

[libvirt] feature request: virDomainSendkeys API
by Dan Kenigsberg 13 Oct '09

13 Oct '09

Our QA team makes use of qemu's sendkeys monitor command to test interesting things such as Windows guest installation-from-scratch. When we move to libvirt, we loose this ability. On irc, dpb suggested using vnc to send the required keys. Are you aware of a command-line tool for that, or should we build our own? If we go down this path, we would have to build a similar tool for testing VMs with SPICE. (and have nothing to do for VMs with no graphics support, but these are not used in the product anyway) Alternatively, libvirt can supply a new API for injecting key strokes into a domain. Surely it's useful only for debug and qa, but so is virDomainMemoryPeek. Is that reasonable? Dan.

3 2

[libvirt] cannot set ownership on /nfs/mounted/image.iso: Operation not permitted
by Dan Kenigsberg 13 Oct '09

13 Oct '09

I having libvirt run qemu as non-root, but I don't think libvirt should try chowning image files. Cerntainly not if they sit on a root_squashing nfs server. Also, the behavior of libvirt in this regard is a bit surprising for local image files and even for host devices. It should be enough that an image is accessible to qemu; it does not have to be owned by it. The fact that qemuDomainSetAllDeviceOwnership() with restore=1 does not really restore ownership of device/file but force-chowns it to root:root is anouther surprise. Dan

2 1