Closed aitorarjona closed 2 years ago
@aitorarjona The idea was to avoid the feeling that the job is stuck somewhere when logs are not shown, specially for long-running workflows. So, what do you think if at least we show the log once or twice a minute even if it is the same as before?
@aitorarjona ready?
@JosepSampe If you approve this change, then it's ready 🙌 - What do you think, is 60s enough? Should this value be a configurable parameter?
Maybe you can put a global var in monitor.py called LOG_FREQUENCY or so
This patch aims to reduce logging verbosity for the monitor with debug mode. Currently, it prints pending/running/done futures every update. It could only print the log when there is a future status change (from running to done or pending to running) in order to reduce the number of logs.
Before:
After:
Developer's Certificate of Origin 1.1