On Sun, Dec 31, 2023 at 03:25:41PM -0500, Neal Gompa wrote: > On Sun, Dec 31, 2023 at 2:48 PM Darrick J. Wong <djwong@xxxxxxxxxx> wrote: > > > > Hi all, > > > > This series fixes deficiencies in the systemd services that were created > > to manage background scans. First, improve the debian packaging so that > > services get installed at package install time. Next, fix copyright and > > spdx header omissions. > > > > Finally, fix bugs in the mailer scripts so that scrub failures are > > reported effectively. Finally, fix xfs_scrub_all to deal with systemd > > restarts causing it to think that a scrub has finished before the > > service actually finishes. > > > > If you're going to start using this code, I strongly recommend pulling > > from my git trees, which are linked below. > > > > This has been running on the djcloud for months with no problems. Enjoy! > > Comments and questions are, as always, welcome. > > > > --D > > > > xfsprogs git tree: > > https://git.kernel.org/cgit/linux/kernel/git/djwong/xfsprogs-dev.git/log/?h=scrub-service-fixes > > --- > > debian/rules | 1 + > > include/builddefs.in | 2 +- > > scrub/Makefile | 26 ++++++++++++++------ > > scrub/xfs_scrub@xxxxxxxxxxx | 6 ++--- > > scrub/xfs_scrub_all.in | 49 ++++++++++++++++---------------------- > > scrub/xfs_scrub_fail.in | 12 ++++++++- > > scrub/xfs_scrub_fail@xxxxxxxxxxx | 4 ++- > > 7 files changed, 55 insertions(+), 45 deletions(-) > > rename scrub/{xfs_scrub_fail => xfs_scrub_fail.in} (62%) > > > > In your Makefile changes, you should be able to drop > PKG_LIB_SCRIPT_DIR entirely from your Makefiles since it should be > unused now, can you fold that into > https://git.kernel.org/pub/scm/linux/kernel/git/djwong/xfsprogs-dev.git/commit/?h=scrub-service-fixes&id=1e0dce5c54270f1813f5661c266989917f08baf8 > ? Already done in: https://lore.kernel.org/linux-xfs/170405001964.1800712.10514067731814883862.stgit@frogsfrogsfrogs/ Sorry I forgot to cc you there. --D > > > -- > 真実はいつも一つ!/ Always, there's only one truth! >