Users with `fixed_count` not being relocated after rebalance

Describe the bug

Running Locust with multiple workers, where UserA has fixed_count = 1 and UserB has no weight attributes.

As expected when starting 4 workers and 8 users, 1 UserA is started with 7 UserBs. Though when the worker UserA is running on is terminated, Locust rebalances such that theres now 8 UserBs and no UserAs. The same issue seems sometimes occur when just scaling up the number of workers, triggering a rebalance.

Adding some logs to the code, looks like self._try_dispatch_fixed is False on the rebalance (dispatch.py#L387) - which is only set to True when it first runs or when the number of users is reduced.

I’m happy have a go at fixing this - though adding issue for visibility.

Steps to reproduce

# locustfile.py
class UserA:
  fixed_count = 1
  # ...

class UserB:
  # ...

1. Add 4 workers
2. Add 1 user: This adds a single UserA as expected
3. Add 7 more users: This adds 7 UserBs as expected
4. Terminate the worker running the single UserA
5. After the rebalance now have 8 UserBs and no UserAs
6. Adding 2 more users (taking total to 10) still has no UserAs

All users spawned: {"UserA": 1, "UserB": 0} (1 total users)
All users spawned: {"UserA": 1, "UserB": 7} (8 total users)
All users spawned: {"UserA": 0, "UserB": 10} (10 total users)

Environment

• OS: python:3.9 container
• Locust version: running master
• Locust command line that you ran: /usr/local/bin/python /usr/local/bin/locust --master --locustfile .../locustfile.py
• Locust file contents (anonymized if necessary)

class UserA:
  fixed_count = 1
  # ...

class UserB:
  # ...

[EzR1d3r]

@cyberw @andydunstall Hi! It is not known issue. I dont remember rebalance in details right now. Does a master request from workers remained users before rebalancing? Overall, changing self._try_dispatch_fixed to True looks reasonable. And I guess, we need a test for this case, may be in locust/test/test_main.py?

[andydunstall]

@EzR1d3r Ok great thanks - I’ve added a PR here https://github.com/locustio/locust/pull/2093