Re: [libvirt] [PATCH v2] leaseshelper: improvements to support all events
On 07/23/2014 02:32 PM, Eric Blake wrote:
I think the enhanced format should look like:
{
"server-duid": "...",
"leases": [
{
"iaid": "1221229",
"ip-address": "2001:db8:ca2:2:1::95",
"mac-address": "52:54:00:12:a2:6d",
"hostname": "Fedora20",
"client-id":
"00:04:1a:c1:d9:6b:5a:0a:e2:bc:f8:4b:1e:37:2e:38:22:55"
"expiry-time": 1393244216
},
...
]
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.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
Attachments:
- signature.asc (application/pgp-signature — 604 bytes)