Ubuntu
openldap package

slapd.postinst output doesn't mention configuration conversion step

Bug #538848 reported by Nathan Stratton Treadway
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
openldap (Ubuntu)
Fix Released
Wishlist
Unassigned
Lucid
Fix Released
Wishlist
Unassigned

Bug Description

When I upgrade the slapd package from Hardy to Lucid, the slapd.postinst script automatically attempts to convert from the slapd.conf file to the slapd.d configuration directory. If that conversion attempt fails, a notification message is printed ("Migrating slapd.conf file (/etc/ldap/slapd.conf) to slapd.d failed with the following error while running slaptest:....").

And the postinst script prints other notices about its activities: " Backing up /etc/ldap/slapd.conf....", " Moving old database directories....", "Loading from /var/backups/....", etc.

However, if the configuation-conversion succeeds, the script currently does not print any message at all to notifiy the user that the conversion has taken place.

Attached is a one-line patch to generate such a message. (The patch is pretty simple, but I don't currently have an environment in which I can actually test it.)

Revision history for this message
Nathan Stratton Treadway (nathanst) wrote :

Attaching corrected patch file.

Nathan Stratton Treadway (nathanst)
summary: - slapd.postinst doesn't mention configuration conversion step
+ slapd.postinst output doesn't mention configuration conversion step
Revision history for this message
Chuck Short (zulcss) wrote :

Thanks for the patch. Ill take a look at this for lucid.

Regards
chuck

Changed in openldap (Ubuntu):
importance: Undecided → Wishlist
status: New → Confirmed
Brian Murray (brian-murray)
tags: added: patch
Revision history for this message
No (11111nonononono-deactivatedaccount-deactivatedaccount-deactivatedaccount) wrote :

Nathan, thanks for the patch. Looks good!
I've update the package and it's building in my PPA right now:
https://edge.launchpad.net/~simono/+archive/ppa
Please merge the above branch.

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

This bug was fixed in the package openldap - 2.4.21-0ubuntu4

---------------
openldap (2.4.21-0ubuntu4) lucid; urgency=low

  [ Simon Olofsson ]
  * debian/slapd.postinst:
    - Show a message after successful migration (LP: #538848)

  [ Jorgen Rosink ]
  * debian/slapd.init: add simple status checking with LSB compatible exit
    codes (LP: #562377)
  * debian/slapd.init.ldif:
    - remove admin user in default config database (LP: #556176)
    - in default config, add olcAccess entries giving access to controls
      available and cn=subschema (LP: #427842)

  [ Scott Moser ]
  * debian/slapd.scripts-common: Do not create /nonexistent directory
     for openldap user's home (LP: #556176)
  * debian/slapd.postinst: fix cn=config olcAccess migration (LP: #559070)
 -- Scott Moser <email address hidden> Mon, 12 Apr 2010 16:16:47 -0400

Changed in openldap (Ubuntu Lucid):
status: Confirmed → Fix Released
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.