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

chaosblade prepare jvm prepare with javaHome flags returns "java command not found" #286

Closed
zousheng opened this issue Jan 13, 2020 · 1 comment · Fixed by #287
Closed
Assignees
Labels
chaosblade-cli chaosblade project type/bug Something isn't working
Milestone

Comments

@zousheng
Copy link
Contributor

Issue Description

Type: bug report or feature request
bug report

Describe what happened (or what feature you want)

 when I called prepare api with javaHome flag (host:9526/chaosblade?cmd=prepare%20jvm%20--javaHome=/home/q/java/default%20--process=pf_noah_webapp%), it returned "java command not found" error

{"code":604,"success":false,"error":"sudo: java: command not found\n exit status 1","result":"426204ac84dc27a3"}

Describe what you expected to happen

success: true

How to reproduce it (as minimally and precisely as possible)

  1. start java application with non-root user
  2. start blade server mode with sudo
  3. call the prepare api and pass javaHome flag

Tell us your environment

centos7

Anything else we need to know?

@xcaspar xcaspar self-assigned this Jan 13, 2020
@xcaspar xcaspar added the type/bug Something isn't working label Jan 13, 2020
@xcaspar xcaspar added this to the 0.5.0 milestone Jan 13, 2020
@xcaspar xcaspar assigned zousheng and unassigned xcaspar Jan 13, 2020
@xcaspar
Copy link
Member

xcaspar commented Jan 13, 2020

@zousheng Thank you for reporting and fixing it.

@xcaspar xcaspar added the chaosblade-cli chaosblade project label Feb 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chaosblade-cli chaosblade project type/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants