Re: [tor-bugs] #28099 [Core Tor/sbws]: Make a policy for adding new sbws relay keys

["Tor Bug Tracker & Wiki" <blackhole@xxxxxxxxxxxxxx>]

#28099: Make a policy for adding new sbws relay keys
---------------------------+-------------------------------------
 Reporter:  teor           |          Owner:  (none)
     Type:  task           |         Status:  new
 Priority:  Medium         |      Milestone:  sbws 1.0 (MVP nice)
Component:  Core Tor/sbws  |        Version:
 Severity:  Normal         |     Resolution:
 Keywords:                 |  Actual Points:
Parent ID:                 |         Points:
 Reviewer:                 |        Sponsor:
---------------------------+-------------------------------------

Comment (by teor):

 Replying to [comment:1 juga]:
 > Replying to [ticket:28099 teor]:
 > > There are some keys in the spec and the code that are not being used
 by anyone.
 >
 > It is what i meant.
 > Also, maybe i used wrong the word policy, and i meant any key/value in
 the bandwidth-file-spec, not only in sbws, and not only for the relays,
 also for the header
 >
 > >
 > > > For instance, do we have tickets of cases where these key/values
 were needed/requested?.
 > >
 > > When we find this information, we should update the spec so it tells
 us why we need each value.
 >
 > Header key example not being used so far: `earliest_bandwidth`. with it
 we know from which time in the past we considered results in the current
 bandwidth file.
 > But in which cases we might need to look at that?, when a relay had a
 very different bandwidth 11 days ago and the file `earliest_bandwidth` is
 10 days ago?.
 > There was any case in the past (ticket, metrics, irc, ...) where having
 that information would have helped?.
 > Maybe we don't need a justification for all key/values, but i think
 having it will also help to understand why it would be useful to archive
 the file and how it can helps to understand the metrics or problems.

 earliest_bandwidth is Timestamp formatted as a human-readable date.
 Timestamp is used by Tor to work out when a bandwidth file is too old to
 use to vote.

 > A relay key example is `rtt`. I used it once and generated a graph with
 the bandwidths. With that i could check that all relays' bandwidth were
 slightly faster but the shape of the graph was not changing.
 > We are not using it anymore, and the code to obtain it seems to slow
 down a lot `sbws`. If we keep it, do we know for what we might want to use
 it?.

 I think we should disable rtt by default. rtt is useful for debugging, so
 we should allow operators to turn it on, with a warning that it slows down
 sbws.

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

_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs