Re: [PATCH v2] leaseshelper: improvements to support all events

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

 



On 07/23/14 23:42, Nehal J Wani wrote:
>> On IRC, we discussed another alternative - keep the top-level item as an
>> array, and instead of adding "server-duid" as an array element, just add
>> it as an optional field member of each {} ipv6 lease in the array
>> (multiple copies of the string, but oh well).  At least that way, you
>> aren't artificially adding a non-lease to the array itself, and
>> hopefully libvirt 1.2.6 ignores unknown fields of a lease array entry.
> 
> This seems to be the most easy option. Also, how about adding another
> field to every lease:
> "generated-by": "leaseshelper v$X" (where X is the value provided by
> the macro PACKAGE_VERSION)
> so that whenever a new field is added in the future, we don't have
> trouble finding out which version of leaseshelper added it.

Hmmm, that doesn't seem to be such a good idea. If we want to add the
"generated-by" statement, we should wrap the leases array into an object.

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]