Adds support to the qemu driver for the VIR_DOMAIN_START_PERSISTENT flag --- Figured this would be better to "define" the domain before starting it, rather than starting and then defining but I can change as people prefer. --- src/qemu/qemu_driver.c | 45 +++++++++++++++++++++++++++++++++++---------- 1 file changed, 35 insertions(+), 10 deletions(-) diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index 76b2800..0a3bbc2 100644 --- a/src/qemu/qemu_driver.c +++ b/src/qemu/qemu_driver.c @@ -1448,14 +1448,17 @@ static virDomainPtr qemuDomainCreate(virConnectPtr conn, const char *xml, virDomainDefPtr def = NULL; virDomainObjPtr vm = NULL; virDomainPtr dom = NULL; - virDomainEventPtr event = NULL; - virDomainEventPtr event2 = NULL; + virDomainEventPtr event_define = NULL; + virDomainEventPtr event_start = NULL; + virDomainEventPtr event_pause = NULL; unsigned int start_flags = VIR_QEMU_PROCESS_START_COLD; virQEMUCapsPtr qemuCaps = NULL; virCapsPtr caps = NULL; + virQEMUDriverConfigPtr cfg = NULL; virCheckFlags(VIR_DOMAIN_START_PAUSED | - VIR_DOMAIN_START_AUTODESTROY, NULL); + VIR_DOMAIN_START_AUTODESTROY | + VIR_DOMAIN_START_PERSISTENT, NULL); if (flags & VIR_DOMAIN_START_PAUSED) start_flags |= VIR_QEMU_PROCESS_START_PAUSED; @@ -1491,6 +1494,24 @@ static virDomainPtr qemuDomainCreate(virConnectPtr conn, const char *xml, def = NULL; + /* Attempt to make the domain persistent before we start it */ + if (flags & VIR_DOMAIN_START_PERSISTENT) { + vm->persistent = 1; + + cfg = virQEMUDriverGetConfig(driver); + + if (virDomainSaveConfig(cfg->configDir, vm->newDef) < 0) { + VIR_INFO("Deleting domain '%s'", vm->def->name); + qemuDomainRemoveInactive(driver, vm); + vm = NULL; + goto cleanup; + } + + event_define = virDomainEventNewFromObj(vm, + VIR_DOMAIN_EVENT_DEFINED, + VIR_DOMAIN_EVENT_DEFINED_ADDED); + } + if (qemuDomainObjBeginJob(driver, vm, QEMU_JOB_MODIFY) < 0) goto cleanup; /* XXXX free the 'vm' we created ? */ @@ -1504,21 +1525,22 @@ static virDomainPtr qemuDomainCreate(virConnectPtr conn, const char *xml, goto cleanup; } - event = virDomainEventNewFromObj(vm, + event_start = virDomainEventNewFromObj(vm, VIR_DOMAIN_EVENT_STARTED, VIR_DOMAIN_EVENT_STARTED_BOOTED); - if (event && (flags & VIR_DOMAIN_START_PAUSED)) { + if (event_start && (flags & VIR_DOMAIN_START_PAUSED)) { /* There are two classes of event-watching clients - those * that only care about on/off (and must see a started event * no matter what, but don't care about suspend events), and * those that also care about running/paused. To satisfy both * client types, we have to send two events. */ - event2 = virDomainEventNewFromObj(vm, + event_pause = virDomainEventNewFromObj(vm, VIR_DOMAIN_EVENT_SUSPENDED, VIR_DOMAIN_EVENT_SUSPENDED_PAUSED); } virDomainAuditStart(vm, "booted", true); + VIR_INFO("Creating domain '%s'", vm->def->name); dom = virGetDomain(conn, vm->def->name, vm->def->uuid); if (dom) dom->id = vm->def->id; @@ -1530,13 +1552,16 @@ cleanup: virDomainDefFree(def); if (vm) virObjectUnlock(vm); - if (event) { - qemuDomainEventQueue(driver, event); - if (event2) - qemuDomainEventQueue(driver, event2); + if (event_define) + qemuDomainEventQueue(driver, event_define); + if (event_start) { + qemuDomainEventQueue(driver, event_start); + if (event_pause) + qemuDomainEventQueue(driver, event_pause); } virObjectUnref(caps); virObjectUnref(qemuCaps); + virObjectUnref(cfg); return dom; } -- 1.7.12.4

Added support for a --start flag to the define command --- Implements https://www.redhat.com/archives/libvir-list/2013-January/msg00490.html in a new way using the persistent flag for create. --- tools/virsh-domain.c | 9 ++++++++- tools/virsh.pod | 5 +++-- 2 files changed, 11 insertions(+), 3 deletions(-) diff --git a/tools/virsh-domain.c b/tools/virsh-domain.c index cb56b3d..5a18316 100644 --- a/tools/virsh-domain.c +++ b/tools/virsh-domain.c @@ -6475,6 +6475,9 @@ static const vshCmdInfo info_define[] = { {.name = "desc", .data = N_("Define a domain.") }, + {.name = "starT", + .data = N_("Start domain after defining it.") + }, {.name = NULL} }; @@ -6501,7 +6504,11 @@ cmdDefine(vshControl *ctl, const vshCmd *cmd) if (virFileReadAll(from, VSH_MAX_XML_FILE, &buffer) < 0) return false; - dom = virDomainDefineXML(ctl->conn, buffer); + if (vshCommandOptBool(cmd, "start")) + dom = virDomainCreateXML(ctl->conn, buffer, + VIR_DOMAIN_START_PERSISTENT); + else + dom = virDomainDefineXML(ctl->conn, buffer); VIR_FREE(buffer); if (dom != NULL) { diff --git a/tools/virsh.pod b/tools/virsh.pod index 3ea4ba6..91bbd6c 100644 --- a/tools/virsh.pod +++ b/tools/virsh.pod @@ -558,11 +558,12 @@ B<Example> vi domain.xml (or make changes with your other text editor) virsh create domain.xml -=item B<define> I<FILE> +=item B<define> I<FILE> [I<--start>] Define a domain from an XML <file>. The domain definition is registered but not started. If domain is already running, the changes will take -effect on the next boot. +effect on the next boot. If I<--start> is requested, start the domain +after defining it. =item B<desc> I<domain> [[I<--live>] [I<--config>] | [I<--current>]] [I<--title>] [I<--edit>] [I<--new-desc> -- 1.7.12.4

On Fri, Feb 22, 2013 at 11:40:32AM -0600, Doug Goldstein wrote: As previously discussed, I'm against including this in the API because it doesn't offer anything that can't already be done with the existing APIs. Indeed this is worse than the existing APIs because this has only been wired up for the QEMU driver and none others. It also increases the size of the code and thus maintenance work for each driver for no feature gain. By all means add the flags to virsh, but they can be done using the existing APIs IMHO. 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 :|