Merge lp://staging/~dave-cheney/juju-core/118-log-mongo-to-syslog into lp://staging/~juju/juju-core/trunk

Proposed by Dave Cheney
Status: Merged
Approved by: William Reade
Approved revision: no longer in the source branch.
Merged at revision: 1193
Proposed branch: lp://staging/~dave-cheney/juju-core/118-log-mongo-to-syslog
Merge into: lp://staging/~juju/juju-core/trunk
Diff against target: 33 lines (+3/-2)
2 files modified
environs/cloudinit/cloudinit.go (+1/-0)
environs/cloudinit/cloudinit_test.go (+2/-2)
To merge this branch: bzr merge lp://staging/~dave-cheney/juju-core/118-log-mongo-to-syslog
Reviewer Review Type Date Requested Status
Juju Engineering Pending
Review via email: mp+160546@code.staging.launchpad.net

Description of the change

environs/cloudinit: send mongodb output to syslog

Updates lp #1172110.

This should be backported to 1.10.

By default mongo sends its logs to stdout, which is connected to a pty by upstart, which is pretty useless on a virtual machine.

https://codereview.appspot.com/8931044/

To post a comment you must log in.
Revision history for this message
John A Meinel (jameinel) wrote :

LGTM

I seem to remember mongod being very chatty. Does this mean all that
chatter is going to end up synchronized to machine/0?

https://codereview.appspot.com/8931044/

Revision history for this message
Dimiter Naydenov (dimitern) wrote :

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches