openstack/keystone
Code Issues Proposed changes

Update keystone wsgi httpd script for oslo logging

Browse Source 
When updating to the oslo logging module, it appears that this script
was missed, and erroneously still references .DEBUG, which is not
available in the oslo logging module.  This commit corrects the issue
and updates the reference to oslo log module to not be aliased to
"logging".

The extra logic to set the debug on the instantiated logger is still
needed to ensure all logging is seen (startup config logging, etc).

fixes-bug: 1220012
Change-Id: If5b4f9f28cb9d05f914848cbdb54b10c2c5f3b2d
This commit is contained in:
Morgan Fainberg 2013-09-02 17:07:32 -07:00
parent 372a06236a
commit d98dca9f98
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
httpd/keystone.py
View File

@ -14,6 +14,7 @@
# License for the specific language governing permissions and limitations
# under the License.
import logging
import os
from paste import deploy
@ -31,10 +32,9 @@ gettextutils.install('keystone', lazy=True)
from keystone.common import environment
from keystone import config
from keystone.openstack.common import log as logging
from keystone.openstack.common import log
LOG = logging.getLogger(__name__)
CONF = config.CONF
CONF(project='keystone')
config.setup_logging(CONF)
@ -43,7 +43,7 @@ environment.use_stdlib()
name = os.path.basename(__file__)
if CONF.debug:
CONF.log_opt_values(logging.getLogger(CONF.prog), logging.DEBUG)
CONF.log_opt_values(log.getLogger(CONF.prog), logging.DEBUG)
# NOTE(ldbragst): 'application' is required in this context by WSGI spec.
# The following is a reference to Python Paste Deploy documentation