slapd.postinst should put all backed-up items together in one place under /var/backups

Bug #571498 reported by Nathan Stratton Treadway
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
openldap (Ubuntu)
Confirmed
Wishlist
Ryan Tandy

Bug Description

Currently the slapd.postinst script uses /var/backups/slapd-<old-package-version>/ to store both the backup copy of $SLAPD_CONF and the "slapcat"-generated .ldif file. However, if there is a need to move the BDB files out of the way, they are instead moved to separate <rootdn>-<old-packge-version>.ldapdb destination directories found directly under /var/backups/ .

Assuming the move_incompatible_databases_away logic is kept in post-Lucid, it might be worth changing the destination path so that it keeps the BDB files together with the other backups under /var/backups/slapd-<old-package-version>/ .

(I think that could be accomplished by simply changing the hard-coded "/var/backups" strings found in the compute_backup_path and move_incompatible_databases_away functions to instead call database_dumping_destdir, similar to the call in backup_config_once .)

description: updated
Revision history for this message
C de-Avillez (hggdh2) wrote :

I think this makes sense. There is already a backup place, and ideally packages should backup things there. Marking Confirmed/WishList.

Changed in openldap (Ubuntu):
importance: Undecided → Wishlist
status: New → Confirmed
Ryan Tandy (rtandy)
Changed in openldap (Ubuntu):
assignee: nobody → Ryan Tandy (rtandy)
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.