Code review comment for lp:~allenap/maas/cluster-worker-logging

Revision history for this message
Raphaƫl Badin (rvb) wrote :

Looks good.

[0]

101 + setup_logging_subsystem(loglevel="INFO", logfile=get_maas_celery_log())

I think you should make the log level a configuration option in the celery config file.

review: Approve

« Back to merge proposal