[tor-bugs] #28355 [Core Tor/sbws]: Disable measurement timing rules on sbws by default (was: Do not notify about bwauths out of sync)

[Tor Bug Tracker & Wiki]

#28355: Disable measurement timing rules on sbws by default
---------------------------+-------------------------------------
 Reporter:  juga           |          Owner:  juga
     Type:  defect         |         Status:  needs_information
 Priority:  Medium         |      Milestone:  sbws 1.0 (MVP must)
Component:  Core Tor/sbws  |        Version:
 Severity:  Normal         |     Resolution:
 Keywords:                 |  Actual Points:
Parent ID:                 |         Points:
 Reviewer:                 |        Sponsor:
---------------------------+-------------------------------------
Changes (by teor):

 * type:  task => defect
 * milestone:   => sbws 1.0 (MVP must)
 * component:  Core Tor/DocTor => Core Tor/sbws


Old description:

> Currently sbws include less relays in its report, mostly due to
> https://trac.torproject.org/projects/tor/ticket/27338#comment:6.
>
> While sbws is not changed to include more relays, maybe the notice
> message about
> "Bandwidth authorities have a substantially different number of measured
> entries" should not be sent.

New description:

 Currently sbws include less relays in its report, mostly due to
 https://trac.torproject.org/projects/tor/ticket/27338#comment:6.

 ~~While sbws is not changed to include more relays, maybe the notice
 message about
 "Bandwidth authorities have a substantially different number of measured
 entries" should not be sent.~~

 sbws should disable these restrictions, so it matches torflow.

--

Comment:

 > i think torflow is just including more relays because it doesn't check
 the restrictions we added.

 We have a policy that sbws should match torflow, unless we have a very
 good reason. (Like a bad bug in torflow.)

 Adding the restrictions to sbws in #27338 has caused a major bug: it does
 not match torflow.

 Here's what I think we've tried so far:
 * fixing the sbws scheduler
 * disabling rtt

 Are these fixes in longclaw's version of sbws?

 If we can't get the restrictions to work, we should disable them by
 default, so sbws matches torflow.
 Then we should open a ticket to review them in a later release.

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