Skip to content
GitLab
  • Menu
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • Tor Tor
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 309
    • Issues 309
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 33
    • Merge requests 33
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • The Tor Project
  • Core
  • TorTor
  • Merge requests
  • !401

Down-weight current runs too when computing wfu/tk

  • Review changes

  • Download
  • Email patches
  • Plain diff
Open Roger Dingledine requested to merge arma/tor:bug40395 into maint-0.4.5 Jun 23, 2021
  • Overview 16
  • Commits 1
  • Pipelines 1
  • Changes 2

Otherwise the earlier runs decay to small numbers, while the current run becomes a huge number, and then relays with high uptime dominate all the wfu and tk stats.

For now I'm leaving the mtbf tracking alone -- that is, mtbf is still dominated by the uptime of the current run -- because otherwise many relays lose their Stable flag and thus their Guard flag. It's not at all clear that we're assigning the Stable flag in a smart way, but also it's not clear that changing it to downweight the current run is what we want (if we downweight the total_run_weights variable, it decays more than it should, or if we don't, it decays less than it should).

The long term answer is that we need to use past data to assess how various approaches predict future behavior. But I'm doing this minimal change now since this bugfix paves the way for the HSDir assignment changes in #19162.

Resolves #40395.

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: bug40395