Opened 3 years ago

Closed 3 years ago

#21630 closed defect (fixed)

Remove the Descriptor Published field

Reported by: cypherpunks Owned by: irl
Priority: Medium Milestone:
Component: Metrics/Relay Search Version:
Severity: Normal Keywords:
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

From ticket:12692#comment:9

I find "Published: Never" confusing.

Others as well:
https://lists.torproject.org/pipermail/tor-relays/2017-March/012048.html

What onionoo field is used for that?

This field isn't backed by any data. A quick grep only turns up that field in the template and no assignments in the model. Onionoo also doesn't mention desc_published.

Child Tickets

Attachments (1)

0001-Remove-the-descriptor-published-field.patch (1.3 KB) - added by cypherpunks 3 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 3 years ago by karsten

Ah, looks like we removed that field from Onionoo five years ago, long before we even assigned a protocol version number:

<li><b><font color="red">"desc_published":</font></b> UTC timestamp
(YYYY-MM-DD hh:mm:ss) when the relay last published a descriptor.
Deprecated field.
Deprecated, because this information is only useful to calculate the last
restart time of the relay, together with <i>"uptime"</i>.
We can as well provide that time directly in <i>"last_restarted"</i>.
Announced on January 26, will be removed after February 26, 2012.</li>
<li><b><font color="red">"uptime"</font></b> Uptime of the relay when
publishing its last descriptor.
Deprecated field.
Deprecated, because this information is only useful to calculate the last
restart time of the relay, together with <i>"desc_published"</i>.
Even worse, it may be mistaken for the current uptime of the relay at the
time of publishing this detail document.
Announced on January 26, will be removed after February 26, 2012.</li>

I'd say just remove this field from Atlas as well.

Changed 3 years ago by cypherpunks

comment:2 Changed 3 years ago by cypherpunks

Status: newneeds_review

comment:3 Changed 3 years ago by cypherpunks

can we get this removed anytime soon? It is confusing a lot.

comment:4 Changed 3 years ago by irl

Resolution: fixed
Status: needs_reviewclosed

Merged. Thanks.

Note: See TracTickets for help on using tickets.