On 2/1/22 13:08, Daniel P. Berrangé wrote:
I still feel like this is rather verbose, and should be simplified down to.{ "return": { "vm": { "kvm": [ ... ] "provider-XYZ": [ ... ], ... } }
My main qualm with this is that not just QEMU, but every layer above then needs to either treat stats as a "dynamic" type unless they want to only handle providers that they know.
The main reason why I asked Mark to do all this work, was so that new stats and stats providers could be added transparently, and only new *targets* would need work all over the stack (but those are fewer, for example blockdev/netdev/iothread).
Paolo
