Re: [libvirt] [PATCH] leasetime support per and

2017-04-06 Thread Daniel P. Berrange
Sorry, we missed this 2nd patch posting. FYI, when sending updated versions of patches, we generally recommend to send them as a new top level thread, with 'PATCH v2' in the subject line, not as reply to the v1 patch. We find that makes it less likely to miss the second version when processing em

Re: [libvirt] [PATCH] leasetime support per and

2016-10-13 Thread Alberto Ruiz
I'm attaching the patch because I've been unable to use send-email properly. 2) is a good point, I wanted to discuss it here. Yes 120 is the minimum allowed by dnsmasq, erroring out seems fine to me, I'll change the patch accordingly. On Thu, Oct 13, 2016 at 7:07 AM, Michal Privoznik wrote: > O

Re: [libvirt] [PATCH] leasetime support per and

2016-10-12 Thread Michal Privoznik
On 13.10.2016 00:03, Alberto Ruiz wrote: > Support for custom dhcp wide and per host leasetime. > > It is specified as a child tag for : > > 24h > ... > > > And as an attribute for : > > > > > These are the different notations: > > -1 (infinite/unlimited lease) > 120 (seconds are

[libvirt] [PATCH] leasetime support per and

2016-10-12 Thread Alberto Ruiz
Support for custom dhcp wide and per host leasetime. It is specified as a child tag for : 24h ... And as an attribute for : These are the different notations: -1 (infinite/unlimited lease) 120 (seconds are the default unit, 120 seconds is the minimum, if less is specified it will