[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH 1/3] Allow per-driver config file for lock manager plugins



On Fri, Jun 17, 2011 at 01:38:19PM +0100, Daniel P. Berrange wrote:
> Allow a 'configFile' parameter to be passed into the lock
> drivers to provide configuration. Wire up the QEMU driver
> to pass in file names '/etc/libvirt/qemu-$NAME.conf
> eg qemu-sanlock.conf
[...]
> @@ -162,11 +163,8 @@ virLockManagerPluginPtr virLockManagerPluginNew(const char *name,
>          }
>      }
>  
> -    if (driver->drvInit(VIR_LOCK_MANAGER_VERSION, flags) < 0) {
> -        virLockError(VIR_ERR_INTERNAL_ERROR, "%s",
> -                     _("plugin ABI is not compatible"));
> +    if (driver->drvInit(VIR_LOCK_MANAGER_VERSION, configFile, flags) < 0)
>          goto cleanup;
> -    }
>  
>      if (VIR_ALLOC(plugin) < 0) {
>          virReportOOMError();

  ACK, patch looks fine to me, I'm just surprized by the above chunk
which now seems to lack the error reporting, were we reporting twice,
or should we pass on this condition ?

Daniel

-- 
Daniel Veillard      | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
daniel veillard com  | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library  http://libvirt.org/


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]