#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 acute): Replying to [comment:5 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. Agreed! >For this ticket, how about we leave out `payload_progress` for measurements where are unable to extract that information from log lines we parse. >And step 2 could easily happen as a separate enhancement ticket to be done in the future. What do you think? Do you mean that for now we should change the parser to leave the field out entirely in cases where the transfer does not complete? Also, +1 to discussing the parser refactoring in a new ticket! -- Ticket URL: <https://trac.torproject.org/projects/tor/ticket/29374#comment:6> Tor Bug Tracker & Wiki <https://trac.torproject.org/> The Tor Project: anonymity online
_______________________________________________ tor-bugs mailing list tor-bugs@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs