keystone logrotate has output when restarting keystone

Bug #1029766 reported by Sam Morrison
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
keystone (Ubuntu)
Fix Released
Medium
Unassigned

Bug Description

/etc/cron.daily/logrotate:
keystone start/running, process 3663

Need to pipe the restart keystone command to dev null

If someone could show me how to provide a patch I can do this. I usually do it all through gerrit but never done a patch to the specific package before

Revision history for this message
Clint Byrum (clint-fewbar) wrote :

For the Ubuntu packaging, you can

bzr branch lp:~ubuntu-server-dev/keystone/essex

edit the file (debian/keystone.logrotate)

Then propose merging.

I think.. not entirely sure if thats the workflow the team uses, but it would be the usual Ubuntu packaging workflow.

Anyway, confirmed that this bug exists and needs fixing.

Changed in keystone (Ubuntu):
status: New → Triaged
importance: Undecided → Medium
Revision history for this message
Adam Gandelman (gandelman-a) wrote :

Hey Sam-

Thanks for the patch. I've merged the change into our -proposed packaging branches, where the change will be incorporated into our CI builds (available in ppa:openstack-ubuntu-testing/folsom-trunk-testing) and be merged to ~ubuntu-server-dev branch (and Ubuntu archive) with the next upload of Keystone.

Note, I added an entry to debian/changelog. In the future, please include an update there. See the first section of https://wiki.ubuntu.com/DistributedDevelopment/Documentation/SeekingSponsorship to see how you can easily do this with dch and bzr.

Thanks!

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package keystone - 2012.2~f3-0ubuntu1

---------------
keystone (2012.2~f3-0ubuntu1) quantal; urgency=low

  [ Adam Gandelman ]
  * debian/{keystone.conf, rules, keytone.install}: Install patched
    keystone.conf.sample configured for SQL backends to /etc/keystone,
    no longer maintain our own version in packaging. (LP: #1031012)
  * debian/patches/sql_connection.patch: Refreshed against current
    keystone.conf.sample.
  * debian/rules: Use debian/tests as HOME to avoid test suite FTFBS.

  [ Sam Morrison ]
  * debian/keystone.logrotate: Sent output of keystone restart in logrotate
    to /dev/null. (LP: #1029766)

  [Chuck Short]
  * New upstream version.
 -- Chuck Short <email address hidden> Thu, 16 Aug 2012 13:59:29 -0500

Changed in keystone (Ubuntu):
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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