rsnapshot.conf should include default parameters for lvm integration

Bug #713464 reported by AmenophisIII
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
rsnapshot (Ubuntu)
Fix Released
Wishlist
Unassigned

Bug Description

Binary package hint: rsnapshot

i was pleased to discover that rsnapshot natively supports lvm snapshots since 1.3.1.
the default config file was not adapted to reflect that change.
the parameters are:
linux_lvm_cmd_lvcreate
linux_lvm_cmd_lvremove
linux_lvm_cmd_mount
linux_lvm_cmd_umount

linux_lvm_snapshotsize
linux_lvm_snapshotname
linux_lvm_vgpath
linux_lvm_mountpath

also the config uses the deprecated 'interval' instead of 'retain'.

 (i did not check if/notice other new features/parameters were introduced without a trace in the config file.)
i'm using 1.3.1-1 on maverick at the moment.

Tags: patch
Revision history for this message
AmenophisIII (amenophisiii) wrote :
Revision history for this message
AmenophisIII (amenophisiii) wrote :

woops the last "+" should probably be one paragraph further up, but i guess you get the idea ;)

tags: added: patch
Revision history for this message
Andreas Olsson (andol) wrote :

Just contacted the Debian rsnapshot maintainer regarding rsnapshot.conf in general. Will keep this suggestion in mind.

Thanks for highlighting this new rsnapshot feature.

Changed in rsnapshot (Ubuntu):
importance: Undecided → Wishlist
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package rsnapshot - 1.3.1-2

---------------
rsnapshot (1.3.1-2) unstable; urgency=low

  * Reorganize patches:
    Merge 02_default_rsnapshot_conf_manpage and 04_default_backup_dir
      to 01_logfile_rsnapshot_conf, renamed 01_rsnapshot_conf
  * Refresh debian/patches/01_rsnapshot_conf:
    + Change commented path to rsnapshot-diff (Closes: #572751)
    + Replace interval (deprecated alias) with retain
  * debian/patches/08_manpage_hourly_to_daily:
    + Add precision on weekly rotation (Closes: #573253)
  * debian/patches/11_lvm_snapshots: Prepare use of lvm snapshot (LP: #713464)
  * debian/patches/09_strip_backtick: Strip backtick in include_conf
    (Closes: #614167)
  * debian/patches/10_space_destdir: Remove backslash in destpath
    (Closes: #609443)
  * debian/control:
    + Add ssh-client as virtual alternative in Depends
    + Bump Standards-Version 3.9.1 (no changes needed)
    + Bump debhelper version to 8 (debian/compat too)
    + Add lvm2, udev and dh-autoreconf as Build-Depends-Indep
      (path detection of binaries)
  * Switch to dpkg-source 3.0 (quilt) format
    + Remove {un,}patch, quilt inclusion in debian/rules
    + Delete unnecessary debian/README.source
    + Remove Build-Depends on quilt
  * debian/rules: Change to use more recent template with debhelper 8
  * debian/manpages: Add rsnapshot.1 and rsnapshot-diff.1
  * debian/README.Debian: Delete unnecessary spaces
  * debian/docs: Remove docs/HOWTOs/rsnapshot-HOWTO.en.html
    (appears twice in /usr/share/doc/rsnapshot/{,html})
  * debian/watch: remove unnecessary default comments
 -- Ubuntu Archive Auto-Sync <email address hidden> Sat, 30 Apr 2011 13:40:19 +0000

Changed in rsnapshot (Ubuntu):
status: New → 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.