<div dir="ltr">Hi,<br><div class="gmail_extra"><br><div class="gmail_quote">2015-09-18 11:03 GMT+02:00 Helmut Schaa <span dir="ltr"><<a href="mailto:helmut.schaa@googlemail.com" target="_blank">helmut.schaa@googlemail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi John,<br>
<span class=""><br>
On Fri, Sep 18, 2015 at 10:18 AM, John Crispin <<a href="mailto:blogic@openwrt.org">blogic@openwrt.org</a>> wrote:<br>
> Hi<br>
><br>
> On 18/09/2015 09:59, Helmut Schaa wrote:<br>
>> Allow to pass RESPAWN_THESHOLD_DEFAULT, DRESPAWN_TIMEOUT_DEFAULT<br>
>> and RESPAWN_RETRY_DEFAULT as parameters to cmake to change the<br>
>> default respawn behavior.<br>
>><br>
><br>
> technically ok but why cant you tweak them in your packages initd script<br>
> ? i am wondering what the use case is and if there are other possible<br>
> solutions<br>
<br>
</span>In our tree we've patched most (maybe even all) services to respawn forever<br>
(respawn_retry=-1). Including all OpenWrt provided services. Instead of keeping<br>
these local modifications to several packages it's easier to just<br>
override procds<br>
default behavior.<br>
<br>
I think there might be other people around running OpenWrt on headless boxes<br>
where the respawn retry should not be limited by default. However,<br>
this is of course<br>
not suitable for a default OpenWrt box.<br>
<br>
If there are good reasons not to include this in procd feel free to<br>
drop this patch.<br>
However, it causes zero runtime overhead and is quite simple.<br>
<br>
As a followup I'd add a config flag for procd "respawn_forver_mode" or so that<br>
just sets respawn_retry to -1.<br>
<span class=""><font color="#888888"><br>
Helmut<br>
</font></span><div class=""><div class="h5"><br>
</div></div></blockquote></div>It would be great to be able to configure these settings at runtime (ie in an uci file)<br><br></div><div class="gmail_extra">Etienne<br></div></div>