[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

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

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

Comment (by asn):

 Replying to [comment:53 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.

 Thanks for the merge.

 Opened ticket at #14957.

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/9321#comment:55>
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