[tor-commits] [tor/master] Remove the "weight=10" line from fallback directory mirror entries

[nickm at torproject.org]

commit ac5058ac8a6eeb3ebb0790d33dbeee4473dcaae7
Author: teor <teor2345 at gmail.com>
Date:   Fri Dec 22 23:05:45 2017 +1100

    Remove the "weight=10" line from fallback directory mirror entries
    
    This removes some redundant repeated lines.
    
    Ticket 24681 will maintain the current fallback weights by changing
    Tor's default fallback weight to 10.
    
    Implements ticket 24679.
---
 changes/ticket24679                 |  5 +++++
 scripts/maint/updateFallbackDirs.py | 19 ++-----------------
 2 files changed, 7 insertions(+), 17 deletions(-)

diff --git a/changes/ticket24679 b/changes/ticket24679
new file mode 100644
index 000000000..183e60ae9
--- /dev/null
+++ b/changes/ticket24679
@@ -0,0 +1,5 @@
+  o Minor features (fallback directory mirrors):
+    - Remove the "weight=10" line from fallback directory mirror entries.
+      Ticket 24681 will maintain the current fallback weights by changing
+      Tor's default fallback weight to 10.
+      Implements ticket 24679.
diff --git a/scripts/maint/updateFallbackDirs.py b/scripts/maint/updateFallbackDirs.py
index 7869d942c..7bd6fae0e 100755
--- a/scripts/maint/updateFallbackDirs.py
+++ b/scripts/maint/updateFallbackDirs.py
@@ -245,21 +245,6 @@ CONSENSUS_DOWNLOAD_SPEED_MAX = 15.0
 # This avoids delisting a relay due to transient network conditions
 CONSENSUS_DOWNLOAD_RETRY = True
 
-## Fallback Weights for Client Selection
-
-# All fallback weights are equal, and set to the value below
-# Authorities are weighted 1.0 by default
-# Clients use these weights to select fallbacks and authorities at random
-# If there are 100 fallbacks and 9 authorities:
-#  - each fallback is chosen with probability 10.0/(10.0*100 + 1.0*9) ~= 0.99%
-#  - each authority is chosen with probability 1.0/(10.0*100 + 1.0*9) ~= 0.09%
-# A client choosing a bootstrap directory server will choose a fallback for
-# 10.0/(10.0*100 + 1.0*9) * 100 = 99.1% of attempts, and an authority for
-# 1.0/(10.0*100 + 1.0*9) * 9 = 0.9% of attempts.
-# (This disregards the bootstrap schedules, where clients start by choosing
-# from fallbacks & authoritites, then later choose from only authorities.)
-FALLBACK_OUTPUT_WEIGHT = 10.0
-
 ## Parsing Functions
 
 def parse_ts(t):
@@ -1346,7 +1331,7 @@ class Candidate(object):
   def fallbackdir_info(self, dl_speed_ok):
     # "address:dirport orport=port id=fingerprint"
     # "[ipv6=addr:orport]"
-    # "weight=FALLBACK_OUTPUT_WEIGHT",
+    # ,
     #
     # Do we want a C string, or a commented-out string?
     c_string = dl_speed_ok
@@ -1370,7 +1355,7 @@ class Candidate(object):
     if self.has_ipv6():
       s += '" ipv6=%s:%d"'%(cleanse_c_string(self.ipv6addr), self.ipv6orport)
       s += '\n'
-    s += '" weight=%d",'%(FALLBACK_OUTPUT_WEIGHT)
+    s += ','
     if comment_string:
       s += '\n'
       s += '*/'