Hi!

In Ubuntu we've changed libvirt to connect to qemu:///session rather
than xen:/// if no URI is passed to virConnectOpen. fence_xvmd
incidentally does not pass a URI to virConnectOpen and hence will fail.
This patch fixes this old and deprecated use of virConnectOpen.

Thank you!

(Also available as http://people.ubuntu.com/~soren/libvirt_xen_uri.diff)

Index: fence/agents/xvm/fence_xvmd.c
================================================== =================
RCS file: /cvs/cluster/cluster/fence/agents/xvm/fence_xvmd.c,v
retrieving revision 1.13
diff -u -r1.13 fence_xvmd.c
--- fence/agents/xvm/fence_xvmd.c 17 Oct 2007 18:29:57 -0000 1.13
+++ fence/agents/xvm/fence_xvmd.c 14 Dec 2007 08:07:01 -0000
@@ -54,6 +54,8 @@
#include "libcman.h"
#include "debug.h"

+#define LIBVIRT_XEN_URI "xen:///"
+
static int running = 1;


@@ -541,7 +543,7 @@
virt_list_t *vl = NULL;
virt_state_t *dom = NULL;

- vp = virConnectOpen(NULL);
+ vp = virConnectOpen(LIBVIRT_XEN_URI);
if (!vp)
perror("virConnectOpen");

@@ -577,7 +579,7 @@
continue;

/* Request and/or timeout: open connection */
- vp = virConnectOpen(NULL);
+ vp = virConnectOpen(LIBVIRT_XEN_URI);
if (!vp) {
printf("NOTICE: virConnectOpen(): %s; cannot fence!
",
strerror(errno));

--
Soren Hansen
Ubuntu Server Team
http://www.ubuntu.com/

On Fri, 14 Dec 2007, Soren Hansen wrote:


Hi!

In Ubuntu we've changed libvirt to connect to qemu:///session rather


[SNIP]

now in CVS HEAD.

Thanks
Fabio

--
I'm going to make him an offer he can't refuse.