keystone logrotate has output when restarting keystone

Bug #1029766 reported by Sam Morrison on 2012-07-27
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
keystone (Ubuntu)
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

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
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!

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  Edit
Everyone can see this information.

Other bug subscribers