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 <jjong...@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.

Attachment: signature.asc
Description: PGP signature

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

Reply via email to