Opened 9 years ago

Closed 3 years ago

#2862 closed defect (wontfix)

T-Shirt uptime is absolute, not considering tiny interruptions

Reported by: cypherpunks Owned by: StalkR
Priority: Medium Milestone:
Component: Metrics/Tor Weather Version:
Severity: Keywords:
Cc: tor+trac@… Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

I've been running my non-exit Tor relay for many months now and for the last 3-4 months unthrottled. It averaged well above 500 Kilobyte/s. Since I signed up for Tor Weather earlier, I assumed I would receive the "get your shirt!" mail some day. But now it has been so long and I still did not get it.

My router has not been running without interruptions. I sometimes updated Tor, sometimes other things required me to stop it for short periods. I assumed Tor Weather would take that in mind.

I read https://gitweb.torproject.org/weather.git/blob?f=doc/design.txt#l42 and also asked in IRC:

<me> hm, according to the https://gitweb.torproject.org/weather.git/blob?f=doc/design.txt i need to run it uninterrupted for the timespan
<Sebastian> that sounds not what the intention of our tshirt giveaway is
<Sebastian> because a node can become unavailable through no fault of the operator for a short period of time
<Sebastian> please definitely file that bug now

So here it is! And I will send a mail to the donations address about the shirt. :-)

Child Tickets

Change History (4)

comment:1 Changed 8 years ago by kaner

Sebastian thinks we should change the algorithm as follows:

Even if a node goes down for longer periods of time, we should simply add up all the uptime we have seen, and, if it reaches the tshirt threshold, send out tshirt email.

comment:2 Changed 7 years ago by lachesis

I've had my server up for a few months with small outages as well, and I've been excitedly waiting for my T-shirt as well.

comment:3 Changed 6 years ago by StalkR

Cc: tor+trac@… added
Owner: changed from kaner to StalkR
Status: newaccepted

I noticed that as well, it resets all data when node is down: https://gitweb.torproject.org/weather.git/blob/HEAD:/weather/weatherapp/updaters.py#l138

I agree with Sebastian's change idea (via kaner above) and implemented it, please review at https://codereview.appspot.com/9837051/

comment:4 Changed 3 years ago by karsten

Resolution: wontfix
Status: acceptedclosed

Tor Weather has been discontinued as of May 24, 2016: https://lists.torproject.org/pipermail/tor-relays/2016-June/009424.html. Batch-closing all remaining tickets as announced in #19382. A list of these tickets and any other Weather tickets modified after June 26, 2016 will be available here: https://trac.torproject.org/projects/tor/query?changetime=Jun+27%2C+2016..&component=^Metrics%2FTor+Weather

Note: See TracTickets for help on using tickets.