Merge lp:~allenap/maas/cluster-worker-logging into lp:~maas-committers/maas/trunk
Proposed by
Gavin Panella
Status: | Merged |
---|---|
Approved by: | Gavin Panella |
Approved revision: | no longer in the source branch. |
Merged at revision: | 1293 |
Proposed branch: | lp:~allenap/maas/cluster-worker-logging |
Merge into: | lp:~maas-committers/maas/trunk |
Diff against target: |
228 lines (+32/-55) 9 files modified
etc/celeryconfig_common.py (+4/-3) etc/democeleryconfig.py (+1/-1) etc/democeleryconfig_cluster.py (+2/-2) src/provisioningserver/boot_images.py (+4/-1) src/provisioningserver/dhcp/leases.py (+5/-1) src/provisioningserver/logging.py (+0/-22) src/provisioningserver/start_cluster_controller.py (+9/-20) src/provisioningserver/tags.py (+6/-5) src/provisioningserver/tests/test_start_cluster_controller.py (+1/-0) |
To merge this branch: | bzr merge lp:~allenap/maas/cluster-worker-logging |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
John A Meinel (community) | Approve | ||
Raphaël Badin (community) | Approve | ||
Review via email: mp+130576@code.launchpad.net |
Commit message
Capture log messages emitted during cluster worker start-up.
Previously these messages were being lost, so debugging the start-up of a cluster worker was difficult.
To post a comment you must log in.
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.