Fix logic error introduced in previous commit.
Along the way make some efficiency improvements. Submitted by: jilles Approved by: kib (mentor) MFC after: 3 days
This commit is contained in:
parent
f5f3977b8d
commit
9283f726be
Notes:
svn2git
2020-12-20 02:59:44 +00:00
svn path=/head/; revision=217822
@ -5,6 +5,10 @@
|
||||
|
||||
# If there is a global system configuration file, suck it in.
|
||||
#
|
||||
|
||||
newline="
|
||||
" # A single newline
|
||||
|
||||
if [ -r /etc/defaults/periodic.conf ]
|
||||
then
|
||||
. /etc/defaults/periodic.conf
|
||||
@ -24,17 +28,19 @@ case "$daily_scrub_zfs_enable" in
|
||||
|
||||
for pool in ${daily_scrub_zfs_pools}; do
|
||||
# sanity check
|
||||
_status=$(zpool list ${pool} | sed -n -e '$p')
|
||||
_status=$(zpool list "${pool}" 2> /dev/null)
|
||||
if [ $? -ne 0 ]; then
|
||||
echo " WARNING: pool '${pool}' specified in"
|
||||
echo " '/etc/periodic.conf:daily_scrub_zfs_pools'"
|
||||
echo " does not exist"
|
||||
continue
|
||||
fi
|
||||
if echo ${_status} | grep -q FAULTED; then
|
||||
_status=${_status##*$newline}
|
||||
case ${_status} in
|
||||
*FAULTED*)
|
||||
echo "Skipping faulted pool: ${pool}"
|
||||
continue
|
||||
fi
|
||||
continue ;;
|
||||
esac
|
||||
|
||||
# successful only if there is at least one pool to scrub
|
||||
rc=0
|
||||
|
Loading…
Reference in New Issue
Block a user