On /pifr/status, if a client is currently testing, it should list how long that client has been testing that patch. This can have a dual use so that if a client is testing longer than X, it can reset the client/patch. It's also fun information and good for helping debug what server config changes make an improvement without having to do math with timestamps in the logs.

CommentFileSizeAuthor
#3 537116.png9.15 KBboombatower

Comments

deekayen’s picture

Title: Master keep stats on client test » Master keep time of test

Perhaps rather than just displaying the current time of the test in progress, record and display the time to test the patch in total. It might in some way aide catch in his efforts to improve the performance. The time would vary between clients of course, but it might help identify a patch with slow code.

boombatower’s picture

Status: Active » Postponed (maintainer needs more info)

It currently records all the information you are requesting.

Last requested - time client requested test
Last tested - time client returned test

So obviously that includes XML-RPC time, CVS checkout, but seems like what we are looking for? Or do you want something more specific...like the actually time it took to run tests? (which is somewhat different then how long client has been running).

It would be easy to diff the amount of time passed since client requested test, but not sure if that's what your looking for.

boombatower’s picture

Assigned: Unassigned » boombatower
Status: Postponed (maintainer needs more info) » Fixed
StatusFileSize
new9.15 KB

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.