[LU-678] conf_sanity test_24a, test_24b check for the existance of a device before setting the variable that points to it.

• Status: Closed
• Project: Lustre
• Type: Bug
• Priority: Minor
• Reporter: Chris Gearing (Inactive)
• Assignee: Chris Gearing (Inactive)
• Resolution: Incomplete
• Severity: 3

Description

conf_sanity test_24a, test_24b check for the existance of a device before setting the variable that points to it.

As an example looking at test_24a it checks for the existance of fs2ost_DEV but may actually use MDSDEV_2 if it's not defined. This means the ability to not define it doesn't really exist. need to more the test for existance to after the setting of fs2mdsdev etc.

if [ -z "$fs2ost_DEV" -o -z "$fs2mds_DEV" ]; then
is_blkdev $SINGLEMDS $MDSDEV && \
skip_env "mixed loopback and real device not working" && return
fi

[ -n "$ost1_HOST" ] && fs2ost_HOST=$ost1_HOST

local fs2mdsdev=${fs2mds_DEV:-${MDSDEV}_2}
local fs2ostdev=${fs2ost_DEV:-$(ostdevname 1)_2}

Comments

[Chris Gearing (Inactive)]

I misunderstood the code on that one!