[metrics-bugs] #21414 [Metrics]: Include currently running software versions in responses (collector.tp.o, onionoo.tp.o) and on the website (metrics.tp.o)

Tor Bug Tracker & Wiki blackhole at torproject.org
Fri Oct 6 09:26:10 UTC 2017


#21414: Include currently running software versions in responses (collector.tp.o,
onionoo.tp.o) and on the website (metrics.tp.o)
--------------------------+---------------------------------
 Reporter:  iwakeh        |          Owner:  metrics-team
     Type:  enhancement   |         Status:  needs_review
 Priority:  Low           |      Milestone:  CollecTor 1.4.0
Component:  Metrics       |        Version:
 Severity:  Normal        |     Resolution:
 Keywords:  metrics-2017  |  Actual Points:
Parent ID:                |         Points:
 Reviewer:                |        Sponsor:
--------------------------+---------------------------------

Comment (by iwakeh):

 I don't see my fixup commits in your branch (which is a little confusing)
 and some of the changes you made were in there.
 But, the changes look ok; except that I prefer to explicitly set the
 fallback value 'null' instead of relying on the default in `getProperty`,
 because this states that the decision was made consciously and not by
 accident -- some way to document in code.

 Regarding the changelog of metrics-lib:  This needs to be clearer, like
 "the Git revision supplied by the calling software" or similar.  It should
 make clear that this is a revision of the software __using__ the API and
 that metrics-lib doesn't interfere with it.  metrics-lib creates the
 files, but the git-revision supplied is not metrics-lib.

 The same for CollecTor, maybe:
 //Git revision of the CollecTor instance's software used to create this
 file, which will be omitted if unknown.//

 (Wondering if the metrics-lib revision should be appended to the one given
 by CollecTor as it does have some influence on the files, i.e.,
 `"build_revision":"abcdfe7-zyxabc5"`? It would be easy, but of course we
 would have to discuss and explain what happens when there is only one of
 these available)

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/21414#comment:15>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online


More information about the metrics-bugs mailing list