Seekfried / greedybot-d2irc

A pickupbot for managing Xonotic pickup games, that also syncs messages between Discord and IRC.
MIT License
2 stars 2 forks source link

Write warning and errors to a log file #25

Closed Seekfried closed 1 month ago

Seekfried commented 1 month ago

If it will be deployed as a container, it's better standard output.

nauar commented 1 month ago

Tasks:

nauar commented 1 month ago

Added timestamp and colors to the logs:

image

The default log level now can be changed at utils.py.

nauar commented 1 month ago

Pending to decide:

  1. With this new format, should we still move the log level to WARNING or keep it at INFO?
  2. Should we keep a streaming log? If it runs in a container, it makes sense to keep it.
Seekfried commented 1 month ago

I really like the new format.

  1. I would say keep it INFO until version 1.0 then WARNING
  2. With streaming log you mean the bridged messages?
nauar commented 1 month ago

I would say keep it INFO until version 1.0 then WARNING

Agreed!

With streaming log you mean the bridged messages?

I mean keeping the logs in stdout and stderr channels instead of writing them in files. It's the de facto standard in containers. In complex setups, you can put an EFK or ELK stack (for instance) to manage the logs as you want (not needed here, though).

Seekfried commented 1 month ago

If that is the best for container, then I will agree.