Re: [libvirt PATCH] schema: Add maxphysaddr element to hostcpu

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(a)redhat.com&gt; >>> --- >>>  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.