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

stress mem bug #94

Closed
yujinqiu opened this issue Sep 16, 2021 · 2 comments
Closed

stress mem bug #94

yujinqiu opened this issue Sep 16, 2021 · 2 comments

Comments

@yujinqiu
Copy link
Contributor

How to reproduce

POC

./bin/chaosd attack stress mem -s 10%

Output

stress memory option: [][2021/09/16 18:27:28.221 +08:00] [INFO] [stress.go:67] ["stressors normalize"] [arguments=" --vm 1 --vm-keep--vm-bytes 10%"]
[2021/09/16 18:27:28.223 +08:00] [INFO] [stress.go:83] ["Start stress-ng process successfully"] [command="/usr/local/sbin/stress-ng --vm 1 --vm-keep--vm-bytes 10%"] [Pid=240670]
Attack stress mem successfully, uid: 72493bd6-5a0d-4e81-9860-50ac1b0cefde

you'll notice that the stress command is wrong

/usr/local/sbin/stress-ng --vm 1 --vm-keep--vm-bytes

should be

/usr/local/sbin/stress-ng --vm 1 --vm-keep --vm-bytes

a SPACE between --vm-keep and --vm-bytes is missing

Root Cause

chaosd depend on github.com/chaos-mesh/chaos-mesh/api/v1alpha1 v0.0.0-20210329070828-9be168b2b489
which has bug and is fixed on this commit

Solution

update dependency github.com/chaos-mesh/chaos-mesh/api/v1alpha1 after the fixed commit.
I have not idea why the dependency version is v0.0.0, so I don't make a PR :-(

Hope that will be help.

@WangXiangUSTC
Copy link
Collaborator

thanks! we will update the dependency

@WangXiangUSTC
Copy link
Collaborator

#65 in this pr update the dependency, fix this issue

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