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

docs (#2188): Add Locust Kubernetes Operator #2288

Merged
merged 1 commit into from
Jan 16, 2023

Conversation

AbdelrhmanHamouda
Copy link
Contributor

#2188

  • Update Running a distributed load test on Kubernetes docs section to include Locust Kubernetes Operator.
  • Small reformat of the section to clearly list all possibilities (HELM & Operator) in a structured manner.

locustio#2188 

- Update `Running a distributed load test on Kubernetes` docs section to include Locust Kubernetes Operator. 
- Small reformat of the section to clearly list all possibilities (HELM & Operator)  in a structured manner.
HELM
*****

One of the ways to run Locust on Kubernetes is to use a `Helm chart <https://helm.sh/>`_.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This rephrasing of the statement is purely an opinion and I fully understand if Locust want to state that HELM is the easiest way to run distributed tests. If the rephrasing is not okay, please let me know and I will restore the original phrasing.

Opinion justification:
HELM is an incredible tool and the provided chart does a lot to make sure that the processes is easy. However, the user is not really "shielded" from Kubernetes and have to be exposed and aware of many other moving peaces (e.g. deployments, configmaps, etc...) in order to get things right. That in my opinion makes it so the term "easiest" may cause a bit of misguidance.

@AbdelrhmanHamouda
Copy link
Contributor Author

Hello @cyberw , I attempted to follow the contribution docs as close as i could. Things look good locally. Please let me know if something is missing.

Also, I have a proposal to move the "Running in Kubernetes" section to it's own dedicated page. The topic is too big "and irrelevant IMHO" to be under Running in Docker.

@cyberw cyberw merged commit 15c9e7a into locustio:master Jan 16, 2023
@cyberw
Copy link
Collaborator

cyberw commented Jan 16, 2023

Great stuff. Yes, splitting it to a separate page makes sense.

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.

None yet

2 participants