[libvirt] [PATCH 1/5] lib: add API to query info about logged-in users

[Peter Krempa]

On Thu, Jul 11, 2019 at 12:09:23 +0100, Daniel Berrange wrote:
> On Wed, Jul 10, 2019 at 03:07:43PM -0500, Jonathon Jongsma wrote:
> > Add API for querying logged-in users from a domain implemented via
> > guest agent.
> > 
> > Signed-off-by: Jonathon Jongsma <jjongsma at redhat.com>
> > ---
> >  include/libvirt/libvirt-domain.h | 18 ++++++++++
> >  src/driver-hypervisor.h          |  6 ++++
> >  src/libvirt-domain.c             | 62 ++++++++++++++++++++++++++++++++
> >  src/libvirt_public.syms          |  5 +++
> >  4 files changed, 91 insertions(+)

[...]

> > +int virDomainGetGuestUsers(virDomainPtr domain,
> > +                           virDomainUserInfoPtr **info,
> > +                           unsigned int flags);
> 
> I can easily imagine QEMU returning more info per user in future, so
> using a struct is not futureproof design.
> 
> We should use the virTypedParameter approach instead here.

In addition to that, once we go with typed parameters, we can make the
API more universal similarly to virDomainListGetStats. Working on one
VM only in this case, but returning all the possible queryable stuff
from the guest agent, so that we don't have to add APIs for every single
thing an user would want to query.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20190711/d22d7098/attachment-0001.sig>