Re: [Qemu-devel] [PATCH 2/3] qapi: Add ImageInfo.

[Eric Blake <eblake@redhat.com>]

[-- Attachment #1: Type: text/plain, Size: 2279 bytes --]

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 <benoit@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   eblake@redhat.com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 620 bytes --]