do not change global system params from extension

[vladmandic]

First, confirm

• I have read the instruction carefully
• I have searched the existing issues
• I have updated the extension to the latest version

What happened?

this may look silly, but why do this?

logging.addLevelName(logging.INFO, "STATUS")

if you want to change something on your logger, perfectly fine, but do not override global logging module.

Steps to reproduce the problem

N/A

Sysinfo

N/A

Relevant console log

N/A

Additional information

N/A

[Gourieff]

My friend, be so kind, a little more respect when you want to convey some claims
No need for instructive notes :)

[vladmandic]

I have full respect, but I don't understand why would you close issue and move it to discussions.

This is an issue as it breaks core app logger.

[Gourieff]

I heard you, no need to repeat
It will be changed in future updates
There's no Issue here, because nothing is broken and everything works ok, it's just a label of logging level, it affects nothing, no need to get so worked up about it

[vladmandic]

FYI, it's an issue because log parsers expect specific format that this breaks. And that is common in large service provider installs. I wouldn't raise an issue if it was just cosmetic.

[vladmandic]

i see you closed the issue with a commit.
note that logger levels are standardized for the better part of several decades. if you really want to have a custom level, i'd suggest to use NOTICE as its unused by python logger and its a POSIX standard so its not unexpected by anyone attempting to parse logs.

[Gourieff]

Ok, thanks for the advice