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

(fix)resource-chaoslib: run command with a shell instance #81

Merged
merged 1 commit into from
Aug 14, 2020

Conversation

ksatchit
Copy link
Member

@ksatchit ksatchit commented Aug 14, 2020

  • Modify the resource (cpu, memory) chaoslib to read commands from the command_string operand instead of from the standard input while performing the exec operation.

  • This should help make the command execution more predictable as we force execution with /bin/sh regardless of entrypoint.

  • Towards fix: CPU hog experiment fails to inject chaos litmus#1861

Again, these still apply only to target containers that offer a shell.

Signed-off-by: ksatchit karthik.s@mayadata.io

Signed-off-by: ksatchit <karthik.s@mayadata.io>
@ksatchit
Copy link
Member Author

cc: @andreas131989 !

@ksatchit ksatchit merged commit eee392c into litmuschaos:master Aug 14, 2020
@ksatchit ksatchit deleted the cmd_fix_resource_exp branch August 14, 2020 18:13
ksatchit pushed a commit to ksatchit/litmus-go that referenced this pull request Aug 15, 2020
ksatchit pushed a commit that referenced this pull request Aug 15, 2020
* chore(probe): Adding probes in all go experiments (#80)

Signed-off-by: shubhamchaudhary <shubham.chaudhary@mayadata.io>

* (fix)resource-chaoslib: run command with a shell instance (#81)

Signed-off-by: ksatchit <karthik.s@mayadata.io>

* refactor(go-experiments): Refactor all the go experiments (#82)

Signed-off-by: shubhamchaudhary <shubham.chaudhary@mayadata.io>

* chore(exp): Add pod autoscaler experiment used to check the scalability of the application pod (#65)

* chore(exp): Add pod autoscaler experiment used to check the salability of the application pod

Signed-off-by: Udit Gaurav <uditgaurav@gmail.com>

* Adding abort in the experiment

Signed-off-by: Udit Gaurav <udit.gaurav@mayadata.io>

* update(chaosresult): updating the chaosresult for probe score (#85)

Signed-off-by: shubhamchaudhary <shubham.chaudhary@mayadata.io>

* refactor(pod-autoscaler): refactor the pod-scaler experiment (#86)

Signed-off-by: shubhamchaudhary <shubham.chaudhary@mayadata.io>

Co-authored-by: Shubham Chaudhary <shubham.chaudhary@mayadata.io>
Co-authored-by: UDIT GAURAV <35391335+uditgaurav@users.noreply.github.com>
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

Successfully merging this pull request may close these issues.

CPU hog experiment fails to inject chaos
2 participants