Remove dirreq-v2-* and dirreq-v?-share statistics

The v2 directory protocol is (almost?) dead, and we stopped using dirreq-v?-share lines, because they were highly inaccurate. Should we stop including these lines in extra-info descriptors? If so, I can prepare a patch.

changed milestone to %Tor: 0.2.4.x-final in legacy/trac

added component::core tor/tor in Legacy / Trac easy in Legacy / Trac milestone::Tor: 0.2.4.x-final in Legacy / Trac owner::karsten in Legacy / Trac priority::low in Legacy / Trac resolution::implemented in Legacy / Trac status::closed in Legacy / Trac tor-relay in Legacy / Trac type::enhancement in Legacy / Trac labels

That would be fine by me.

Fine by me also. (We should be extra sure we never want dirreq-v3-share though.)

Hmm, I'm not extra super sure we never want to use dirreq-v3-share lines ever. Maybe we should only remove dirreq-v2-* lines then.

Will prepare a patch once there's an 0.2.4.x branch.

Trac:
Keywords: N/A deleted, easy added
Owner: N/A to karsten
Status: new to assigned
Priority: normal to minor

Damian found a relay that reports a share of over 100%:

extra-info siltornado 995D0FE5A89563D79A383CCC2444D0E26C6BE625
published 2012-06-24 19:30:31
...
dirreq-v2-share 159.81%
dirreq-v3-share 159.81%

Rather than hunting down that bug, I'd rather want to kill dirreq-v?-share lines entirely. Now they're not only highly inaccurate but also buggy. If we ever want to calculate the share of directory requests that a relay should receive in theory, we can use the consensus just like the relay does.

Trac:
Keywords: easy deleted, easy tor-relay added

Trac:
Component: Tor Relay to Tor

karsten -- will anything break if we remove these now? See branch "bug5823".

Trac:
Status: assigned to needs_review

Replying to nickm:

karsten -- will anything break if we remove these now? See branch "bug5823".

Looks sane.

Should we also remove dirreq-v2-* lines as discussed above? Please see my bug5823 branch. If you're fine with merging this, I'll test-run it somewhere for 24+ hours to make sure it doesn't explode.

Yes, please test that out and let me know when it looks ok.

Here's the stats/dirreq-stats file from my test relay written 24 hours after the relay was started:

dirreq-stats-end 2013-01-19 07:28:27 (86400 s)
dirreq-v3-ips it=16,us=16,br=8,ca=8,cz=8,de=8,eg=8,es=8,fr=8,gb=8,ie=8,il=8,ir=8,ng=8,nl=8,ru=8,sa=8,sd=8,sg=8,si=8,sy=8,tg=8,th=8,tr=8
dirreq-v3-reqs it=16,us=16,br=8,ca=8,cz=8,de=8,eg=8,es=8,fr=8,gb=8,ie=8,il=8,ir=8,ng=8,nl=8,ru=8,sa=8,sd=8,sg=8,si=8,sy=8,tg=8,th=8,tr=8
dirreq-v3-resp ok=64,not-enough-sigs=0,unavailable=0,not-found=0,not-modified=16,busy=0
dirreq-v3-direct-dl complete=0,timeout=0,running=0
dirreq-v3-tunneled-dl complete=60,timeout=4,running=0,min=2176,d1=20257,d2=31323,q1=39711,d3=45486,d4=62359,md=71595,d6=73982,d7=76214,q3=77018,d8=78153,d9=78977,max=81316

Everything looks good there. That relay didn't publish a new extra-info descriptor with these statistics, but I don't see how that should fail. The relay was still running 30 hours after I started it, and now I shut it down. Should be safe to merge my bug5823 branch. Thanks!

Excellent; merging.

Trac:
Status: needs_review to closed
Resolution: N/A to implemented

closed

moved from legacy/trac#5823 (moved)

added Feature First Contribution labels and removed 1 deleted label

added Relay label and removed 1 deleted label