[tor-bugs] #9321 [Tor]: Load balance right when we have higher guard rotation periods

[Tor Bug Tracker & Wiki]

#9321: Load balance right when we have higher guard rotation periods
-------------------------+-------------------------------------------------
     Reporter:  arma     |      Owner:
         Type:  project  |     Status:  needs_review
     Priority:  major    |  Milestone:  Tor: 0.2.6.x-final
    Component:  Tor      |    Version:
   Resolution:           |   Keywords:  needs-proposal, tor-auth, tor-
Actual Points:           |  client, 026-triaged-1, unfrozen, nickm-review
       Points:           |  Parent ID:  #11480
-------------------------+-------------------------------------------------

Comment (by nickm):

 Replying to [comment:52 asn]:
 > OK, please check the `bug9321_rebase` branch again. I improved the
 comment wrt the total bandwidth weights. Hopefully the situation is more
 clear now.
 >
 > Your comment:
 > {{{
 > It's a little weird that should_apply_guardfraction() applies changes
 only when we next parse the networkstatus, but that probably won't hurt
 anything, right?
 > }}}
 > is interesting. You mean that because we call
 `should_apply_guardfraction()` during consensus parsing,
 `networkstatus_get_param()` will actually check the previous consensus and
 not the one we are currently parsing, right? That's true.
 >
 > I don't think it's tera-bad, except if we plan to be toggling the
 `UseGuardFraction` consensus parameter frequently. Under normal
 circumstances, it will delay the effect of `UseGuardFraction` by one hour.

 I agree that it is wrong but not so bad. Either approach is fine, though
 the former needs lots of documentation so that the relevant functions are
 explicit that they accept a partially constructd consensus.

 Could you please make a new ticket for this, targeting 0.2.7?  I'm going
 to merge what's here now and close.

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