<div dir="ltr">I'm attaching the patch because I've been unable to use send-email properly.<br><br>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.</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Oct 13, 2016 at 7:07 AM, Michal Privoznik <span dir="ltr"><<a href="mailto:mprivozn@redhat.com" target="_blank">mprivozn@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On <a href="tel:13.10.2016" value="+13102016">13.10.2016</a> 00:03, Alberto Ruiz wrote:<br>
> Support for custom dhcp wide and per host leasetime.<br>
><br>
> It is specified as a child tag for <dhcp>:<br>
> <dhcp><br>
>   <leasetime>24h</leasetime><br>
>   ...<br>
> </dhcp><br>
><br>
> And as an attribute for <host>:<br>
> <dhcp><br>
>   <host leasetime="7d" .../><br>
> </dhcp><br>
><br>
> These are the different notations:<br>
><br>
> -1   (infinite/unlimited lease)<br>
> 120  (seconds are the default unit, 120 seconds is the minimum, if less is<br>
> specified it will use 120)<br>
> 300s (seconds)<br>
> 5m   (minutes)<br>
> 24h  (hours)<br>
> 7d   (days)<br>
> ---<br>
<br>
</span>I know I'm stepping on a moving train (sorry for that), but I have two<br>
points to raise:<br>
<br>
1) use git send-email, this patch is mangled by your MTA and does not apply.<br>
2) 120 seconds is minimum because of dnsmasq? If so, I think we should<br>
error out instead of silently changing this to a different value behind<br>
user's back.<br>
<span class="HOEnZb"><font color="#888888"><br>
Michal<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div>Alberto Ruiz<br></div><span>Associate </span>Engineering Manager - Desktop Management Tools<br></div>Red Hat<br></div></div></div></div>
</div>