use different colors to distinguish logs

[EdwardLi-coder]

Description

Use different colors to distinguish logs

Related Issues

[LOGGING-IMPR] Use different colors to distinguish logs #2327

Checklist

• Is this code covered by new or existing unit tests or integration tests?
• Did you run make unit_testing and make integration-testing successfully?
• Do new classes, functions, methods and parameters all have docstrings?
• Were existing docstrings updated, if necessary?
• Was external documentation updated, if necessary?

Additional Notes or Comments

[blythed]

This should be configurable. We had this before, and it made life difficult with our ray deployments.

[EdwardLi-coder]

Hi @blythed.I update it. please review it. Thank you.

[blythed]

Please address the comment.

[blythed]

I wouldn't do it like this.
I would add a variable to the config colorize and then in the logging module do one or the other.

[blythed]

config.py

colorize: bool = True

logging.py

if CFG.logging.colorize:
    message = (
        "<green>{time:YYYY-MMM-DD HH:mm:ss.SS}</green>"
        "| <level>{level: <8}</level> "
        "| <cyan>{extra[hostname]: <8}</cyan>"
        "| <cyan>{name}</cyan>:<cyan>{line: <4}</cyan> "
        f"| <level>{message}</level>"
    )
else:
    message = (
        "{time:YYYY-MMM-DD HH:mm:ss.SS}"
        "| {level: <8} "
        "| {name}:{line: <4} "
        f"| {message}"
    )

[EdwardLi-coder]

config.py

colorize: bool = True

logging.py

if CFG.logging.colorize:
    message = (
        "<green>{time:YYYY-MMM-DD HH:mm:ss.SS}</green>"
        "| <level>{level: <8}</level> "
        "| <cyan>{extra[hostname]: <8}</cyan>"
        "| <cyan>{name}</cyan>:<cyan>{line: <4}</cyan> "
        f"| <level>{message}</level>"
    )
else:
    message = (
        "{time:YYYY-MMM-DD HH:mm:ss.SS}"
        "| {level: <8} "
        "| {name}:{line: <4} "
        f"| {message}"
    )

Ok,I have changed

[blythed]

Thanks @EdwardLi-coder!