myzhan / locust4j

Locust4j is a load generator for locust, written in Java.
MIT License
81 stars 30 forks source link

Include user_classes_count on stats message #33

Closed stephen-harris closed 3 years ago

stephen-harris commented 3 years ago

This fixes a bug where errors are appearing in master. Although there doesn't appear to be any noticeable impact to Locust.

Locust version: 2.0.0b3 Locust4j version: master

The following error appears when running atest:

master_1 | [2021-08-14 21:42:06,719] 1e6ea2aa04b2/ERROR/root: Uncaught exception in event handler: master_1 | Traceback (most recent call last): master_1 | File "/usr/local/lib/python3.8/site-packages/locust/event.py", line 40, in fire master_1 | handler(**kwargs) master_1 | File "/usr/local/lib/python3.8/site-packages/locust/runners.py", line 621, in on_worker_report master_1 | self.clients[client_id].user_classes_count = data["user_classes_count"] master_1 | KeyError: 'user_classes_count'

Locust master expects user_classes_count to be included in stats message. This PR adds that.

myzhan commented 3 years ago

Thanks!