jewel: common: Log.cc: Assign LOG_INFO priority to syslog calls #11231

Merged
merged 1 commit into from Oct 14, 2016

Projects

None yet

3 participants

@badone
Contributor
badone commented Sep 27, 2016 edited
@badone badone log: Log.cc: Assign LOG_INFO priority to syslog calls
LOG_DEBUG prio messages are not logged by a default syslog
configuration so log at LOG_INFO instead.

Fixes: http://tracker.ceph.com/issues/15808
Signed-off-by: Brad Hubbard <bhubbard@redhat.com>
(cherry picked from commit 3ab5a66)
9cb45e1
@badone badone added this to the jewel milestone Sep 27, 2016
@dachary dachary self-assigned this Sep 27, 2016
@dachary dachary added bug fix and removed needs-review labels Sep 27, 2016
@dachary
Member
dachary commented Sep 28, 2016

LGTM

@dachary dachary changed the title from log: Log.cc: Assign LOG_INFO priority to syslog calls to jewel: common: Log.cc: Assign LOG_INFO priority to syslog calls Oct 10, 2016
@dachary dachary merged commit 67d0ded into ceph:jewel Oct 14, 2016

2 checks passed

Signed-off-by all commits in this PR are signed
Details
default Build finished.
Details
@dachary
Member
dachary commented Oct 14, 2016

It passed the rados (http://tracker.ceph.com/issues/17487#note-12), upgrade/jewel-x and upgrade/hammer-x (http://tracker.ceph.com/issues/17487#note-15) suites (except for two bugs that are, I believe, unrelated)

@badone badone deleted the badone:wip-17376 branch Oct 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment