[FR] Change log level about ha-shinobi HOT 5 CLOSED

Hi,

Sure, but the question is why?
This is a crucial action to make sure device is being synced, it doesn't contain any transportation data but the action being performed and used as opposed to debug messages that allows to debug by adding the transportation data between classes and API.

Thanks

from ha-shinobi.

Hi,

Sure, but the question is why?

Because it spams the log as every minute you have a line for it. And it grows a lot if you leave HA running for weeks.

This is a crucial action to make sure device is being synced, it doesn't contain any transportation data but the action being > performed and used as opposed to debug messages that allows to debug by adding the transportation data between classes and API.

I think the normal behavior should be to log warning (for example slow updates) or errors (for example failed updates) but a standard behavior message should not be so interesting in everyday life. Instead is important if you need to debug a misbehavior, so I suggested to move it to another level and not at all to remove it.

Hope it's more clear and it makes sense for you as well.
Thank you,

Simone

from ha-shinobi.

Default configuration if HA is Warning for a reason, if you would like to raise the log level you should set the specific component to INFO or DEBUG,
INFO and DEBUG are to document an action performed successfully, debug allows you to view the underlying data of the action,
Using just warning to inform that an action failed will make error and debug as redundant

Thanks

from ha-shinobi.

I understand you want to leave the component as it is. Ok, no problem.
Feel free to close this request.

Just take care that is the only one behaving like that. Many users leave the default logging level as INFO and so do I.

Best regards,

Simone

from ha-shinobi.

Not trying to argue, but... according to HA, default is debug if you set the integration without log level parameter, if you didn't set the logger integration it will be warning and above

The log severity level is warning if the logger integration is not enabled in configuration.yaml.

Taken from https://www.home-assistant.io/integrations/logger/

Anyway, thanks for the suggestion, will consider it for next release

from ha-shinobi.