From 1d0eb928e70f5cc02dde5cf46e724cd082f31fc3 Mon Sep 17 00:00:00 2001
From: David Schroeder <david@schroedercity.com>
Date: Sun, 28 Jan 2024 19:16:16 -0600
Subject: [PATCH] Update temp.inc.sh

---
 inc/temp.inc.sh | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/inc/temp.inc.sh b/inc/temp.inc.sh
index f51cbbf1..90811134 100644
--- a/inc/temp.inc.sh
+++ b/inc/temp.inc.sh
@@ -886,7 +886,7 @@ ESXI_FANSPEED_MGMT(){
 		if ([ ${MN} -eq 1 ] && [[ "${ESXI_HOSTS}" = *"${PW_HOST_ADDRESSES[${PW_iDRACHOST_SHORTNAMES[${host}]}]}"* ]]) || ([ "${idracip}" != "" ] && [ "${idracip}" == "${host}" ]); then
 			tmp=temp
 			echo -en "${idsCL[LightCyan]}${1}) ${PW_iDRACHOST_SHORTNAMES[${host}]} [${PW_HOST_ADDRESSES[${PW_iDRACHOST_SHORTNAMES[${host}]}]}]"
-			[ ! -f ${PW_TMPFOLDER}/${PW_iDRACHOST_SHORTNAMES[${host}]}.down ] && echo " - Offline, skipping" || echo
+			[ -f ${PW_TMPFOLDER}/${PW_iDRACHOST_SHORTNAMES[${host}]}.down ] && echo " - Offline, skipping" || echo
 		fi
 	done
 	DIVIDER true yellow
@@ -896,8 +896,11 @@ ESXI_FANSPEED_MGMT(){
 		if ([ ${MN} -eq 1 ] && [[ "${ESXI_HOSTS}" = *"${PW_HOST_ADDRESSES[${PW_iDRACHOST_SHORTNAMES[${host}]}]}"* ]]) || ([ "${idracip}" != "" ] && [ "${idracip}" == "${host}" ]); then
 			echo -en "${idsCL[Yellow]}Setting new FanSpeeds for ${PW_iDRACHOST_SHORTNAMES[${host}]} [${PW_HOST_ADDRESSES[${PW_iDRACHOST_SHORTNAMES[${host}]}]}]: "
 			if [ ! -f ${PW_TMPFOLDER}/${PW_iDRACHOST_SHORTNAMES[${host}]}.down ]; then
-				SET_ESXI_FANSPEED ${FANSPEED} ${PW_iDRACHOST_SHORTNAMES[${host}]} >/dev/null 2>&1 &
-				echo -e "Commands Sent${idsCL[Default]}"
+				if ! SET_ESXI_FANSPEED ${FANSPEED} ${PW_iDRACHOST_SHORTNAMES[${host}]} >/dev/null 2>&1 ; then
+					echo -e "${idsCL[Green]}FanSpeed set to ${FANSPEED}${idsCL[Default]}"
+				else
+					echo -e "${idsCL[Red]}Command failed${idsCL[Default]}"
+				fi
 			else
 				echo -e "${idsCL[Yellow]}Skipping, host offlinet${idsCL[Default]}"
 			fi