Re: git: b6e33f0cd536 - main - rc.d/mountcritlocal: Make sure zpools are imported for legacy ZFS
Date: Sun, 15 Jun 2025 14:21:20 UTC
Van: Cy Schubert <cy@FreeBSD.org> Datum: 15 juni 2025 07:08 Aan: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org Onderwerp: git: b6e33f0cd536 - main - rc.d/mountcritlocal: Make sure zpools are imported for legacy ZFS > > > The branch main has been updated by cy: > > URL: https://cgit.FreeBSD.org/src/commit/?id=b6e33f0cd53683e06570606d5bd88fc04a4b76fe > > commit b6e33f0cd53683e06570606d5bd88fc04a4b76fe > Author: Cy Schubert > AuthorDate: 2025-06-14 13:16:16 +0000 > Commit: Cy Schubert > CommitDate: 2025-06-15 04:39:36 +0000 > > rc.d/mountcritlocal: Make sure zpools are imported for legacy ZFS > > Legacy ZFS uses fstab to mount its datasets. In an attempt to fix > another problem 900bc0206348 broke legacy ZFS in fstab(5). This > comit works around the problem by mountcritlocal scanning /etc/fstab > for zfs mountpoint and if any are found invoke /etc/rc.d/zpool start. > > Reviewed by: imp > Differential Revision: https://reviews.freebsd.org/D50844 > --- > libexec/rc/rc.d/mountcritlocal | 9 +++++++++ > 1 file changed, 9 insertions(+) > > diff --git a/libexec/rc/rc.d/mountcritlocal b/libexec/rc/rc.d/mountcritlocal > index f91eaf44457c..6dce672657fc 100755 > --- a/libexec/rc/rc.d/mountcritlocal > +++ b/libexec/rc/rc.d/mountcritlocal > @@ -27,6 +27,15 @@ mountcritlocal_start() > ;; > esac > > + while read a b vfstype rest; do > + if [ "$vfstype" = "zfs" -a "$a" != "#" ]; then > + # zpool is needed for legacy ZFS > + echo 'Importing zpools for legacy ZFS' > + /etc/rc.d/zpool start > + break > + fi > + done < /etc/fstab > + > # Mount everything except nfs filesystems. > startmsg -n 'Mounting local filesystems:' > mount_excludes='no' > > > > > Does this mean that a comment # must always be folllowed by a whitespace? Regards, Ronald