Remove unused pubsub stuff

Remove unused pubsub stuff

BUG=626501
Committed: https://github.com/luci/luci-py/commit/e3fbb1f5dafa59a2c57cf3a9fe3708f4309ab653

[smut, 2016-11-04 00:13:12]

smut@google.com changed reviewers:
+ ryanmartens@google.com

[smut, 2016-11-04 00:13:12]

See the bug for more context, but basically GCE Backend no longer requires
Pub/Sub.

[ryanmartens, 2016-11-04 14:02:52]

lgtm with some minor comments.
It's nice to simplify the deployment a bit :)

https://codereview.chromium.org/2480653002/diff/1/appengine/gce-backend/catal...
File appengine/gce-backend/catalog.py (left):

https://codereview.chromium.org/2480653002/diff/1/appengine/gce-backend/catal...
appengine/gce-backend/catalog.py:39: 'backend_topic':
pubsub.get_machine_provider_topic(),
We should probably update the comment in the Policies message to capture the
changes regarding the communication with the GCE Backend specifically:
https://github.com/luci/luci-py/blob/master/appengine/components/components/m...

https://codereview.chromium.org/2480653002/diff/1/appengine/gce-backend/catal...
appengine/gce-backend/catalog.py:39: 'backend_topic':
pubsub.get_machine_provider_topic(),
Also, are you planning on pulling out the notification on this channel from
machine provider in another CL, or should you combine it with this one?
https://github.com/luci/luci-py/blob/884aff5a309833270d9243e919e19a4a35ed143a...

[smut, 2016-11-04 21:09:21]

https://codereview.chromium.org/2480653002/diff/1/appengine/gce-backend/catal...
File appengine/gce-backend/catalog.py (left):

https://codereview.chromium.org/2480653002/diff/1/appengine/gce-backend/catal...
appengine/gce-backend/catalog.py:39: 'backend_topic':
pubsub.get_machine_provider_topic(),
On 2016/11/04 14:02:52, ryanmartens wrote:
> We should probably update the comment in the Policies message to capture the
> changes regarding the communication with the GCE Backend specifically:
>
https://github.com/luci/luci-py/blob/master/appengine/components/components/m...

Everything there is technically true, MP will still communicate with its
backends on a backend_topic if provided.

https://codereview.chromium.org/2480653002/diff/1/appengine/gce-backend/catal...
appengine/gce-backend/catalog.py:39: 'backend_topic':
pubsub.get_machine_provider_topic(),
On 2016/11/04 14:02:52, ryanmartens wrote:
> Also, are you planning on pulling out the notification on this channel from
> machine provider in another CL, or should you combine it with this one?
>
https://github.com/luci/luci-py/blob/884aff5a309833270d9243e919e19a4a35ed143a...

Right now backend notification is optional, if backend_topic is present. I'm
guessing we will want to remove this altogether, but I'm not 100% sure that no
backend will use it. I'll save MP and Policies changes for a separate CL if I
completely remove support for backend_topic.

[smut, 2016-11-04 22:16:02]

The CQ bit was checked by smut@google.com

[commit-bot: I haz the power, 2016-11-04 22:16:13]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-04 22:18:50]

Description was changed from

==========
Remove unused pubsub stuff

BUG=626501
==========

to

==========
Remove unused pubsub stuff

BUG=626501

Committed:
https://github.com/luci/luci-py/commit/e3fbb1f5dafa59a2c57cf3a9fe3708f4309ab653
==========

[commit-bot: I haz the power, 2016-11-04 22:18:51]

Committed patchset #1 (id:1) as
https://github.com/luci/luci-py/commit/e3fbb1f5dafa59a2c57cf3a9fe3708f4309ab653