Re: [PATCH] test_driver: implement virConnectGetLibVersion

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

 



On Mon, Jun 10, 2019 at 03:57:04PM +0200, Peter Krempa wrote:
> On Mon, Jun 10, 2019 at 15:34:57 +0200, Erik Skultety wrote:
> > On Mon, Jun 10, 2019 at 02:38:15PM +0200, Ilias Stamatis wrote:
> > > Signed-off-by: Ilias Stamatis <stamatis.iliass@xxxxxxxxx>
> > > ---
> > >  src/test/test_driver.c | 9 +++++++++
> > >  1 file changed, 9 insertions(+)
> > >
> > > diff --git a/src/test/test_driver.c b/src/test/test_driver.c
> > > index 1aa79ce898..dc267b6ecd 100644
> > > --- a/src/test/test_driver.c
> > > +++ b/src/test/test_driver.c
> > > @@ -1453,6 +1453,14 @@ static char *testConnectGetHostname(virConnectPtr conn ATTRIBUTE_UNUSED)
> > >  }
> > >
> > >
> > > +static int testConnectGetLibVersion(virConnectPtr conn ATTRIBUTE_UNUSED,
> > > +                                    unsigned long *libVer)
> > > +{
> > > +    *libVer = LIBVIR_VERSION_NUMBER;
> > > +    return 0;
> > > +}
> >
> > Test driver is implemented entirely on the client side, so implementing this
> > API is unnecessary, since the public API returns the value in the client
> > library if a specific driver doesn't implement it. If test driver was
> > implemented within the daemon, then it could be different, but in this case it
>
> Note that you can access the test driver also remotely:

Right, but it doesn't change anything in that case either, the value will be
filled in on the destination.

Erik

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux