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

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

 



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
>         },
> ...
>     ]
> }

Another idea: make it easier for any future changes.  In your top-level
object, add some other fields:

{
  "file-format": 1,
  "comment": "leases file generated by leaseshelper from libvirt 1.2.7",
  "server-duid"...

where file-format was implicitly 0 for the old style, and where you can
bump it to 2 if we have to make future changes; and where the comment is
free-form (human-readable, not machine-parseable) information that makes
it easier to debug what the file contains and which build of
leaseshelper generated it (since the leaseshelper version might be
different than libvirtd).


-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

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]