519882 - Display install_age field on report page

Status: VERIFIED FIXED

(Socorro :: General, task)

Description

[Justin Dolske [:Dolske]]

In bug 519340 I've noticed a sudden recent spike in a particular type of crash, which seems to happen during startup.

It might be useful if we knew when the user had updated to the version of Firefox they're running, relative to the crash time. If they have been using this version of Firefox for a long time, then the sudden onset of a crash would imply something else on their system changed. [Or it's been there all along, but presumably a 100% reproducible crash-on-startup bug would quickly make people switch to another browser or find a fix.]

I know there's also been interest in looking at crash trends vs. day-of-release, so this data might  help refine that to crash trends vs. day-of-user-install.

Side note: The last-update time should probably only have a 1-day granularity, to eliminate concerns about user tracking. [A million or so users install Firefox on any given day, whereas any given microsecond probably only has 1 person installing Firefox.]

Comment 1

[(not currently active) Ted Mielczarek]

We currently send a "InstallTime" variable that is defined as "time the version with this Build ID was first run" (this was implemented in bug 376720):
http://mxr.mozilla.org/mozilla-central/source/toolkit/crashreporter/nsExceptionHandler.cpp#614

I guess we're not displaying that data currently, since we're definitely storing it:
http://code.google.com/p/socorro/source/browse/trunk/socorro/processor/processor.py#661

Over to Socorro to get the data displayed.

Comment 2

[(not currently active) Ted Mielczarek]

Note that Socorro does the math, taking "Crash Time" - "Install Time", and storing the difference as "install_age", which is more useful anyway.

Comment 3

[Michael Morgan [:morgamic]]

This should be pretty quick.  Austin, can you add this?

Comment 4

[Austin King [:ozten]]

Attachment: Adds install_age to report/index. Display's a rough estimate of relative time to be helpful.

Wanted to do something lightweight to make the number of seconds useful, without going crazy and implementing Ruby On Rails "distance_of_time_in_words" in php.

Comment 5

[Ryan Snyder [:ryansnyder] [:rsnyder] [:rysny]]

Comment on attachment 444791 [details] [diff] [review]
Adds install_age to report/index. Display's a rough estimate of relative time to be helpful.

Looks good to me.

Comment 6

[Austin King [:ozten]]

Sending        webapp-php/application/libraries/timeutil.php
Sending        webapp-php/application/views/report/index.php
Sending        webapp-php/tests/TimeUtilTest.php
Transmitting file data ...
Committed revision 2064.

Comment 7

[(not currently active) Ted Mielczarek]

Looks good on stage, thanks!

Comment 8

[Laura Thomson :laura]

Comment on attachment 444791 [details] [diff] [review]
Adds install_age to report/index. Display's a rough estimate of relative time to be helpful.

I guess you don't need review from me, but this looks good (and I am easily amused by the function name).