Closed Bug 1362985 Opened 3 years ago Closed 3 years ago

Autophone - remove support for consuming buildbot pulse messages

Categories

(Testing :: Autophone, enhancement)

enhancement
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: bc, Assigned: bc)

References

Details

Attachments

(1 file)

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)
This seems completely reasonable to me.
Flags: needinfo?(gbrown)
++
Flags: needinfo?(jmaher)
works well in testing.
Attachment #8865866 - Flags: review?(jmaher)
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+
https://github.com/mozilla/autophone/commit/18c1a868ae75877b624d212dc999ce4d753e2ce7
Assignee: nobody → bob
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
and the build queues deleted as well.
You need to log in before you can comment on or make changes to this bug.