Shell Programming and Scripting

I have script as following..

server_status= some command | grep "Total error: 0"
if [ "$server_status= "server may down""]; then
echo " Server $(hostname) is Down" >>Result
fi
else
echo " Server is OK on $(hostname)" >>Result


the if command seems to be not working properly for some reason. I dont even see the out put file.

when I do
server_status= some command | grep "Total error: 0"

it gave me the error "server may down".

Any ideas. Please help

Your 'fi' is in the wrong place.

Code:

if [ something ] then;
  do something
else
  do something else
fi

if I put 'fi', it should give me the error right?

if [-f /src/srv1] then
server_status= some command | grep "Total error: 0"
if [ "$server_status= "server may down""]; then
echo " Server $(hostname) is Down" >>Result
fi
else
echo " Server is OK on $(hostname)" >>Result
fi

its going to else condition, and now I got a output file and i see that server is OK, even the server is down. I think problem with "if [ "$server_status= "server may down""]; then" . I'm really not sure if this is right. Thanks.

If you used code tags it would be easier to read for everyone who's trying to help you.

Please use the code tags to make this more readable. I didn't at first notice the nested if statements. Also, something burned into my head in high school (many, many moons ago) was code formatting for easier reading/debugging.

Code:

if [-f /src/srv1]
then
  server_status= some command | grep "Total error: 0"
  if [ "$server_status= "server may down""]
  then
    echo " Server $(hostname) is Down" >>Result
  else
    echo " Server is OK on $(hostname)" >>Result
  fi
fi

your code will say the server is OK only if /src/srv1 doesn't exist. I've moved the code around a bit, give this a go.

Fix your code.

Code:

if [ "$server_status= "server may down""]

Change it to:

Code:

if [ "$server_status" = "server may down"]

Thanks for your reply. code tags???

I tried with

if [ "$server_status" = "server may down"]

but I dont see the out file name when server is down. I think as soon as it reads this line "server_status= some command | grep "Total error: 0" and as the server is down, it's giving error saying server is down. it should go to the next line and check if condition, then it should create a file.

small change is the code, instead of checking for file, I'm checking for the directory with -d option.
Thanks