[OpenWrt-Devel] [PATCH] procd: Allow override of default respawn parameters
Helmut Schaa
helmut.schaa at googlemail.com
Fri Sep 18 11:34:19 EDT 2015
On Fri, Sep 18, 2015 at 11:52 AM, John Crispin <blogic at openwrt.org> wrote:
>
>
> On 18/09/2015 11:40, Etienne Champetier wrote:
>> Hi,
>>
>> 2015-09-18 11:03 GMT+02:00 Helmut Schaa <helmut.schaa at googlemail.com
>> <mailto:helmut.schaa at googlemail.com>>:
>>
>> Hi John,
>>
>> On Fri, Sep 18, 2015 at 10:18 AM, John Crispin <blogic at openwrt.org
>> <mailto:blogic at openwrt.org>> wrote:
>> > Hi
>> >
>> > On 18/09/2015 09:59, Helmut Schaa wrote:
>> >> Allow to pass RESPAWN_THESHOLD_DEFAULT, DRESPAWN_TIMEOUT_DEFAULT
>> >> and RESPAWN_RETRY_DEFAULT as parameters to cmake to change the
>> >> default respawn behavior.
>> >>
>> >
>> > technically ok but why cant you tweak them in your packages initd script
>> > ? i am wondering what the use case is and if there are other possible
>> > solutions
>>
>> In our tree we've patched most (maybe even all) services to respawn
>> forever
>> (respawn_retry=-1). Including all OpenWrt provided services. Instead
>> of keeping
>> these local modifications to several packages it's easier to just
>> override procds
>> default behavior.
>>
>> I think there might be other people around running OpenWrt on
>> headless boxes
>> where the respawn retry should not be limited by default. However,
>> this is of course
>> not suitable for a default OpenWrt box.
>>
>> If there are good reasons not to include this in procd feel free to
>> drop this patch.
>> However, it causes zero runtime overhead and is quite simple.
>>
>> As a followup I'd add a config flag for procd "respawn_forver_mode"
>> or so that
>> just sets respawn_retry to -1.
>>
>> Helmut
>>
>> It would be great to be able to configure these settings at runtime (ie
>> in an uci file)
>>
>> Etienne
>>
>
> agreed, i think the best would be to add support for -1 == endless
> respawn and then change procd.sh to use current values as default but
> allow overriding them vom /etc/config/system or similar. would that
> solve your use case ?
Hmm, could work. However, changing the setting in /etc/config/system would
only affect all services started after the config file change. In our case we'd
just add it to the default /etc/config/system ...
Let me look into it next week.
Helmut
_______________________________________________
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel
More information about the openwrt-devel
mailing list