2010/6/22 Daniel P. Berrange <berra...@redhat.com>:
> On Tue, Jun 22, 2010 at 08:01:09PM +0200, Matthias Bolte wrote:
>> 2010/6/22 Daniel P. Berrange <berra...@redhat.com>:
>> > If there is no driver for a URI we report
>> >
>> > "no hypervisor driver available"
>> >
>> > This is bad because not all virt drivers are hypervisors (ie container
>> > based virt).
>> >
>> > If there is no driver support for an API we report
>> >
>> > "this function is not supported by the hypervisor"
>> >
>> > This is bad for the same reason, and additionally because it is
>> > also used for the network, interface & storage drivers.
>> >
>> > * src/util/virterror.c: Improve error messages
>> > ---
>> > src/util/virterror.c | 8 ++++----
>> > 1 files changed, 4 insertions(+), 4 deletions(-)
>> >
>> > diff --git a/src/util/virterror.c b/src/util/virterror.c
>> > index 96dd1e7..9f632ec 100644
>> > --- a/src/util/virterror.c
>> > +++ b/src/util/virterror.c
>> > @@ -760,15 +760,15 @@ virErrorMsg(virErrorNumber error, const char *info)
>> > break;
>> > case VIR_ERR_NO_SUPPORT:
>> > if (info == NULL)
>> > - errmsg = _("this function is not supported by the
>> > hypervisor");
>> > + errmsg = _("this function is not supported by the
>> > connection driver");
>> > else
>> > - errmsg = _("this function is not supported by the
>> > hypervisor: %s");
>> > + errmsg = _("this function is not supported by the
>> > connection driver: %s");
>>
>> As you said this message can also be triggered by network, interface,
>> storage, etc. drivers not supporting a function. Therefore, I think we
>> should not include the word 'connection' in the message.
>
> The virConnectPtr is the connection in this context and the driver activated
> there determines what's used for the hypervisor, storage, network, interface
> APIs. Hypervisor was too narrow, but connection is relevant here IMHO.
>
Okay, I can interpret 'connection driver' as a driver attached to (or
used by) a connection.
ACK.
Matthias
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
