/juju

Decrease the log level for uniter.operation executor steps. #3922

Merged
merged 1 commit into from Dec 8, 2015

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@jameinel @jujubot
@jameinel
Owner

jameinel commented Dec 8, 2015

This addresses bug #1524089. We were logging all of the uniter steps at INFO level, but that gets overly chatty when you have lots of units. Generally these are only interesting when debugging something, which makes DEBUG a good level to use.

(Review request: http://reviews.vapour.ws/r/3343/)
@jameinel
Owner

jameinel commented Dec 8, 2015

$$merge$$
@jujubot
Contributor

jujubot commented Dec 8, 2015

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

jujubot added a commit that referenced this pull request Dec 8, 2015

@jujubot
Merge pull request #3922 from jameinel/log-level-1524089 
Decrease the log level for uniter.operation executor steps.

This addresses bug #1524089. We were logging all of the uniter steps at INFO level, but that gets overly chatty when you have lots of units. Generally these are only interesting when debugging something, which makes DEBUG a good level to use.


(Review request: http://reviews.vapour.ws/r/3343/)
6423414

@jujubot jujubot merged commit 6423414 into juju:master Dec 8, 2015

@jameinel jameinel deleted the jameinel:log-level-1524089 branch Dec 13, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment