)]}'
{"lib/neutron-legacy":[{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"f008925b255c7da81792c72f2db56180e3d69a78","unresolved":false,"context_lines":[{"line_number":274,"context_line":"function _determine_config_server {"},{"line_number":275,"context_line":"    local cfg_file"},{"line_number":276,"context_line":"    local opts\u003d\"--config-file $NEUTRON_CONF --config-file /$Q_PLUGIN_CONF_FILE\""},{"line_number":277,"context_line":"    if is_service_enabled q-lbaasv2; then"},{"line_number":278,"context_line":"        if ! is_service_enabled octavia; then"},{"line_number":279,"context_line":"            opts+\u003d\" --config-file $NEUTRON_LBAAS_CONF\""},{"line_number":280,"context_line":"        fi"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"9a89bdaa_2123fcbe","line":277,"updated":"2016-09-05 12:40:11.000000000","message":"I don\u0027t think you do it correct. First, you touch generic neutron code with lbaas specific stuff. And you should actually not touch neutron-legacy but lib/neutron since that\u0027s where new development happens.\n\nThere is a patch for devstack that allows services to add new config files to load: https://review.openstack.org/#/c/342493/ After it\u0027s in, you should be able to use the functionality in octavia plugin.","commit_id":"61ee4c9be776e4ac98cd93ffb7ab3657e6364658"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"9f4ad1805f78d408c59151aa13554a1203910dac","unresolved":false,"context_lines":[{"line_number":274,"context_line":"function _determine_config_server {"},{"line_number":275,"context_line":"    local cfg_file"},{"line_number":276,"context_line":"    local opts\u003d\"--config-file $NEUTRON_CONF --config-file /$Q_PLUGIN_CONF_FILE\""},{"line_number":277,"context_line":"    if is_service_enabled q-lbaasv2; then"},{"line_number":278,"context_line":"        if ! is_service_enabled octavia; then"},{"line_number":279,"context_line":"            opts+\u003d\" --config-file $NEUTRON_LBAAS_CONF\""},{"line_number":280,"context_line":"        fi"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"9a89bdaa_6756f42a","line":277,"in_reply_to":"9a89bdaa_2123fcbe","updated":"2016-09-05 13:19:30.000000000","message":"btw you seem to be able to utilize Q_PLUGIN_EXTRA_CONF_FILES as in line 282 in your plugin.","commit_id":"61ee4c9be776e4ac98cd93ffb7ab3657e6364658"},{"author":{"_account_id":8788,"name":"Miguel Angel Ajo","email":"mangelajo@redhat.com","username":"mangelajo"},"change_message_id":"d10876f7c41f79bfe0a0238ff2706a6c55a10b1d","unresolved":false,"context_lines":[{"line_number":274,"context_line":"function _determine_config_server {"},{"line_number":275,"context_line":"    local cfg_file"},{"line_number":276,"context_line":"    local opts\u003d\"--config-file $NEUTRON_CONF --config-file /$Q_PLUGIN_CONF_FILE\""},{"line_number":277,"context_line":"    if is_service_enabled q-lbaasv2; then"},{"line_number":278,"context_line":"        if ! is_service_enabled octavia; then"},{"line_number":279,"context_line":"            opts+\u003d\" --config-file $NEUTRON_LBAAS_CONF\""},{"line_number":280,"context_line":"        fi"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"9a89bdaa_cc5fe070","line":277,"in_reply_to":"9a89bdaa_6756f42a","updated":"2016-09-05 14:31:57.000000000","message":"Yes, that what I was suggesting in our chat, to extend that list with the files we need, but from the neutron-lbaas plugin instead of here.","commit_id":"61ee4c9be776e4ac98cd93ffb7ab3657e6364658"},{"author":{"_account_id":8788,"name":"Miguel Angel Ajo","email":"mangelajo@redhat.com","username":"mangelajo"},"change_message_id":"21d9a1f7ae72ee666165360385c981c13c246b7f","unresolved":false,"context_lines":[{"line_number":276,"context_line":"    local opts\u003d\"--config-file $NEUTRON_CONF --config-file /$Q_PLUGIN_CONF_FILE\""},{"line_number":277,"context_line":"    if is_service_enabled q-lbaasv2; then"},{"line_number":278,"context_line":"        if ! is_service_enabled octavia; then"},{"line_number":279,"context_line":"            opts+\u003d\" --config-file $NEUTRON_LBAAS_CONF\""},{"line_number":280,"context_line":"        fi"},{"line_number":281,"context_line":"    fi"},{"line_number":282,"context_line":"    for cfg_file in ${Q_PLUGIN_EXTRA_CONF_FILES[@]}; do"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"9a89bdaa_aa88d3e8","line":279,"updated":"2016-09-05 08:03:30.000000000","message":"why is octavia excluded?","commit_id":"61ee4c9be776e4ac98cd93ffb7ab3657e6364658"},{"author":{"_account_id":6579,"name":"Nir Magnezi","email":"nmagnezi@redhat.com","username":"nmagnezi"},"change_message_id":"a288303fb0cf0fae97796bef29cc36b972422d05","unresolved":false,"context_lines":[{"line_number":276,"context_line":"    local opts\u003d\"--config-file $NEUTRON_CONF --config-file /$Q_PLUGIN_CONF_FILE\""},{"line_number":277,"context_line":"    if is_service_enabled q-lbaasv2; then"},{"line_number":278,"context_line":"        if ! is_service_enabled octavia; then"},{"line_number":279,"context_line":"            opts+\u003d\" --config-file $NEUTRON_LBAAS_CONF\""},{"line_number":280,"context_line":"        fi"},{"line_number":281,"context_line":"    fi"},{"line_number":282,"context_line":"    for cfg_file in ${Q_PLUGIN_EXTRA_CONF_FILES[@]}; do"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"9a89bdaa_c4798b31","line":279,"in_reply_to":"9a89bdaa_aa88d3e8","updated":"2016-09-05 08:49:00.000000000","message":"The Octavia exclusion was made per Stephen\u0027s comment on https://review.openstack.org/#/c/299998/\n\nCurrently, when you configure devstack to run with Octavia, you will notice the following:\n1. q-svc get the following config files: neutron.conf and plugins/ml2/ml2_conf.ini\n2. o-api, o-cw, o-hk, o-hm get the following config file: octavia.conf\n\nSo none of the above services use neutron_lbaas.conf (Even though the service_auth and service_providers sections were configured).\nThe octavia.conf file also has a keystone_authtoken section, so that can explain why it does not need to read it from neutron_lbaas.conf\n\nStephen, could you please add more details about this?","commit_id":"61ee4c9be776e4ac98cd93ffb7ab3657e6364658"},{"author":{"_account_id":11685,"name":"Stephen Balukoff","email":"stephen@balukoff.com","username":"sbalukoff"},"change_message_id":"db136bf0e4ea00297693ca4daa669bf44853cacb","unresolved":false,"context_lines":[{"line_number":276,"context_line":"    local opts\u003d\"--config-file $NEUTRON_CONF --config-file /$Q_PLUGIN_CONF_FILE\""},{"line_number":277,"context_line":"    if is_service_enabled q-lbaasv2; then"},{"line_number":278,"context_line":"        if ! is_service_enabled octavia; then"},{"line_number":279,"context_line":"            opts+\u003d\" --config-file $NEUTRON_LBAAS_CONF\""},{"line_number":280,"context_line":"        fi"},{"line_number":281,"context_line":"    fi"},{"line_number":282,"context_line":"    for cfg_file in ${Q_PLUGIN_EXTRA_CONF_FILES[@]}; do"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"9a89bdaa_fdd3675d","line":279,"in_reply_to":"9a89bdaa_c4798b31","updated":"2016-09-05 17:12:59.000000000","message":"If neutron-lbaas is activated (whether or not Octavia is being used, or the namespace driver), then we should be pulling in the neutron_lbaas.conf file on the neutron q-svc command-line.","commit_id":"61ee4c9be776e4ac98cd93ffb7ab3657e6364658"}]}