openstack/deb-senlin
Code Issues Proposed changes

Use get_rpc_transport instead of get_transport

Browse Source 
With the added get_rpc_transport() function in oslo.messaging [1]
and the recent version bump of oslo.messaging to 5.24.2 [2], we can
safely replace get_transport() with get_rpc_transport() which is much
cleaner.

[1] https://review.openstack.org/#/c/454194/
[2] https://review.openstack.org/#/c/470144/6

Change-Id: If38ffb3fec088b94f39626ed7fa27b343e8dfbea
This commit is contained in:
tengqm 2017-06-05 02:21:56 -04:00
parent 97d6bef2da
commit e52b9b6685
2 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
TODO.rst
View File

@ -38,8 +38,6 @@ POLICY
MISC MISC
---- ----
- Remove all usage of enforce_type when calling oslo.config set_override(). - Remove all usage of enforce_type when calling oslo.config set_override().
- Change messaging.get_transport to get_rpc_transport
-
MIDDLE PRIORITY MIDDLE PRIORITY
=============== ===============

2
senlin/common/messaging.py
View File

@ -74,7 +74,7 @@ def setup(url=None, optional=False):
if not TRANSPORT: if not TRANSPORT:
exmods = ['senlin.common.exception'] exmods = ['senlin.common.exception']
try: try:
TRANSPORT = messaging.get_transport( TRANSPORT = messaging.get_rpc_transport(
cfg.CONF, url, allowed_remote_exmods=exmods) cfg.CONF, url, allowed_remote_exmods=exmods)
except messaging.InvalidTransportURL as e: except messaging.InvalidTransportURL as e:
TRANSPORT = None TRANSPORT = None