Opened 17 months ago

Closed 5 months ago

Last modified 5 months ago

#30611 closed enhancement (implemented)

Include partial download times in Time to complete request graphs

Reported by: karsten Owned by: karsten
Priority: Medium Milestone:
Component: Metrics/Website Version:
Severity: Normal Keywords:
Cc: metrics-team Actual Points: 0.6
Parent ID: Points:
Reviewer: Sponsor: Sponsor59-must


Once we have partial completion times in the onionperf database (#30610), we can include more measurements in the 50 KiB and 1 MiB graphs by simply switching from DATACOMPLETE to PARTIAL[51200|1048576]. And maybe we should just use PARTIAL5242880 in the 5 MiB graph, too, for consistency, even though the resulting graph should be the same or at least very similar.

As part of this change we might have to update the text to say that we're plotting the same 1 MiB/5 MiB request in two/three graphs, which reduces diversity to some extent.

In any case we must update the reproducible metrics page on this aspect.

Note that this ticket is not a child of #29507 even though it depends on other children of that ticket.

Child Tickets

Change History (3)

comment:1 Changed 5 months ago by karsten

Owner: changed from metrics-team to karsten
Status: newaccepted

I started working on this over the weekend and will merge and deploy a patch today.

comment:2 Changed 5 months ago by karsten

Actual Points: 0.6
Resolution: implemented
Status: acceptedclosed

Implemented in commit 110cb01, pushed to master, and deployed on the server. Closing.

comment:3 Changed 5 months ago by karsten

Sponsor: Sponsor59-must

This was part of Sponsor 59 work.

Note: See TracTickets for help on using tickets.