Re: [libvirt PATCH v3 5/5] Add second qom-list-types call to test data.

On Mon, Apr 20, 2020 at 15:25:10 -0400, Tobin Feldman-Fitzthum wrote: > We make an additional call to qom-list-types. Adjust > qemucapabilitiesdata accordingly. > > Signed-off-by: Tobin Feldman-Fitzthum <tobin(a)linux.vnet.ibm.com&gt; > --- > .../caps_2.10.0.aarch64.replies | 2699 ++- > .../caps_2.10.0.ppc64.replies | 2799 +++- > .../caps_2.10.0.s390x.replies | 1027 +- > .../caps_2.10.0.x86_64.replies | 1708 +- > .../caps_2.11.0.s390x.replies | 1063 +- > .../caps_2.11.0.x86_64.replies | 1692 +- > .../caps_2.12.0.aarch64.replies | 2912 +++- > .../caps_2.12.0.ppc64.replies | 2947 +++- > .../caps_2.12.0.s390x.replies | 1087 +- > .../caps_2.12.0.x86_64.replies | 1767 +- > .../caps_3.0.0.ppc64.replies | 2979 +++- > .../caps_3.0.0.s390x.replies | 1117 +- > .../caps_3.0.0.x86_64.replies | 1783 +- > .../caps_3.1.0.ppc64.replies | 2999 +++- > .../caps_3.1.0.x86_64.replies | 1803 +- > .../caps_4.0.0.aarch64.replies | 3175 +++- > .../caps_4.0.0.ppc64.replies | 3171 +++- > .../caps_4.0.0.s390x.replies | 1259 +- > .../caps_4.0.0.x86_64.replies | 1915 ++- > .../caps_4.1.0.x86_64.replies | 2155 ++- > .../caps_4.2.0.aarch64.replies | 3355 +++- > .../caps_4.2.0.ppc64.replies | 3208 +++- > .../caps_4.2.0.s390x.replies | 1264 +- > .../caps_4.2.0.x86_64.replies | 2244 ++- > .../caps_5.0.0.aarch64.replies | 3391 +++- > .../caps_5.0.0.ppc64.replies | 13986 > +++++++++------- > .../qemucapabilitiesdata/caps_5.0.0.ppc64.xml | 1214 +- > .../caps_5.0.0.x86_64.replies | 2267 ++- > 28 files changed, 64830 insertions(+), 8156 deletions(-) Note that these changes must be part of the commit which actually addss the calls

> > diff --git a/tests/qemucapabilitiesdata/caps_2.10.0.aarch64.replies > b/tests/qemucapabilitiesdata/caps_2.10.0.aarch64.replies > index c75d4ab8a7..a9587e24ed 100644 > --- a/tests/qemucapabilitiesdata/caps_2.10.0.aarch64.replies > +++ b/tests/qemucapabilitiesdata/caps_2.10.0.aarch64.replies > @@ -3078,12 +3078,2587 @@ > "id": "libvirt-6" > } > > +{ > + "execute": "qom-list-types", > + "id": "libvirt-7" > +} We already do call 'qom-list-types' once visible as command libvirt-6 above. You really should re-use the data rather than calling it again.