[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:14:45 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:
-----------------------------+-------------------------------

Comment (by karsten):

 Replying to [comment:21 iwakeh]:
 > All tests and checks pass and code looks ok.

 Awesome!

 > 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?

 Good point. Let's go with `lastSeenMillis` which is what we use in
 `NodeStatus`, too. I'll change that before merging.

 > 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.

 I'll run another local test before merging. Once that's done, let's
 coordinate deployment as usual.

 Thanks for the review!

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


More information about the metrics-bugs mailing list