Re: [PATCH] qemu: snapshot: Fix incorrect disk type for auto-generated disks

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 12/05/13 20:25, Eric Blake wrote:
> On 12/05/2013 09:30 AM, Peter Krempa wrote:
>> On 12/05/13 17:17, Peter Krempa wrote:
>>> When changing the parsing and formatting functions in commit
>>> 43f2ccdc73090bd03f64de4d58d46ffa0134d705 I forgot to update the qemu
>>> disk alingnment function for snapshots that automatically adds snapshot
> 
> s/alingnment/alignment/
> 
>>> configs for disks that were not mentioned in the XML. The function
>>> allocated a new disk snapshot definition but did not correctly
>>> initialize the snapshot disk source type variable. This resulted into
>>> the disks considered as block devices and invalid XML was generated.
>>>
> 
>>> +            if (disk->type != VIR_DOMAIN_DISK_TYPE_FILE ||
>>
>> I forgot to amend the patch before sending. Please consider && in place
>> of || in the line above.
>>
>>> +                disk->type != -1) {
>>> +                virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
>>> +                               _("cannot generate external snapshot name "
>>> +                                 "for disk '%s' on a '%s' device"),
> 
> ACK as amended.
> 

I've fixed the commit message and pushed this patch.

Thanks.

Peter


Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]