[libvirt PATCH] schema: Add maxphysaddr element to hostcpu
Daniel P. Berrangé
berrange at redhat.com
Wed Aug 10 16:35:28 UTC 2022
On Wed, Aug 10, 2022 at 06:29:25PM +0200, Tim Wiederhake wrote:
> On Tue, 2022-08-09 at 17:35 +0100, Daniel P. Berrangé wrote:
> > On Tue, Aug 09, 2022 at 06:22:12PM +0200, Tim Wiederhake wrote:
> > > The output of "virsh capabilities" was not conformant to the
> > > capability.rng schema. Add the missing element to the schema.
> >
> > The RNG files are applied against every XML file under tests/,
> > so if we have an unnoticed gap in an RNG file, that is in turn
> > a sign that we have insufficient XML files for in tests/
>
> This is the output of "virsh capabilities", not some input file. I
> don't see a good way to test this (or mock for it) short of full
> integration tests.
Any XML files in tests/capabilityschemadata/ will be validated
against the schema.
> > > Fixes: c647bf29afb9890c792172ecf7db2c9c27babbb6
> > > Signed-off-by: Tim Wiederhake <twiederh at redhat.com>
> > > ---
> > > src/conf/schemas/cputypes.rng | 3 +++
> > > 1 file changed, 3 insertions(+)
> >
> > >
> > > diff --git a/src/conf/schemas/cputypes.rng
> > > b/src/conf/schemas/cputypes.rng
> > > index 4ae386c3c0..d02f2f88cf 100644
> > > --- a/src/conf/schemas/cputypes.rng
> > > +++ b/src/conf/schemas/cputypes.rng
> > > @@ -387,6 +387,9 @@
> > > <optional>
> > > <ref name="cpuTopology"/>
> > > </optional>
> > > + <optional>
> > > + <ref name="cpuMaxPhysAddr"/>
> > > + </optional>
> > > <zeroOrMore>
> > > <element name="feature">
> > > <attribute name="name">
> >
> > So this change is fine, but we also ought to have an XML file to
> > illustrate the usage
>
> Is this a "reviewed-by"?
Not without an example XML file being added to the test suite.
With regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
More information about the libvir-list
mailing list