[PATCH 7/7] qemudomainsnapshotxml2xmltest: Add test case for a snapshot with 'inactiveDomain' element
Daniel P. Berrangé
berrange at redhat.com
Mon Sep 26 11:07:07 UTC 2022
On Thu, Aug 25, 2022 at 11:46:10AM +0200, Peter Krempa wrote:
> Signed-off-by: Peter Krempa <pkrempa at redhat.com>
> ---
> .../memory-snapshot-inactivedomain.xml | 148 ++++++++++++++++++
> tests/qemudomainsnapshotxml2xmltest.c | 2 +
> 2 files changed, 150 insertions(+)
> create mode 100644 tests/qemudomainsnapshotxml2xmlout/memory-snapshot-inactivedomain.xml
snip
> diff --git a/tests/qemudomainsnapshotxml2xmlout/memory-snapshot-inactivedomain.xml b/tests/qemudomainsnapshotxml2xmlout/memory-snapshot-inactivedomain.xml
> new file mode 100644
> index 0000000000..3acc6d952b
> --- /dev/null
> +++ b/tests/qemudomainsnapshotxml2xmlout/memory-snapshot-inactivedomain.xml
> @@ -0,0 +1,148 @@
> + <cookie>
> + <cpu mode='custom' match='exact' check='full'>
> + <model fallback='forbid'>qemu64</model>
> + <feature policy='require' name='x2apic'/>
> + <feature policy='require' name='hypervisor'/>
> + <feature policy='require' name='lahf_lm'/>
> + <feature policy='disable' name='svm'/>
> + </cpu>
> + <slirpHelper/>
> + </cookie>
> +</domainsnapshot>
As best I can tell this <cookie> element is undocumented and further
contains arbitrary undocumented hypervisor specific XML content :-(
This is really horrible and really should have been hidden behind a
driver specific XML namespace.
Can someone explain more about the <cookie> ?
If this is a strictly output only element, which applications are
not expected to parse or interpret, just pass along "as is", then
it is possible we can fix/improve this, to clearly demarcate what
is our public XML schema and what is opaque driver specific schema.
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