Re: [Libvir] Record an error when refusing a read-only operation

On Wed, Aug 16, 2006 at 03:30:30PM +0100, Daniel P. Berrange wrote: > The libvirt.c front-end can be compiled in 'pedantic' mode in which case it > explicitly refuses operations with side-effects in read-only connections. > When doing so, however, it was not recording any error message so feedback > to the user was misleading / obsure. This patch adds an extra error code > VIR_ERR_READ_ONLY which is used when an operation is denied due to a read > only connection. argh, I developped a similar patch independantly, using "VIR_ERR_OPERATION_DENIED" instead, it also reports the operation name and drop the #ifdef PEDANTIC , making it the default behaviour.

> This patch also passes 'VIR_DRV_OPEN_QUIET' to the 'open' method so tell > the XenD driver not complain if it fails to open a HV connection. This is > because it was complaining whenever trying to open a test:///default > connection. That should not conflict, yeah it's probably better to have the libvirt front-end function report the connection error than letting the drivers emit errors themselves. The only problem is that you won't detect hypervisor connection failure say if you manage to connect with xend, because at least one access method succeeded ...