Closed Bug 807114 Opened 8 years ago Closed 8 years ago

Update OrangeFactor to use keywords=intermittent-failure rather than whiteboard=[orange]

Categories

(Tree Management :: OrangeFactor, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: emorley, Assigned: emorley)

References

Details

Attachments

(1 file)

Broken out from bug 790571 for clarity in respect to pushing all of the various parts to production.
Attached patch Patch v1Splinter Review
Attachment #676789 - Flags: review?(jgriffin)
Attachment #676789 - Flags: feedback?(mcote)
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.
Depends on: 791758
(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 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 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+
Blocks: 807930
Depends on: 813351
In production.
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Product: Testing → Tree Management
You need to log in before you can comment on or make changes to this bug.