[PATCH procd] ubus: add state measurement

Florian Eckert fe at dev.tdt.de
Wed Mar 22 00:51:29 PDT 2023


Hello Christian


> I wonder if this kind of thing should be under a config flag?

I do not see under system, with the command 'ubus -v list system'
a section 'config'. I do not wanted to add a new section, so I
decided to implement it under 'info' because this are runtime
information as 'load', 'memory' ...

> Like I feel these kind of stats may be needed only for debug purpose?

I do not use it only for debug purpose, but also for tracking changes on
boot when  enabling new services in production. I also use this for some
function on my system to check if we have finished booting completly.

> These are just ideas, if your idea is to use these data for other
> purpose than checking process boot time and investigate some regression
> then it's totally fine to have this present by default.

If it doesn't matter from your point of view whether we leave it there 
or
move it somewhere else, then I plead that we leave it as it is. 
Otherwise,
as already mentioned above, we would have to introduce a new function 
section
under 'system', because all other sections unter 'system' have nothing 
to
do with it.

Best regards

Florian




More information about the openwrt-devel mailing list