1164056 - use provisionerId in buildbot bridge routing key

Status: RESOLVED FIXED

(Release Engineering :: General, defect)

Description

[[e-mail] (:bhearsum)]

We only look at workerType right now, which is only unique per provisioner.

Comment 1

[Jonas Finnemann Jensen (:jonasfj)]

Just change lines:
>   routing_key="*.*.*.*.*.*.%s.#" % worker_type,

To:
>   routing_key="*.*.*.*.*.%s.%s.#" % (provisioner_id, worker_type),

And add provisioner_id to the configuration file.

Comment 2

[[e-mail] (:bhearsum)]

Attachment: add provisioner id to routing key

Super simple! Worked fine in dev.

Comment 3

[Jonas Finnemann Jensen (:jonasfj)]

Comment on attachment 8605414 [details] [diff] [review]
add provisioner id to routing key

Review of attachment 8605414 [details] [diff] [review]:
-----------------------------------------------------------------

Looks great!

Comment 4

[[e-mail] (:bhearsum)]

Attachment: add provisioner_id to bbb config

This will add the new, required config item to the bbb config and bump the version to pick up the new code.

Comment 5

[[e-mail] (:bhearsum)]

I just gave this a try in production and it's working well!