[tor-bugs] #28868 [Core Tor/sbws]: best_priority() can starve the worker threads of good relays

[Tor Bug Tracker & Wiki]

#28868: best_priority() can starve the worker threads of good relays
---------------------------+-----------------------------------
 Reporter:  teor           |          Owner:  juga
     Type:  defect         |         Status:  needs_review
 Priority:  Medium         |      Milestone:  sbws: 1.0.x-final
Component:  Core Tor/sbws  |        Version:  sbws: 1.0.2
 Severity:  Normal         |     Resolution:
 Keywords:                 |  Actual Points:
Parent ID:  #28663         |         Points:
 Reviewer:                 |        Sponsor:
---------------------------+-----------------------------------
Changes (by juga):

 * status:  assigned => needs_review


Comment:

 > Add a unit test and integration test that makes sure every queued relay
 has a result.
 Maybe this could be done as part of #28566 instead?.

 #28933 runs the actual scanner. It is not counting that all the relays get
 measured, though in the test network this is the case.

 Created PR without the tests: https://github.com/torproject/sbws/pull/328

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/28868#comment:5>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online