https://bugzilla.novell.com/show_bug.cgi?id=836235
https://bugzilla.novell.com/show_bug.cgi?id=836235#c6
--- Comment #6 from Russell Miller 2013-10-11 23:04:20 UTC ---
I can verify it's working. But taking a look at the patch, I don't understand
how it works, can you help me out here?
errorstring="Error"
shopt -s nocasematch
if [[ "$ipmioutput" =~ "${errorstring}" ]]
then
if [[ "${errorstring}" =~ ^"Fatal IO Error" ]]
then
checkhint="check BMC availability/username/password"
echo "ipmitool output contains \"$errorstring\" - $checkhint"
exit 2
fi
fi
shopt -u nocasematch
Unless I don't understand how =~ works in shell (I know how it works in perl),
the "check BMC availability" clause will never actually get hit. Error string
is set to "Error", and them immediately afterwards, it's checked to see if it's
"Fatal IO Error"? Should that be
if [[ "${ipmioutput}" =~ ^"Fatal IO Error" ]] ?
Even then, that would just have the effect of ignoring any legitimate errors if
the string "Fatal IO Error" actually exists.
Am I completely off here?
--
Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.