tag1consulting / goose

Load testing framework, inspired by Locust
https://tag1.com/goose
Apache License 2.0
791 stars 70 forks source link

Panic when shutting down via controller #474

Closed slashrsm closed 2 years ago

slashrsm commented 2 years ago

Connected to an idle test via the Telnet controller and did:

goose> start
load test started
goose> stop
load test stopped
goose> shutdown
load test shut down
goose> Connection closed by foreign host.

Which resulted in

thread 'main' panicked at 'internal error: entered unreachable code: there shouldn't be a step after finished', src/metrics.rs:2689:25

The test obvioulsy still did shut down, but it is not nice to do so with a panic :).

jeremyandrews commented 2 years ago

Are you able to duplicate with the latest code? I'm testing this now and am seeing a clean shutdown when repeating the steps you listed above.

slashrsm commented 2 years ago

Yes I am still getting the same error, except that it is on line 3056:25 now. I started the test on drupal_memcache example with the following options -u 10 -r 3 -t 60 --report-file report.html --no-reset-metrics --no-autostart.