First, note that this script adds notes about failures to the end of existing text in /var/tmp/failed.txt. So, once an error is detected, future invocations of this script will add new failures to the end of the existing list instead of starting with a clean slate each time you run the script. Why output anything at all if the server is running the command you're looking for? Why print a blank line?

Second, you should ssh to a single server (presumably $i) instead of to your entire list of servers ($Server_List) in your loop. Assuming $Server_List expands to more than one word, your current command will always fail with a syntax error.

Third, you aways send mail even if no errors were detected.

Fourth, if you are trying to count the number of lines in ps output that match a certain string, why are you throwing away the output before counting the number of lines found?

Fifth, the output from wc -l contains some leading spaces (unless the number of lines counted is 10 million or larger); so a string comparison between the output from wc -l and "1" can never match.

And, finally, if the command:

Code:

ssh -tq <ServerOne> "/bin/ps -ef | /bin/grep  'com.utv.wlrs.mer.mer pop start' " > /dev/null

works to check the status of a single server, why are you using a different command to check the status of each server in your loop? What is it about the command above that tells you whether or not the command you're looking for is running on that server? Why can't you script determine whether or not the command you're looking for is running on any server when you run it inside your for loop?