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

Re: [tor-bugs] #9814 [Metrics/Atlas]: Atlas should make clear when relay details come from outdated consensus



#9814: Atlas should make clear when relay details come from outdated consensus
---------------------------+--------------------------
 Reporter:  wfn            |          Owner:  phw
     Type:  defect         |         Status:  reopened
 Priority:  High           |      Milestone:
Component:  Metrics/Atlas  |        Version:
 Severity:  Normal         |     Resolution:
 Keywords:                 |  Actual Points:
Parent ID:                 |         Points:
 Reviewer:                 |        Sponsor:
---------------------------+--------------------------

Comment (by cypherpunks):

 The uptime column on the search page is confusing. For both running and
 non-running relays the uptime is shown which makes no sense for the later.
 IMO the strike through isn't enough to clear up that confusion.
 Furthermore, the uptime of non-running relays is only shown on the search
 page (with the strike through) while the details page shows the downtime
 (which is better IMO). Finally, the online/offline dot in the first column
 better communicates the status of the relays than the strike through.
 Because of these reasons i think the uptime column should be removed from
 the search page.

 Now on to the details pages. When a relay is running it shows the uptime
 which is based on the `last_restarted` field from Onionoo according to
 [https://gitweb.torproject.org/atlas.git/tree/js/models/relay.js?id=a2b3f23f0212a16ae410d513749b535fdbe6bc5a#n180
 js/models/relay.js:180]. The `last_restarted` field is also shown in its
 unparsed state further down the properties column. This is duplicate
 information (relative versus absolute) so i'm in favor of removing the
 `Last Restarted` field on the details page.

 When a relay is non-running it shows the downtime and the last seen
 timestamp. The downtime is based on the `last_seen` field from Onionoo
 according to
 [https://gitweb.torproject.org/atlas.git/tree/js/models/relay.js?id=a2b3f23f0212a16ae410d513749b535fdbe6bc5a#n187
 js/models/relay.js:187]. This is duplicate information (relative versus
 absolute) so i'm in favor of removing the `Last Seen` field on the details
 page.

 I'd be happy to create a patch for these changes if they are approved.

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