[metrics-bugs] #16513 [Metrics/Onionoo]: Make writing of the out/ directory from the status/ directory deterministic

Tor Bug Tracker & Wiki blackhole at torproject.org
Thu Jan 11 11:06:56 UTC 2018


#16513: Make writing of the out/ directory from the status/ directory deterministic
-----------------------------+-------------------------------
 Reporter:  karsten          |          Owner:  metrics-team
     Type:  enhancement      |         Status:  merge_ready
 Priority:  High             |      Milestone:  Onionoo-2.0.0
Component:  Metrics/Onionoo  |        Version:
 Severity:  Normal           |     Resolution:
 Keywords:  metrics-2018     |  Actual Points:
Parent ID:                   |         Points:
 Reviewer:  iwakeh           |        Sponsor:
-----------------------------+-------------------------------
Changes (by iwakeh):

 * status:  needs_review => merge_ready


Comment:

 All tests and checks pass and code looks ok.

 Shouldn't the new method parameter 'long now' (in various methods) be
 named differently to reflect the meaning of the milliseconds timestamp?
 Maybe 'long lastMeasurementMillis' or similar?

 How will the data be affected when deploying the fix?  The stored data
 timestamps might be more recent than the last seen value used in the new
 code.  Did you run system tests on a local instance?
 This should be clarified before deployment, but the code is merge ready.

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


More information about the metrics-bugs mailing list