On 08/15/2012 10:17 AM, Eric Blake wrote: > On 08/15/2012 08:54 AM, Benoît Canet wrote: >> Signed-off-by: Benoit Canet <ben...@irqsave.net> >> --- >> qapi-schema.json | 33 +++++++++++++++++++++++++++++++++ >> 1 file changed, 33 insertions(+)
>> +# @ImageInfo: >> +# >> +# Information about a QEMU image file >> +# >> +# @filename: name of the image file >> +# >> +# @format: format of the image file These always exist. >> +# >> +# @dirty-flag: true if image is not cleanly closed This exists for QED and qcow2v3, but not for raw or qcow2 without v3 extensions, so probably worth marking #optional >> +# >> +# @cluster-size: size of a cluster in bytes This exists for qcow2, but not for raw, so probably worth marking #optional >> +# >> +# @encrypted: true if the image is encrypted Again, probably #optional >> +# >> +# @actual-size: actual size on disk in bytes of the image Always exists. But is this the size of the contents seen by the guest, or the size of the metadata seen by the host? Both numbers are useful. >> +# >> +# @virtual-size: maximum capacity in bytes of the image Tough call. For qcow2, this would be read from the metadata. For raw, do we list the current size of the file, or do we list the size of free space on the file system containing the file, or do we omit it for raw? >> +# >> +# @backing-filename: name of the backing file > > Should this be #optional, in the case where there is no backing file? > > Missing @backing-filename-format, when that is recorded (as long as > we're going for machine-parseable, print everything useful, even if > qemu-img info doesn't print it right now). Definitely #optional, since > it won't be present when @backing-filename is missing, and even when a > backing file name is present, the format might not have been recorded. > >> +# >> +# @snapshots: list of VM snapshots > > Should this be #optional, in the case where there are no snapshots? > >> +# >> +# Since: 1.3 > > So no way it will make 1.2 hard freeze today :) Actually, I'd like to see this in 1.2, if we can muster up enough other reviewers. -- Eric Blake ebl...@redhat.com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature