[metrics-bugs] #29374 [Metrics/Onionperf]: Analysis files sometimes present negative numbers in the payload_progress field

[Tor Bug Tracker & Wiki]

#29374: Analysis files sometimes present negative numbers in the payload_progress
field
-----------------------------------+------------------------------
 Reporter:  irl                    |          Owner:  metrics-team
     Type:  defect                 |         Status:  new
 Priority:  Medium                 |      Milestone:
Component:  Metrics/Onionperf      |        Version:
 Severity:  Normal                 |     Resolution:
 Keywords:  acute-2019-q1-planned  |  Actual Points:
Parent ID:                         |         Points:
 Reviewer:                         |        Sponsor:
-----------------------------------+------------------------------

Comment (by karsten):

 Interesting! I didn't know that OnionPerf only looks at `[transfer-*]` log
 lines. This is also good to know for #29787, just in case we need
 information from other log lines to further identify the failure case. And
 I'd be curious how we match these log lines to Tor controller event logs
 if we don't know the source port. But this can all be part of #29787.

 For this ticket, how about we leave out `payload_progress` for
 measurements where are unable to extract that information from log lines
 we parse. I could see how we would be able to learn that by making major
 changes the OnionPerf parser. But maybe we can do this in two steps: 1.
 fix information that is wrong and 2. put in information that is right. And
 step 2 could easily happen as a separate enhancement ticket to be done in
 the future. What do you think?

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