Closed tahini closed 1 month ago
I added the possibility to configure the log files and whether to run in debug in the config file by default, with always the possibility to override in individual calls, but we don't support it yet.
@greenscientist fwiw we can now allow more log files and of larger sizes in our instances and we could support running in debug in some instances if required, batch and not batch separately.
Kinda wondering if we should just strip all this out and let the standard OS tools handle log management.
@tahini, as-tu regardé si il existait des modules standard pour gérer tout ça ?
@tahini, as-tu regardé si il existait des modules standard pour gérer tout ça ?
C'est ce que winston fait, on peut envoyer les logs où on veut. Ceci a l'avantage que qqn qui fait rouler Transition peut contrôler le logging. Si on laisse l'OS faire, il faudrait documenter la procédure ou configurer un autre outil séparément.
@greenscientist I rebase and updated to address all your comments. With this, next time we need to investigate some trRouting related errors, we'll be ready to configure debug and logging in our instances.
Use named objects instead of ordinal parameters for the numerous function parameters. This makes a call more readable, make it easier to fine-tune specific parameters while keeping the default values for others, and will make adding yet more parameters simpler.
Let
ProcessManager
support log file count and size as extra parameterLet
TrRoutingProcessManager
support log file count and size as extra parameter.startBatch
now also supports thedebug
option.Add tests for the new functionalities