793131 - Add keyword= support to bzcache's get_bugs()

Status: RESOLVED WONTFIX

(Testing :: General, defect)

Description

[Ed Morley [:emorley]]

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=''):

Comment 1

[Ed Morley [:emorley]]

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.