Wrong "ps" parameter in start/stop scripts when using alpine based docker image

In file $ALLUXIO_HOME/bin/alluxio in line 83 the call ps -Aww fails because busybox doesn’t know the ‘w’ flag. busybox is used when the environment is alpine (docker image!)

I suggest using pgrep instead of parsing the output of ps like this:

replace

for pid in $(ps -Aww -o pid,command | grep -i "[j]ava" | grep ${keyword} | awk '{print $1}'); do

with

for pid in $(pgrep -f "[j]ava.*$keyword"); do

[ZacBlanco]

Additionally, with respect to the proposed solution; while pgrep may be available in busybox, I don’t think it as portable as ps.

ps is a command defined in the UNIX specification: http://pubs.opengroup.org/onlinepubs/7990989775/xcuix.html

The actual command specification can be found here: http://pubs.opengroup.org/onlinepubs/7990989775/xcu/ps.html

I think it would be most portable if we are to only use ps and the arguments defined in the page above. What do you think?

[madanadit]

I agree that we should change our entrypoint (to use exec) and get rid of the docker bin directory