Bash script always true

Question

Is there any reason this script always give me running ? In both cases, when my process is started and when my process is stopped.

if ps ax | grep -v grep | grep "processName" > /dev/null
then
   echo $"running"
else
   echo $"not running"
fi

Thank you very much

UPDATE : I add a full exemple of my script, maybe smothing wrong somewhere else.

case "$1" in
  start)
    # Start daemons.

    echo -n $"Starting daemon: "
    ;;

  stop)
    # Stop daemons.
    echo -n $"Shutting down: "
    echo
    ;;
  status)
    pgrep -f "ProcessName" > /dev/null
    if [ $? -eq 0 ]; then
        echo $"ProcessName is running"
    else
        echo $"ProcessName is not running"
    fi
    ;;
  restart)
    $0 stop
    $0 start
    ;;

  *)
    echo $"Usage: $0 {start|stop|status|restart}"
    exit 1
esac

UPDATE 2 :

[user@dev init.d]# pgrep -f "MyProcess" > /dev/null
[user@dev init.d]# echo $?
0
[user@dev init.d]# service MyProcess stop
Shutting down MyProcess: Terminated
[user@dev init.d]# pgrep -f "MyProcess" > /dev/null
[user@dev init.d]# echo $?
1

But if [ $? -eq 0 ]; then seems to always be true

Answer 1

your not doing a test. put brackets around the test condition like so:

if [ condition your testing ]

Answer 2

Try this instead:

ps aux | grep -q "[p]rocessName"
if [ $? -eq 0 ]; then
    echo "running"
else
    echo "not running"
fi

The brackets around the first letter of processName means do don't need the "grep -v grep", while the -q means we don't need the pipe to /dev/null

$? gives you the return code of the previous command executed. Hence, testing if it were 0 would indicate if "grep" found what it was looking for.

Update

If your process name is really short (say "cup"), you might get a false positive as it may match other processes too (say "cupsd"). You can overcome this by having grep match whole words - add the "-w" flag.

Not that this technique is not perfect. You may end up matching entries in the username/date fields. If that happens, look up "man ps" and be more selective of what you print out before doing a grep. Alternatively, prefilter the output with awk to extract only the column showing process/cmd name . E.g:

ps aux | awk '{print $11}' | grep -q -w "[p]rocessName"
if [ $? -eq 0 ]; then
    echo "running"
else
    echo "not running"
fi

Update 2

you can also use pgrep as suggested in answer below.

For really short process names, you might want to specify word boundaries (\b) before and after your process name to prevent overmatching (as described above)

pgrep "\bprocname\b" > /dev/null
if [ $? -eq 0 ]; then
    echo "running"
else
    echo "not running"
fi

Update 3

From the updated question, I see that you're running it from an init script. There's always a danger of pgrep matching the script itself. Try:

pgrep Processname | grep -q -v $$
if [ $? -eq 0 ]; then
    echo "running"
else
    echo "not running"
fi

That excludes the PID of the script from pgrep matches.

Update 4

(final update? fingers crossed)

If the init script is run via the "service" command, then we need to filter out the parent PID as well. How about:

pgrep Processname | grep -v $$ | grep -q -v $PPID
if [ $? -eq 0 ]; then
    echo "running"
else
    echo "not running"
fi

Answer 3

Probably becasue grep "processName" finds itself. I found this self same problem yesterday, except I was xarging the results to kill...

As an alternative you might like to try the pgrep command instead of your string of ps and various greps:

sd@camel:~$ pgrep bash
415
3477
sd@camel:~$ echo $?
0
sd@camel:~$ pgrep arf
sd@camel:~$ echo $?
1

Answer 4

G'day,

As an aside, instead of

ps ax | grep -v grep | grep "processName"

Try doing

ps ax | grep "[p]rocessName"

The ps is listing the grep function because it is seeing the string "grep processName" in the process list which is being passed by your grep for the string "processName".

Grepping for "[p]rocessName" will match just "processName" on its own, but not the string "grep [p]rocessName".