Closed Bug 769295 Opened 10 years ago Closed 10 years ago

Consider lowering OrangeFactor's default timespan of 28 days

Categories

(Tree Management Graveyard :: OrangeFactor, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: emorley, Assigned: emorley)

Details

Attachments

(2 files)

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?
Whoops, wrong part of the form had focus when I hit enter.
CCing a few people who might use OrangeFactor more than others.
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.
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).
Presuming no one else objects: Lowers default number of prior days displayed, from 28 to 7.
Assignee: nobody → bmo
Status: NEW → ASSIGNED
Attachment #637557 - Flags: review?(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.
Attachment #637557 - Flags: review?(mcote) → review+
Comment on attachment 637880 [details] [diff] [review]
Set the x-axis minTickSize for line graphs to 1 day

Thanks!
Attachment #637880 - Flags: review+
Thank you for the quick reviews :-)

http://hg.mozilla.org/automation/orangefactor/rev/e7d20f96487f
http://hg.mozilla.org/automation/orangefactor/rev/4e0671de1ed1
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Deployed to brasstacks.
Product: Testing → Tree Management
Product: Tree Management → Tree Management Graveyard
You need to log in before you can comment on or make changes to this bug.