Closed Bug 793131 Opened 9 years ago Closed 9 years ago

Add keyword= support to bzcache's get_bugs()

Categories

(Testing :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: emorley, Unassigned)

References

Details

In bug 790571, both autolog and orangefactor use bzcache to find intermitent orange bugs, eg:

http://hg.mozilla.org/automation/orangefactor/file/0894762c1567/server/handlers.py#l365
> orange_bugs = bzcache.get_bugs(list(buglist), whiteboard='[orange]')

http://hg.mozilla.org/automation/orangefactor/file/0894762c1567/server/handlers.py#l837
> orange_bugs = bzcache.get_bugs(map(lambda x: str(x), 
>                                    self.correlations.keys()),
>                                whiteboard='[orange]')

http://hg.mozilla.org/automation/autolog/file/a2209f94aaea/autolog_server.py#l433
> result = self.bzcache.query(include={ 'summary': tuple(test) },
>                             doc_type=['bugs'])

However we are wanting to switch intermittent failures from using the whiteboard to keyword "intermittent-failure".

So need bzcache to support get_bugs(foo, keyword='intermittent-failure') at:

http://hg.mozilla.org/users/jgriffin_mozilla.com/bzcache/file/15a2b7dae33c/bzcache/bzcache.py#l79
> def get_bugs(self, bugids, whiteboard=''):
After discussion on IRC, it seems like it would make more sense for OrangeFactor and Autolog to just not specify the whiteboard argument. bzcache is only populated with orange bugs anyway (I'll update the cache refresh part to use the keyword in another bug), so we don't in fact need to filter again.

As such, we don't need to add get_bugs() support for filtering by keyword (we currently don't even store keyword in bzcache, so it wouldn't be just a 3 line fix), so can WONTFIX this.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.