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

Update to accommodate latest version of locust #27

Merged
merged 2 commits into from
Mar 26, 2023

Conversation

jayudey-wf
Copy link
Contributor

@jayudey-wf jayudey-wf commented Mar 9, 2023

First off I'm a big fan 😄 , this is my attempt at making the library compatible w/ the most recent versions of locust

Problem

Swarm doesn't work with the latest versions of locust, #26

Solution

Update to handle some of the architectural changes that came with locust

  • item one
    • previously it looks like the 'master' was passing in the total number of clients and hatch rate to the workers
    • currently it looks like that information is passed down via the userClassesCount and the total number of workers is incremented from there
  • item two
    • for the initial ready check locust is expecting a data field where previously it wasn't (this prompted the changes to the visitor and messages class)

this was heavily influenced by reviewing what was done in locust4j

@anhldbk
Copy link
Owner

anhldbk commented Mar 13, 2023

@jayudey-wf Thank you for this MR. Gonna have a look at it.

@anhldbk
Copy link
Owner

anhldbk commented Mar 26, 2023

@jayudey-wf LGTM. Thank you!

@anhldbk anhldbk merged commit f6d2ee3 into anhldbk:master Mar 26, 2023
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.

3 participants