1194495 - [MozillaPulse] Add consumer for the TaskCluster exchanges

Status: RESOLVED FIXED

(Webtools :: Pulse, defect)

Description

[Henrik Skupin [:whimboo][⌚️UTC+1]]

There is no consumer for the TaskCluster exchange yet. We should get this added.

http://docs.taskcluster.net/queue/exchanges/

I will dig into this tomorrow.

Comment 1

[Henrik Skupin [:whimboo][⌚️UTC+1]]

There are multiple exchanges defined and we also need a publisher for it.

Comment 2

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Bug 1180897 implemented the multiple exchange feature recently. So i will make of it even it will be hard to figure out which topic to set given that 7 exchanges are present. The constructor will look like the following in case you are interested in completed tasks only:

pulse.configure(topic=['', '', '', '', '#', '', ''], [..])

Comment 3

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Attachment: taskcluster

This patch should add everything. Sadly the tests are failing and I do not know why. Jonathan, do you have an idea?

Comment 4

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Given that we wont publish to taskcluster this is for consumers only.

Comment 5

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Attachment: taskcluster v2

The problem here was that I missed to update the configuration for taskcluster with the local host settings. That's why it tried to connect to pulse.mozilla.org, which can indeed not work.

Comment 6

[Jonathan Griffin (:jgriffin)]

landed with version bump to 1.2: https://hg.mozilla.org/automation/mozillapulse/rev/8ae4557e0f7a

Comment 7

[Jonathan Griffin (:jgriffin)]

https://pypi.python.org/pypi/MozillaPulse/1.2

Comment 8

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Sadly to the final slashes of the exchange values the consumer does not work and errors out with:

> Error occurred during pulse.listen(): Exchange.declare: (404) NOT_FOUND - no exchange 'exchange/taskcluster-queue/v1/task-completed/' in vhost '/'

I haven't thought that this is important. But given it is we need a follow-up patch here with the removal of those two trailing slashes. :/

Comment 9

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Attachment: taskcluster_exchangekey

Sorry, but it wasn't clear to me that it makes such a difference. :/

Comment 10

[Jonathan Griffin (:jgriffin)]

https://hg.mozilla.org/automation/mozillapulse/rev/e6d10b16c666
https://pypi.python.org/pypi/MozillaPulse/1.2.1