Bug 215828

Summary: sysutils/freebsd-snapshot limits maximum number of zfs snapshots to 20
Product: Ports & Packages Reporter: hostmaster
Component: Individual Port(s)Assignee: Koichiro Iwao <meta>
Status: Closed FIXED    
Severity: Affects Some People CC: gtodd, hostmaster, meta, w.schwarzenfeld
Priority: --- Flags: hostmaster: maintainer-feedback+
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
for "periodic-snapshot" - allows to zfs filesystems to have unlimited snapshots none

Description hostmaster 2017-01-06 13:43:24 UTC
Created attachment 178574 [details]
for "periodic-snapshot" - allows to zfs filesystems to have unlimited snapshots

the periodic-snapshot supplied with the sysutils/freebsd-snapshot port limits the total number of snapshots per file system to 20, which is correct for ufs filesystems but does not apply to zfs which can have 2^64

A patch is included which fixes this small flaw in an otherwise very handy utility
Comment 1 Walter Schwarzenfeld 2018-01-14 05:26:52 UTC
Maintainer feedback?
Comment 2 Koichiro Iwao freebsd_committer freebsd_triage 2019-02-26 04:44:53 UTC
LGTM. Maintainer timeout.
Comment 3 commit-hook freebsd_committer freebsd_triage 2019-02-26 04:52:15 UTC
A commit references this bug:

Author: meta
Date: Tue Feb 26 04:51:49 UTC 2019
New revision: 493933
URL: https://svnweb.freebsd.org/changeset/ports/493933

Log:
  sysutils/freebsd-snapshot: allows to zfs filesystems to have unlimited snapshots

  The periodic-snapshot supplied with the sysutils/freebsd-snapshot port
  limits the total number of snapshots per file system to 20, which is
  correct for ufs filesystems but does not apply to zfs which can have 2^64.

  PR:		215828
  Submitted by:	hostmaster@GTS.NET
  Approved by:	maintainer timeout

Changes:
  head/sysutils/freebsd-snapshot/Makefile
  head/sysutils/freebsd-snapshot/files/patch-allow-unlimited-snapshots-on-zfs
Comment 4 Koichiro Iwao freebsd_committer freebsd_triage 2019-02-26 04:53:12 UTC
Committed, thanks!