Is there a reason that the original line was replaced with this? When using docker to run tests locally with this change we get an error saying can't open file '/app/local_locust.py'. We may need add something like

COPY local_locust.py /app/local_locust.py

Both the old and new code work when running locust in AWS

Is there a reason we need to run the load test inside docker locally? It has caused some complication/problems in the past

Moving forward there shouldn't be as we would likely want all of the tests run in AWS. I mentioned the above because up until this PR local testing was the main method of testing. The setup docs provide instructions to support local testing through docker so thought we were going to keep supporting it.

If its caused some issues in the past and we aren't planning on using it moving forward I don't think this is something we need to address. We will likely want to update the docs to reflect this.

Ah! Yeah, running in docker locally caused problems in the past, and so I think we'd like to not do that anymore. Perhaps that can be a follow up PR to cleanup/remove the documentation to make that clearer?