Ubuntu
keystone package

keystone is not logging

Bug #959610 reported by Adam Gandelman
22
This bug affects 4 people
Affects Status Importance Assigned to Milestone
OpenStack Identity (keystone)
Fix Released
Low
Dolph Mathews
OpenStack Identity (keystone) 2012.1 "essex"
keystone (Ubuntu)
Fix Released
Low
Unassigned

Bug Description

Using most recent Ubuntu packages, the keystone-all deamon does not log anything to its default logfile /var/log/keystone/keystone.log

The current upstream source ships a logging.conf.sample that seems to be lifted directly from the glance project. As a work around, the ubuntu packages install a different logging.conf that was copied from the pre-keystone light branch merge, though it appears this configuration does not generate any log output either.

If it Keystone supports logging, the upstream logging.conf.sample needs to be updated to contain a working example configuration and we can use that as the Ubuntu conffile instead of maintaining our own. If Keystone does not yet provide its own logging facilities, we can modify the upstart job to redirect stdout and stderr to the appropriate log files.

Dolph Mathews (dolph)
Changed in keystone:
assignee: nobody → Dolph Mathews (dolph)
milestone: none → essex-rc1
status: New → In Progress
importance: Undecided → Low
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to keystone (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to keystone (master)

Reviewed: https://review.openstack.org/5528
Committed: http://github.com/openstack/keystone/commit/ee577163e32b88ca1345124c96ae3113d9a5ccdd
Submitter: Jenkins
Branch: master

commit ee577163e32b88ca1345124c96ae3113d9a5ccdd
Author: Dolph Mathews <email address hidden>
Date: Mon Mar 19 14:49:18 2012 -0500

    Improved file logging example (bug 959610)

    - Root logger w/ file handler will log WARNING, ERROR, CRITICAL by default

    Change-Id: I36cd07cec85712640daa013563401a3bc52f290c

Changed in keystone:
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in keystone (Ubuntu):
status: New → Confirmed
Revision history for this message
Vincent Ladeuil (vila) wrote :

For the record, trying to apply the moral equivalent of the proposed patch to my config files I still don't get anything written to the log file :-/

James Page (james-page)
Changed in keystone (Ubuntu):
importance: Undecided → Low
Revision history for this message
Adam Gandelman (gandelman-a) wrote :

@Vincent

Here is the logging conf sitting in the -proposed branch linked here, to be uploaded with rc1: http://paste.ubuntu.com/895301/

Note that the default log level doesn't produce much output, changing

[logger_root]
level=WARNING
handlers=file

to

[logger_root]
level=DEBUG
handlers=file

produces output in /var/log/keystone/keystone.log that is comparable to what you'd get running it in non-daemon mode.

Revision history for this message
Vincent Ladeuil (vila) wrote :

@Adam: Confirmed, using DEBUG produces output \o/

But DEBUG may not be appropriate in the long run ?

Anyway, I'm out of trouble.

Thierry Carrez (ttx)
Changed in keystone:
status: Fix Committed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package keystone - 2012.1~rc1-0ubuntu1

---------------
keystone (2012.1~rc1-0ubuntu1) precise; urgency=low

  [Chuck Short]
  * New usptream version.
  * debian/control: Add python-iso8601 as a depends.
  * debian/patches/fix-ubuntu-tests.patch: Disable git checkout on some
    of the tests.
  * dropped swift as a depends.

  [Adam Gandelman]
  * debian/patches/sql_connection.patch: Refresh
  * debian/logging.conf: Update and enable file logging (LP: #959610)
  * debian/keystone.prerm: Only attempt to cleanup database if it was
    configured during installation. (LP: #948719)
  * debian/rules: Fix doc builds + clean (LP: #956019)
  * debian/control: Add python-{nova, swift} as Build-Depends, required
    for doc building
  * debian/rules, debian/tests/test_overrides.conf: Setup a proper environment
    for unit testing
 -- Chuck Short <email address hidden> Mon, 26 Mar 2012 13:41:45 -0400

Changed in keystone (Ubuntu):
status: Confirmed → Fix Released
Thierry Carrez (ttx)
Changed in keystone:
milestone: essex-rc1 → 2012.1
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.