openstack/masakari
Code Issues Proposed changes

Fix devstack installation of masakari

Browse Source 
Recently a commit: 188173a8072b143a28779b875af617ac7f35a92d got
merged into masakari which replaces a rpc method 'get_transport'
with 'get_rpc_transport'. The 'get_rpc_transport' expects the
oslo.messaging config option 'transport_url' [1] to be present in
the [DEFAULT] section of masakari.conf. The current masakari devstack
script doesn't add 'transport_url' in masakari.conf which results
into failure of devstack installation of masakari.

This patch fixes this issue by using an inbuilt devstack function
'iniset_rpc_backend' to add the 'transport_url' config option in
masakari.conf.

[1] * https://github.com/openstack/oslo.messaging/blob/master/oslo_messaging/transport.py#L44
    * https://github.com/openstack/oslo.messaging/blob/master/oslo_messaging/transport.py#L201

Closes-Bug: #1728544
Change-Id: I75da054c6669aa90dd96c3ede3ae98de74275de2
This commit is contained in:
Dinesh Bhor 2017-10-30 12:58:14 +05:30 committed by Sampath Priyankara (samP)
parent bdafc3ef32
commit 57bbe7dead
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
devstack/plugin.sh
View File

@ -122,6 +122,8 @@ function configure_masakari {
iniset $MASAKARI_CONF DEFAULT os_privileged_user_tenant "$SERVICE_PROJECT_NAME"
iniset $MASAKARI_CONF DEFAULT graceful_shutdown_timeout "$SERVICE_GRACEFUL_SHUTDOWN_TIMEOUT"
iniset_rpc_backend masakari $MASAKARI_CONF DEFAULT
if is_service_enabled tls-proxy; then
iniset $MASAKARI_CONF DEFAULT masakari_api_listen_port $MASAKARI_SERVICE_PORT_INT
fi