Created attachment 180615 [details] 1-line patch to line 73 of 800.scrub-zfs to add " -v -70M" The enclosed patch improves the time-since-last-scrub calculation. In general, for, say, the default interval of 35 days, the old code treats 34d 23h 59m 59s as 34 days and defers scrubbing to the next day, making the actual interval 36 days. In addition, there are two typical causes of being a few seconds (or minutes) shy of a full day: 1) a few seconds pass between the date test and the actual starting of a scrub; and 2) when 410.pkg-audit is run before 800.scrub-zfs, a random interval of up to 1 hour is introduced. Adding " -v 70M" to the date command effectively rounds up anything within 70 minutes of a full day to the next full day. This affects everyone using periodic daily/800.scrub-zfs. -WBE
Typo in the description. The subject line and the patch were correct, but "Adding " -v 70M" to the date command..." should have said "Adding " -v -70M" to the date command...". -WBE
A commit references this bug: Author: mr Date: Sat May 20 16:47:00 UTC 2017 New revision: 318577 URL: https://svnweb.freebsd.org/changeset/base/318577 Log: Improve time-since-last-scrub calculation. This can be needed to compensate anticongestion delays in 410.pkg-audit or 480.leapfile-ntpd. PR: 217622 Submitted by: wbe@psr.com MFC after: 2 weeks Changes: head/etc/periodic/daily/800.scrub-zfs
batch change of PRs untouched in 2018 marked "in progress" back to open.
A commit references this bug: Author: mr Date: Mon Dec 31 11:05:39 UTC 2018 New revision: 342647 URL: https://svnweb.freebsd.org/changeset/base/342647 Log: Merge forgotten r318577 to close PR. PR: 217622 Changes: stable/11/etc/periodic/daily/800.scrub-zfs