Closed
Bug 807114
Opened 12 years ago
Closed 12 years ago
Update OrangeFactor to use keywords=intermittent-failure rather than whiteboard=[orange]
Categories
(Tree Management Graveyard :: OrangeFactor, defect)
Tree Management Graveyard
OrangeFactor
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: emorley, Assigned: emorley)
References
Details
Attachments
(1 file)
(deleted),
patch
|
jgriffin
:
review+
mcote
:
feedback+
|
Details | Diff | Splinter Review |
Broken out from bug 790571 for clarity in respect to pushing all of the various parts to production.
Assignee | ||
Comment 1•12 years ago
|
||
Attachment #676789 -
Flags: review?(jgriffin)
Attachment #676789 -
Flags: feedback?(mcote)
Assignee | ||
Comment 2•12 years ago
|
||
Meant to say, per bug 793131, we're not going to try and filter the bugs returned from bzcache, since they should just be the orange bugs anyway.
Assignee | ||
Comment 3•12 years ago
|
||
(In reply to Ed Morley [:edmorley UTC+0] from comment #2)
> Meant to say, per bug 793131, we're not going to try and filter the bugs
> returned from bzcache, since they should just be the orange bugs anyway.
Thinking about this more - whilst bzcache's bz_cache_refresh.py will only populate bz_cache with orange bugs, whenever TBPL submits a comment containing a non-orange bug number (eg infra issue), bzcache's get_bugs() will go out and fetch any bugs not already in the cache, so will still add non-intermittents.
However, I don't see this as a problem, since it means that we'll be able to track transient infra issues on OF and see how they compare to the intermittents.
Comment 4•12 years ago
|
||
Comment on attachment 676789 [details] [diff] [review]
Patch v1
Review of attachment 676789 [details] [diff] [review]:
-----------------------------------------------------------------
Can't test this atm but it looks good. Maybe file a bug to remove the references to whiteboard from bzcache as well? No point in having that specific use case left in there.
Attachment #676789 -
Flags: feedback?(mcote) → feedback+
Comment 5•12 years ago
|
||
Comment on attachment 676789 [details] [diff] [review]
Patch v1
Review of attachment 676789 [details] [diff] [review]:
-----------------------------------------------------------------
I think this is good, but I haven't tested it either. I guess we'll test it when we deploy it. :)
Attachment #676789 -
Flags: review?(jgriffin) → review+
Assignee | ||
Comment 6•12 years ago
|
||
Assignee | ||
Comment 7•12 years ago
|
||
In production.
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•10 years ago
|
Product: Testing → Tree Management
Updated•4 years ago
|
Product: Tree Management → Tree Management Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•