Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RemoteAccount.kill_process doesn't work for java processes with ridiculous classpaths #125

Open
hachikuji opened this issue Nov 7, 2015 · 1 comment

Comments

@hachikuji
Copy link

For java processes with very long classpath options (such as those in the kafka command line tools), the full command is truncated in the process list, which makes the ps ax | grep {name} strategy ineffective. A more reliable option might be to use jps. I also noticed that kill_process currently only works for java processes. It might be better to make kill_process generic and include a java-specific kill_java_process so that ducktape can work with non-java processes.

@ewencp
Copy link
Contributor

ewencp commented Nov 8, 2015

Both of these are reasonable. This has come up a couple of times recently and the better solution is to just move services over to using pid files. I think the existing approach was mainly based off of the kafka bin scripts which use the janky ps + grep approach.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants