Closed
Bug 1362985
Opened 7 years ago
Closed 7 years ago
Autophone - remove support for consuming buildbot pulse messages
Categories
(Testing Graveyard :: Autophone, enhancement)
Testing Graveyard
Autophone
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: bc, Assigned: bc)
References
Details
Attachments
(1 file)
|
7.02 KB,
patch
|
jmaher
:
review+
|
Details | Diff | Splinter Review |
We currently listen to
exchange/build/ with build.#.finished
for buildbot messages related to builds. However, android is now being totally built by taskcluster and all of the pulse messages and platform checking etc we are performing on the buildbot pulse messages is wasted cpu cycles.
I think we can and should eliminate the exchange/build/ listeners totally which will eliminate of our biggest cpu and network consumers.
This will also simplify the work needed to support GeckoView (geckoview_example.apk) since there is no need to even consider it on the buildbot handle_build.
jmaher, gbrown: See anything wrong with this?
Flags: needinfo?(jmaher)
Flags: needinfo?(gbrown)
|
Assignee | |
Comment 3•7 years ago
|
||
works well in testing.
Attachment #8865866 -
Flags: review?(jmaher)
|
||
Comment 4•7 years ago
|
||
Comment on attachment 8865866 [details] [diff] [review]
bug-1362985-v1.patch
Review of attachment 8865866 [details] [diff] [review]:
-----------------------------------------------------------------
I love code removals
Attachment #8865866 -
Flags: review?(jmaher) → review+
|
|
Assignee | |
Comment 5•7 years ago
|
||
Assignee: nobody → bob
Blocks: autophone-deployments
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Comment 6•7 years ago
|
||
and the build queues deleted as well.
|
||
Updated•3 years ago
|
Product: Testing → Testing Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•