-
Notifications
You must be signed in to change notification settings - Fork 94
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Handle suite out and err files like state dumps? #846
Comments
[meeting] @oliver-sanders to investigate |
A couple of quick questions to gauge opinion on. @cylc/core
|
@oliver-sanders - are you suggesting using a single log file for all, or connecting stdout and stderr to different files handled by the logging module? (in the latter case, can the roll-over of multiple logs be synchronised?). I'm open to any suggestions as the current situation isn't very clean. However, @matthewrmshin seems to strongly dislike the Python logging module?? |
@hjoliver I was suggesting writing stdout and stderr messages through the logging module to their respective files (not merging them into one). If it was done like this synchronisation should be relatively straight forward to implement by extending the functionality of the logging module. |
OK, sounds fine to me if others agree. |
The logging module is appropriate for logging suite activity. (I do have a dislike of it, because it is too flexible for the things we don't normally need, but incredibly rigid for the things we want to be flexible.) |
ref #828 (comment)
The text was updated successfully, but these errors were encountered: