Don't log state change if it's the same

[max-rocket-internet]

Resolves #1577

[codecov]

Codecov Report

Merging #1580 into master will increase coverage by 0.19%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1580      +/-   ##
==========================================
+ Coverage   81.37%   81.57%   +0.19%     
==========================================
  Files          28       28              
  Lines        2583     2583              
  Branches      393      393              
==========================================
+ Hits         2102     2107       +5     
+ Misses        384      379       -5     
  Partials       97       97              

Impacted Files	Coverage Δ
locust/runners.py	81.54% <100.00%> (+0.36%)	⬆️
locust/stats.py	89.87% <0.00%> (-0.41%)	⬇️
locust/clients.py	95.09% <0.00%> (+4.90%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bfd98b1...5e4b1b7. Read the comment docs.

[cyberw]

Are you sure there is not an underlying issue? I dont see why we should make tons of state changes to the same state...

[max-rocket-internet]

True. I can find out why something is continually updating the state to the same value.

[max-rocket-internet]

It's from here: https://github.com/locustio/locust/blob/master/locust/runners.py#L624

I will test a different fix then 🙂

[cyberw]

Nice! maybe just add and not self.state == STATE_STOPPED to that if statement...

[max-rocket-internet]

OK done that now some test is failing. I will look into it.