Re: [PATCH v2 2/5] conf: add XML schema for domain XML

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

 



 Thank you for reviewing and sorry for my late response.

> >
> > Is following better? :-)
> >
> > Domain process is allowed to...
> >
> >> + by cap element. What capabilities host supports can be found at
> >> + capability XML.
> >
> > Better to add the virsh command. e.g.
> >
> > capability XML (virsh capabilities)
> 
> Also we need to declare the caps are OS dependant.

  I don't know proper "cap" name, so it complied 
  with that of libcap-ng.

> >
> >> +</p>
> >> +
> >> +<pre>
> >> + ...
> >> +&lt;process&gt;
> >> +&lt;cap name="chown"/&gt;
> >> +&lt;cap name="dac_override"/&gt;
> >> +&lt;cap name="dac_read_search"/&gt;

....

> >
> > Signed-off-by: Taku Izumi<izumi.taku@xxxxxxxxxxxxxx>
> > Signed-off-by: Shota Hirae<m11g1401@xxxxxxxxxxxxxx>
> > ---
> >   docs/formatdomain.html.in     |   48 ++++++++++++++++++++++++++++++++++++++
> >   docs/schemas/domaincommon.rng |   52 ++++++++++++++++++++++++++++++++++++++++++
> >   src/conf/domain_conf.c        |   33 ++++++++++++++++++++++++++
> >   src/conf/domain_conf.h        |    2 +
> >   4 files changed, 135 insertions(+)
> >
> 
> And better to add some tests.

 What type of test?  domain schema test?

--
Best regards, 
Taku Izumi <izumi.taku@xxxxxxxxxxxxxx>

--
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]