Merge lp:~thomir-deactivatedaccount/adt-cloud-worker/trunk-fix-queue-name into lp:adt-cloud-worker
Proposed by
Thomi Richards
Status: | Merged |
---|---|
Approved by: | Thomi Richards |
Approved revision: | 26 |
Merged at revision: | 26 |
Proposed branch: | lp:~thomir-deactivatedaccount/adt-cloud-worker/trunk-fix-queue-name |
Merge into: | lp:adt-cloud-worker |
Diff against target: |
12 lines (+1/-1) 1 file modified
adt_cloud_worker/__init__.py (+1/-1) |
To merge this branch: | bzr merge lp:~thomir-deactivatedaccount/adt-cloud-worker/trunk-fix-queue-name |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Celso Providelo (community) | Approve | ||
Review via email: mp+253450@code.launchpad.net |
Commit message
Add API version to queue name.
Description of the change
Add API version to a-c-w queue name.
To post a comment you must log in.
Thanks Thomi,
It looks like cloud-service is already declaring a versioned queue name, so only workers need production update.
Rabbit it so amazing that it even survived to our silliness and fanout requests from the exchange to both queues (due to the same topic/routing- queue):
{{{ amd64.nova 0 1 amd64.nova. v1 10 0 i386.nova 0 1
$ sudo rabbitmqctl list_queues name messages consumers
Listing queues ...
adt.requests.
adt.requests.
adt.requests.
adt.results.v1 0 1
...done.
}}}
We better drop the versioned queue otherwise the new worker will replay test request and there will be a conflict trying to re-create the swift container.
The unversioned queues should also be removed because the will never be used again.