Pecan logging config duplicates oslo logging config

[Mark McLoughlin]

Playing with Ironic, I was confused to see duplicate log entries and I also see them in Ceilometer:

  $> ceilometer-api --debug
  ...
  2013-05-28 08:10:46,535 DEBUG [ceilometer.storage][MainThread] looking for 'sqlite' driver in 'ceilometer.storage'
  2013-05-28 08:10:46 DEBUG [ceilometer.storage] looking for 'sqlite' driver in 'ceilometer.storage'
  2013-05-28 08:10:46,540 INFO [ceilometer.storage.impl_sqlalchemy][MainThread] connecting to sqlite:////ceilometer/openstack/common/db/ceilometer.sqlite
  2013-05-28 08:10:46 INFO [ceilometer.storage.impl_sqlalchemy] connecting to sqlite:////ceilometer/openstack/common/db/ceilometer.sqlite

The issue here is that we're using pecan to configure logging for the 'ceilometer' and 'ironic' loggers, even though we use oslo logging to configure the root logger

Removing the pecan logging configuration completely fixes the issue for me and, conceptually, I don't see why we need both ... oslo logging configuration should do everything we need?

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/30697

[Mark McLoughlin (markmc)]

ceilometer: https://review.openstack.org/30695

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/30695
Committed: http://github.com/openstack/ceilometer/commit/e6a083fc5acb6f3ad03cc1df0943e4e746629128
Submitter: Jenkins
Branch: master

commit e6a083fc5acb6f3ad03cc1df0943e4e746629128
Author: Mark McLoughlin <email address hidden>
Date: Tue May 28 11:18:13 2013 +0100

    Don't use pecan to configure logging

    Fixes bug #1184941

    openstack.common.log gives a mechanism to configure logging and, by
    default, we will set up a handler on root logger. Using pecan to also
    configure a handler on the 'ironic' logger means every log message gets
    logged to console twice.

    It seems that the pecan logging configuration stuff is redundant with
    our existing logging configuration infrastructure.

    Change-Id: Iec0630b0e842e3b6a90021bf552d9b857c5a0552

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/30697
Committed: http://github.com/openstack/ironic/commit/46b9a88ba110e5489749327f6dd381a8af695c3e
Submitter: Jenkins
Branch: master

commit 46b9a88ba110e5489749327f6dd381a8af695c3e
Author: Mark McLoughlin <email address hidden>
Date: Mon May 27 19:42:57 2013 +0100

    Don't use pecan to configure logging

    Fixes bug #1184941

    openstack.common.log gives a mechanism to configure logging and, by
    default, we will set up a handler on root logger. Using pecan to also
    configure a handler on the 'ironic' logger means every log message gets
    logged to console twice.

    It seems that the pecan logging configuration stuff is redundant with
    our existing logging configuration infrastructure.

    Change-Id: Iec0630b0e842e3b6a90021bf552d9b857c5a0552

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/grizzly
Review: https://review.openstack.org/31196

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/31196
Committed: http://github.com/openstack/ceilometer/commit/ff94b0e6ed99469fec6830b0fb5c0aa3ee667195
Submitter: Jenkins
Branch: stable/grizzly

commit ff94b0e6ed99469fec6830b0fb5c0aa3ee667195
Author: Mark McLoughlin <email address hidden>
Date: Tue May 28 11:18:13 2013 +0100

    Don't use pecan to configure logging

    Fixes bug #1184941

    openstack.common.log gives a mechanism to configure logging and, by
    default, we will set up a handler on root logger. Using pecan to also
    configure a handler on the 'ironic' logger means every log message gets
    logged to console twice.

    It seems that the pecan logging configuration stuff is redundant with
    our existing logging configuration infrastructure.

    Change-Id: Iec0630b0e842e3b6a90021bf552d9b857c5a0552
    (cherry picked from commit e6a083fc5acb6f3ad03cc1df0943e4e746629128)

[Eoghan Glynn (eglynn)]

Backport patch: https://review.openstack.org/31196

Merged as: https://github.com/openstack/ceilometer/commit/ff94b0e6ed99469fec6830b0fb5c0aa3ee667195