800888 - getBugsForTestFailure() should ignore "1" and "Shutdown", since they return too many results

Status: RESOLVED FIXED

(Tree Management Graveyard :: TBPL, defect)

Description

[Ed Morley [:emorley]]

"Shutdown":
https://bugzilla.mozilla.org/buglist.cgi?quicksearch=ALL%20Shutdown%20sw%3Aorange;list_id=4648629
-> 81 bugs

"1":
https://bugzilla.mozilla.org/buglist.cgi?quicksearch=ALL%201%20sw%3Aorange;list_id=4648612
-> 1783 bugs

Are the top two longest to complete bzapi calls in bug 799534, we should ignore them.

I suggest we add "Shutdown" and then just check the length is say >3 characters.

This will reduce our load on b.m.o - and is also needed for bug 799637 so that we don't return a silly number of suggestions once we no longer time out.

Comment 1

[Ed Morley [:emorley]]

Attachment: Patch v1

* From the CSV in bug 799534, we want to stop searching for "1" and "Shutdown"
* For the former, it makes more sense just to check for the length of the filename, so I've gone with less than 5 as I don't recall having ever seen any tests shorter than that. (And looking at the CSV, the next largest term after "1" was "head.js").

Comment 2

[Ed Morley [:emorley]]

Once again, thank you :-)

https://hg.mozilla.org/users/mstange_themasta.com/tinderboxpushlog/rev/42f0cede7d33