* src/openvz_driver.c: patch from Evgeniy Sokolov avoiding domain

  lookups based on ID as this doesn't work for a stopped domain.
Daniel

diff --git a/ChangeLog b/ChangeLog
index b757a1032a614f72a1262c633d66c52ba76dd8c3..42d4f2fccf39abec8634d4692159c7445236b8b3 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+Mon Jul 21 09:29:22 CEST 2008 Daniel Veillard <veillard@redhat.com>
+
+       * src/openvz_driver.c: patch from Evgeniy Sokolov avoiding domain
+         lookups based on ID as this doesn't work for a stopped domain.
+
 Sat Jul 19 09:41:53 CEST 2008 Jim Meyering <meyering@redhat.com>
 
        remove unnecessary "V = NULL;" stmts after VIR_FREE(V)

diff --git a/src/openvz_driver.c b/src/openvz_driver.c
index 61ba6ff41ca60117348dcc02fe20ea5d430cfce9..c15d9ea108753bacf50934f5f2bcfa8aded908d6 100644 (file)
--- a/src/openvz_driver.c
+++ b/src/openvz_driver.c
@@ -289,7 +289,7 @@ static int openvzDomainGetInfo(virDomainPtr dom,
 
 static int openvzDomainShutdown(virDomainPtr dom) {
     struct openvz_driver *driver = (struct openvz_driver *)dom->conn->privateData;
-    struct openvz_vm *vm = openvzFindVMByID(driver, dom->id);
+    struct openvz_vm *vm = openvzFindVMByUUID(driver, dom->uuid);
     const char *prog[] = {VZCTL, "--quiet", "stop", vm->vmdef->name, NULL};
 
     if (!vm) {
@@ -321,7 +321,7 @@ static int openvzDomainShutdown(virDomainPtr dom) {
 static int openvzDomainReboot(virDomainPtr dom,
                               unsigned int flags ATTRIBUTE_UNUSED) {
     struct openvz_driver *driver = (struct openvz_driver *)dom->conn->privateData;
-    struct openvz_vm *vm = openvzFindVMByID(driver, dom->id);
+    struct openvz_vm *vm = openvzFindVMByUUID(driver, dom->uuid);
     const char *prog[] = {VZCTL, "--quiet", "restart", vm->vmdef->name, NULL};
 
     if (!vm) {
@@ -358,7 +358,7 @@ openvzDomainDefineXML(virConnectPtr conn, const char *xml)
     if ((vmdef = openvzParseVMDef(conn, xml, NULL)) == NULL)
         return NULL;
 
-    vm = openvzFindVMByID(driver, strtoI(vmdef->name));
+    vm = openvzFindVMByName(driver, vmdef->name);
     if (vm) {
         openvzLog(OPENVZ_ERR, _("Already an OPENVZ VM active with the id '%s'"),
                   vmdef->name);
@@ -404,7 +404,7 @@ openvzDomainCreateLinux(virConnectPtr conn, const char *xml,
     if (!(vmdef = openvzParseVMDef(conn, xml, NULL)))
         return NULL;
 
-    vm = openvzFindVMByID(driver, strtoI(vmdef->name));
+    vm = openvzFindVMByName(driver, vmdef->name);
     if (vm) {
         openvzFreeVMDef(vmdef);
         openvzLog(OPENVZ_ERR,
@@ -547,7 +547,7 @@ openvzDomainGetAutostart(virDomainPtr dom, int *autostart)
         return -1;
     }
 
-    if (openvzReadConfigParam(vm->vpsid , "ONBOOT", value, sizeof(value)) < 0) {
+    if (openvzReadConfigParam(strtoI(vm->vmdef->name), "ONBOOT", value, sizeof(value)) < 0) {
         openvzError(conn, VIR_ERR_INTERNAL_ERROR, _("Cound not read container config"));
         return -1;
     }