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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improvement to Documentation #4564

Open
alininja opened this issue Apr 1, 2024 · 7 comments
Open

Improvement to Documentation #4564

alininja opened this issue Apr 1, 2024 · 7 comments
Labels

Comments

@alininja
Copy link

alininja commented Apr 1, 2024

Hi 馃憢

I was using these docs to get Litmus working on an OpenShift cluster:

In order for the kill command to work on my OpenShift cluster and pod, I had to update the suggested alternative kill commands to the following:

  • for memory: kill -9 $(ps afx | grep "[dd] if=/dev/zero" | awk '{print $1}' | tr '\n' ' ')
  • for cpu: $(ps afx | grep "[md5sum] /dev/zero" | awk '{print $1}' | tr '\n' ' ')

I am not familiar enough to know if the suggested alternative kill commands have a typo or is intended for Windows or is intended for other pod configurations or something else.

  • suggested alternative kill command for memory: kill -9 $(ps afx | grep \"[dd] if=/dev/zero\" | awk '{print $1}' | tr '\n' ' ')
  • suggested alternative kill command for cpu: kill -9 \((ps afx | grep \"[md5sum] /dev/zero\" | awk '{print\)1}' | tr '\n' ' ')

Hopefully, this can be helpful to someone else.

Thank you for making this project 馃檹

@alininja alininja added the bug label Apr 1, 2024
@prithvi1307
Copy link
Contributor

@Saranya-jena can you PTAL? @alininja would you like to raise a PR on this?

@alininja
Copy link
Author

alininja commented Apr 3, 2024

@alininja would you like to raise a PR on this?

@prithvi1307 sure thing. I was trying to find the source for the documentation, but could not find it. Would you be able to link me to it? Thanks 馃檹

@Saranya-jena
Copy link
Contributor

Hi @alininja you can raise a PR here: https://github.com/litmuschaos/litmus-docs

@alininja
Copy link
Author

Hi @alininja you can raise a PR here: https://github.com/litmuschaos/litmus-docs

Thank you @Saranya-jena for the link to the repo. Sorry, but I am still unable to find the source file that corresponds to these pages:

Sorry for the newbie confusion. 馃檹

@Saranya-jena
Copy link
Contributor

Hi @alininja you can raise a PR here: https://github.com/litmuschaos/litmus-docs

Thank you @Saranya-jena for the link to the repo. Sorry, but I am still unable to find the source file that corresponds to these pages:

Sorry for the newbie confusion. 馃檹

Hey @neelanjan00 Can you please help here?

@alininja
Copy link
Author

Thanks @neelanjan00 for the links.
I've created the PR here: #4609

I hope that works. It sounds like the original alternative kill command had typo's so I replaced it instead of adding an additional alternative kill command.

Thank you for making such a wonderful product 馃檹

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

No branches or pull requests

4 participants