admin interface: Warning after changing plugin loglevel about smarthome HOT 8 CLOSED

the same happens after a change in modules logger.

from smarthome.

@aschwith Do you have more details? I cannot reproduce it (running the develop branch)
@onkelandy I can't reproduce that either.

Changeing a logger's level, I get the following log entries, but no WARNING:

2023-02-05  14:53:36 NOTICE   modules.admin.api_loggers set_active_logger_level(): logger=lib.module, level=INFO
2023-02-05  14:53:36 NOTICE   modules.admin.api_loggers set_active_logger_level(): lg=<Logger lib.module (NOTICE)>, lglevel=20
2023-02-05  14:53:36 NOTICE   modules.admin.api_loggers load_logging_config: shng_version=x
2023-02-05  14:53:36 NOTICE   modules.admin.api_loggers  - old level=NOTICE

from smarthome.

Think you wanted to mention @wvhn ..?
I don't have any issues when changing the log level with most recent develop branch.

from smarthome.

I noticed the issue on master branch v1.9.3. Haven't tested yet on develop.

from smarthome.

Same for me: Issue occurs in Master v1.9.3. Did not doublecheck current develop version.

from smarthome.

I couldn't find it at first, because it only happens if the etc/logging.yaml hasn't been changed by the admin interface before. In this case SmartHomeNG tries to create a backup file before writing the new configuration to disk.

Since the error occurred for you, the new configuration wasn't written to disk.

It will be fixed in the next release.

from smarthome.

SmartHomeNG v1.9.4 has been released

from smarthome.

Issue no longer occurs under v1.9.4. Thanks for fixing.

from smarthome.