# HG changeset patch
# User Richard Maciel <rmaciel(a)linux.vnet.ibm.com>
# Date 1234981406 10800
# Node ID 9ffb5f88de2b5abc58d80bb647c352e5ddf94de4
# Parent 9aeb111b89fb3b1428f8b21af5368a6b5c57f948
This patch exposes error messages from libvirt calls in ComputerSystem
Signed-off-by: Richard Maciel <rmaciel(a)linux.vnet.ibm.com>
diff -r 9aeb111b89fb -r 9ffb5f88de2b src/Virt_ComputerSystem.c
--- a/src/Virt_ComputerSystem.c Wed Feb 18 15:23:02 2009 -0300
+++ b/src/Virt_ComputerSystem.c Wed Feb 18 15:23:26 2009 -0300
@@ -570,10 +570,11 @@
dom = virDomainLookupByName(conn, name);
if (dom == NULL) {
- cu_statusf(broker, &s,
- CMPI_RC_ERR_NOT_FOUND,
- "No such instance (%s).",
- name);
+ virt_set_status(broker, &s,
+ CMPI_RC_ERR_NOT_FOUND,
+ conn,
+ "Referenced domain `%s' does not exist",
+ name);
goto out;
}
@@ -790,9 +791,10 @@
xml = virDomainGetXMLDesc(dom, 0);
if (xml == NULL) {
CU_DEBUG("Unable to retrieve domain XML");
- cu_statusf(_BROKER, &s,
- CMPI_RC_ERR_FAILED,
- "Unable to get domain definition");
+ virt_set_status(_BROKER, &s,
+ CMPI_RC_ERR_FAILED,
+ conn,
+ "Unable to get domain definition");
return s;
}
@@ -1003,16 +1005,18 @@
dom = virDomainLookupByName(conn, name);
if (dom == NULL) {
- cu_statusf(_BROKER, &s,
- CMPI_RC_ERR_FAILED,
- "Domain not found");
+ virt_set_status(_BROKER, &s,
+ CMPI_RC_ERR_FAILED,
+ conn,
+ "Domain not found");
goto out;
}
if (virDomainGetInfo(dom, &info) != 0) {
- cu_statusf(_BROKER, &s,
- CMPI_RC_ERR_FAILED,
- "Unable to get current state");
+ virt_set_status(_BROKER, &s,
+ CMPI_RC_ERR_FAILED,
+ conn,
+ "Unable to get current state");
goto out;
}