769295 - Consider lowering OrangeFactor's default timespan of 28 days

Status: RESOLVED FIXED

(Tree Management Graveyard :: OrangeFactor, defect)

Description

[Ed Morley [:emorley]]

By default, OrangeFactor displays the last 28 days of failures.

When I'm looking through the list a few times a week, there are often transient failures listed that are no longer occurring (and are not yet marked resolved, since more time is needed before we can call them WFM) that get in the way of chasing of the top N oranges (by CC'ing devs, pinging on IRC etc).

As such, every time I use OrangeFactor, I end up reducing the date range to just the last 3/5/7 days. I'm sure the use-cases for frequent users of OrangeFactor vary and so those amounts wouldn't suit everyone - but would it be ok to at least lower the 28 days default in woo.config.js a bit - to something like 7 or 14 days perhaps?

Comment 1

[Ed Morley [:emorley]]

Whoops, wrong part of the form had focus when I hit enter.
CCing a few people who might use OrangeFactor more than others.

Comment 2

[(no longer active)]

I'm fine with 7 days.  Any orange which we have not seen in the past 7 days should not be a priority for anyone.

Comment 3

[Phil Ringnalda (:philor)]

First thing I do when I load OF is change to 7 days (which is unfortunate on a slow connection, since changing at the wrong time while it's still loading breaks the reload).

Comment 4

[Matt Brubeck (:mbrubeck)]

+1

Comment 5

[Ed Morley [:emorley]]

Attachment: Lower days from 28 to 7

Presuming no one else objects: Lowers default number of prior days displayed, from 28 to 7.

Comment 6

[Mark Côté [:mcote]]

Comment on attachment 637557 [details] [diff] [review]
Lower days from 28 to 7

This looks good. However, while you're at this, perhaps you can set the minTickSize flot axis option to [1, "day"]. I just realized that, when you look at 7 days' worth of data, it displays ticks in 12-hour increments. However we only display data daily, so the extra ticks are at best worthless and at worst misleading.

Comment 7

[Ed Morley [:emorley]]

Attachment: Set the x-axis minTickSize for line graphs to 1 day

Comment 8

[Mark Côté [:mcote]]

Comment on attachment 637880 [details] [diff] [review]
Set the x-axis minTickSize for line graphs to 1 day

Thanks!

Comment 9

[Ed Morley [:emorley]]

Thank you for the quick reviews :-)

http://hg.mozilla.org/automation/orangefactor/rev/e7d20f96487f
http://hg.mozilla.org/automation/orangefactor/rev/4e0671de1ed1

Comment 10

[Mark Côté [:mcote]]

Deployed to brasstacks.