slapd.postinst output doesn't mention configuration conversion step

Bug #538848 reported by Nathan Stratton Treadway on 2010-03-14
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
openldap (Ubuntu)
Wishlist
Unassigned
Lucid
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.)

Related branches

Attaching corrected patch file.

summary: - slapd.postinst doesn't mention configuration conversion step
+ slapd.postinst output doesn't mention configuration conversion step
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
tags: added: patch
Simon Olofsson (simono) 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.

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

Other bug subscribers