File 800_error.patch of Package libvirt-cim
# HG changeset patch
# User Jim Fehlig <jfehlig@novell.com>
# Date 1231965788 25200
# Node ID 611757263eddd2bb97383d250ee636300a91a9e6
# Parent b4f1d84b87c20350fdfe0651a684a24b7e03ba28
Improve error messages for resource manipulation and migration
This patch extends on c/s 768, adding use of virt_set_status() in
resource manipulation extrinsics in Virt_VirtualSystemManagementService
and migration extrinsic in Virt_VSMigrationService.
Signed-off-by: Jim Fehlig <jfehlig@novell.com>
Index: libvirt-cim-0.5.2/src/Virt_VSMigrationService.c
===================================================================
--- libvirt-cim-0.5.2.orig/src/Virt_VSMigrationService.c
+++ libvirt-cim-0.5.2/src/Virt_VSMigrationService.c
@@ -939,9 +939,10 @@ static CMPIStatus handle_migrate(virConn
ddom = virDomainMigrate(dom, dconn, type, NULL, NULL, 0);
if (ddom == NULL) {
CU_DEBUG("Migration failed");
- cu_statusf(_BROKER, &s,
- CMPI_RC_ERR_FAILED,
- "Migration Failed");
+ virt_set_status(_BROKER, &s,
+ CMPI_RC_ERR_FAILED,
+ virDomainGetConnect(dom),
+ "Migration Failed");
}
out:
virDomainFree(ddom);
Index: libvirt-cim-0.5.2/src/Virt_VirtualSystemManagementService.c
===================================================================
--- libvirt-cim-0.5.2.orig/src/Virt_VirtualSystemManagementService.c
+++ libvirt-cim-0.5.2/src/Virt_VirtualSystemManagementService.c
@@ -1336,9 +1336,10 @@ static CMPIStatus _resource_dynamic(stru
CU_DEBUG("Doing dynamic device update for `%s'", dominfo->name);
if (func(dom, dev) == 0) {
- cu_statusf(_BROKER, &s,
- CMPI_RC_ERR_FAILED,
- "Unable to change (%i) device", action);
+ virt_set_status(_BROKER, &s,
+ CMPI_RC_ERR_FAILED,
+ conn,
+ "Unable to change (%i) device", action);
} else {
cu_statusf(_BROKER, &s,
CMPI_RC_OK,