[libvirt] [PATCH 10/14] lib: introduce virDomainSetLifecycleAction() API
Pavel Hrdina
phrdina at redhat.com
Thu Oct 19 08:07:54 UTC 2017
On Wed, Oct 18, 2017 at 04:38:23PM -0400, John Ferlan wrote:
>
>
> On 10/16/2017 07:06 AM, Pavel Hrdina wrote:
> > Signed-off-by: Pavel Hrdina <phrdina at redhat.com>
> > ---
> > docs/formatdomain.html.in | 6 +++++
> > include/libvirt/libvirt-domain.h | 28 ++++++++++++++++++++
> > src/conf/domain_conf.h | 19 --------------
> > src/driver-hypervisor.h | 7 +++++
> > src/libvirt-domain.c | 56 ++++++++++++++++++++++++++++++++++++++++
> > src/libvirt_public.syms | 5 ++++
> > src/remote/remote_driver.c | 1 +
> > src/remote/remote_protocol.x | 14 +++++++++-
> > 8 files changed, 116 insertions(+), 20 deletions(-)
>
> Of course the aforementioned src/remote_protocol-structs will be here...
>
> >
> > diff --git a/docs/formatdomain.html.in b/docs/formatdomain.html.in
> > index b93ace7cba..cb19a547fe 100644
> > --- a/docs/formatdomain.html.in
> > +++ b/docs/formatdomain.html.in
> > @@ -1623,6 +1623,12 @@
> > </dl>
> >
> > <p>
> > + The lifecycle events can be configured via
> > + virDomainSetLifecycleAction() API
> > + <span class="since">since 3.9.0 (QEMU only)</span>.
> > + </p>
>
> s/via/via the/
> s/()//
>
> Add the href like other examples, such as
>
> <a href="html/libvirt-libvirt-domain.html#virDomainSetLifecycleAction">
> <code>virDomainSetLifecycleAction</code>
> </a>
Fixed
> I also don't think you can say "(QEMU only)" since the API exists in
> libvirt - the implementation is only on QEMU, so you have to add some
> text indicating not all hypervisors support.
>
> You may also want to consider text like:
>
> <span class="since">Since 3.9.0</span>, the lifecycle events can be
> configured via the xxx API. Not all hypervisors support all events and
> all actions.
The sentence "Not all hypervisors ..." is used few paragraphs above this
one so I'll just remove the (QEMU only) part. We actually have a matrix
of supported APIs by drivers <https://libvirt.org/hvsupport.html> so
there is no need to mention that not all hypervisors support this API.
> I think having the since afterwards can be "confusing" sometimes, ymmv.
Usually when it's at the end of a whole paragraph it indicates that
everything in that paragraph is covered by the "since" limitation.
It can definitely be at the beginning :) I personally don't care.
Thanks
Pavel
-------------- 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/20171019/675fe594/attachment-0001.sig>
More information about the libvir-list
mailing list