[LEDE-DEV] [PATCH 2/2] [ubox] logd: add ubus reload method

[Alexandru Ardelean]

On Fri, May 13, 2016 at 12:14 AM, Karl Palsson <karlp at tweak.net.au> wrote:
>
> Alexandru Ardelean <ardeleanalex at gmail.com> wrote:
>>
>> Changing the log size dynamically at runtime is a use case that
>> we have.
>
> I'm sorry, but can you talk to me like I'm 5? What is this use
> case? I mean, I can understand it synthetically, like, "If I want
> to resize the log dynamically, I need to be able to reload the
> log config" But I'm simply failing to comprehend what the use
> case is for needing to resize the log dynamically.
>
> Cheers,
> Karl P

It almost feels like we're getting down a slippery slope towards a
never-ending "why ?" set of questions (like with a 5 year old) :)

Short version is: we have a configuration management system on top of
OpenWrt ; system boots with default settings (on every boot), and
config management applies config changes (whether it's right after
boot, or during system's normal operation).
Maybe other people do something similar.
But, I'd like to stop with the use-case stuff here, since it's getting
close to the business logic.

Coming back to the patches. Is it a problem with:
1) The UCI C changes ?
2) The dynamic log resize ?
3) The whole thing ?

Depending on what the problem is, we can adapt the patchset.
Or simply drop the whole thing if it's deemed un-needed by the community.
In the end, it boils down to a use-case that we do (as we do it)
because we want to do it (as we do it), and we've tried to find an
upstream-able approach.

Thanks :)
Alex