Skip to content

Commit dc9e894

Browse files
Zuulopenstack-gerrit
authored andcommitted
Merge "Configure [neutron] in nova_cell*.conf"
2 parents d982367 + e95f2a3 commit dc9e894

3 files changed

Lines changed: 51 additions & 33 deletions

File tree

lib/neutron

Lines changed: 32 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -325,25 +325,27 @@ function configure_neutron_rootwrap {
325325
}
326326

327327
# Make Neutron-required changes to nova.conf
328+
# Takes a single argument which is the config file to update.
328329
function configure_neutron_nova_new {
329-
iniset $NOVA_CONF DEFAULT use_neutron True
330-
iniset $NOVA_CONF neutron auth_type "password"
331-
iniset $NOVA_CONF neutron auth_url "$KEYSTONE_SERVICE_URI"
332-
iniset $NOVA_CONF neutron username neutron
333-
iniset $NOVA_CONF neutron password "$SERVICE_PASSWORD"
334-
iniset $NOVA_CONF neutron user_domain_name "Default"
335-
iniset $NOVA_CONF neutron project_name "$SERVICE_TENANT_NAME"
336-
iniset $NOVA_CONF neutron project_domain_name "Default"
337-
iniset $NOVA_CONF neutron auth_strategy $NEUTRON_AUTH_STRATEGY
338-
iniset $NOVA_CONF neutron region_name "$REGION_NAME"
339-
340-
iniset $NOVA_CONF DEFAULT firewall_driver nova.virt.firewall.NoopFirewallDriver
330+
local conf="$1"
331+
iniset $conf DEFAULT use_neutron True
332+
iniset $conf neutron auth_type "password"
333+
iniset $conf neutron auth_url "$KEYSTONE_SERVICE_URI"
334+
iniset $conf neutron username neutron
335+
iniset $conf neutron password "$SERVICE_PASSWORD"
336+
iniset $conf neutron user_domain_name "Default"
337+
iniset $conf neutron project_name "$SERVICE_TENANT_NAME"
338+
iniset $conf neutron project_domain_name "Default"
339+
iniset $conf neutron auth_strategy $NEUTRON_AUTH_STRATEGY
340+
iniset $conf neutron region_name "$REGION_NAME"
341+
342+
iniset $conf DEFAULT firewall_driver nova.virt.firewall.NoopFirewallDriver
341343

342344
# optionally set options in nova_conf
343-
neutron_plugin_create_nova_conf
345+
neutron_plugin_create_nova_conf $conf
344346

345347
if is_service_enabled neutron-metadata-agent; then
346-
iniset $NOVA_CONF neutron service_metadata_proxy "True"
348+
iniset $conf neutron service_metadata_proxy "True"
347349
fi
348350

349351
}
@@ -568,9 +570,23 @@ function configure_neutron {
568570
function configure_neutron_nova {
569571
if is_neutron_legacy_enabled; then
570572
# Call back to old function
571-
create_nova_conf_neutron "$@"
573+
create_nova_conf_neutron $NOVA_CONF
574+
if [[ "${CELLSV2_SETUP}" == "superconductor" ]]; then
575+
for i in $(seq 1 $NOVA_NUM_CELLS); do
576+
local conf
577+
conf=$(conductor_conf $i)
578+
create_nova_conf_neutron $conf
579+
done
580+
fi
572581
else
573-
configure_neutron_nova_new "$@"
582+
configure_neutron_nova_new $NOVA_CONF
583+
if [[ "${CELLSV2_SETUP}" == "superconductor" ]]; then
584+
for i in $(seq 1 $NOVA_NUM_CELLS); do
585+
local conf
586+
conf=$(conductor_conf $i)
587+
configure_neutron_nova_new $conf
588+
done
589+
fi
574590
fi
575591
}
576592

lib/neutron-legacy

Lines changed: 16 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -366,31 +366,32 @@ function configure_mutnauq {
366366
}
367367

368368
function create_nova_conf_neutron {
369-
iniset $NOVA_CONF DEFAULT use_neutron True
370-
iniset $NOVA_CONF neutron auth_type "password"
371-
iniset $NOVA_CONF neutron auth_url "$KEYSTONE_AUTH_URI"
372-
iniset $NOVA_CONF neutron username "$Q_ADMIN_USERNAME"
373-
iniset $NOVA_CONF neutron password "$SERVICE_PASSWORD"
374-
iniset $NOVA_CONF neutron user_domain_name "$SERVICE_DOMAIN_NAME"
375-
iniset $NOVA_CONF neutron project_name "$SERVICE_PROJECT_NAME"
376-
iniset $NOVA_CONF neutron project_domain_name "$SERVICE_DOMAIN_NAME"
377-
iniset $NOVA_CONF neutron auth_strategy "$Q_AUTH_STRATEGY"
378-
iniset $NOVA_CONF neutron region_name "$REGION_NAME"
369+
local conf="$1"
370+
iniset $conf DEFAULT use_neutron True
371+
iniset $conf neutron auth_type "password"
372+
iniset $conf neutron auth_url "$KEYSTONE_AUTH_URI"
373+
iniset $conf neutron username "$Q_ADMIN_USERNAME"
374+
iniset $conf neutron password "$SERVICE_PASSWORD"
375+
iniset $conf neutron user_domain_name "$SERVICE_DOMAIN_NAME"
376+
iniset $conf neutron project_name "$SERVICE_PROJECT_NAME"
377+
iniset $conf neutron project_domain_name "$SERVICE_DOMAIN_NAME"
378+
iniset $conf neutron auth_strategy "$Q_AUTH_STRATEGY"
379+
iniset $conf neutron region_name "$REGION_NAME"
379380

380381
if [[ "$Q_USE_SECGROUP" == "True" ]]; then
381382
LIBVIRT_FIREWALL_DRIVER=nova.virt.firewall.NoopFirewallDriver
382-
iniset $NOVA_CONF DEFAULT firewall_driver $LIBVIRT_FIREWALL_DRIVER
383+
iniset $conf DEFAULT firewall_driver $LIBVIRT_FIREWALL_DRIVER
383384
fi
384385

385386
# optionally set options in nova_conf
386-
neutron_plugin_create_nova_conf
387+
neutron_plugin_create_nova_conf $conf
387388

388389
if is_service_enabled q-meta; then
389-
iniset $NOVA_CONF neutron service_metadata_proxy "True"
390+
iniset $conf neutron service_metadata_proxy "True"
390391
fi
391392

392-
iniset $NOVA_CONF DEFAULT vif_plugging_is_fatal "$VIF_PLUGGING_IS_FATAL"
393-
iniset $NOVA_CONF DEFAULT vif_plugging_timeout "$VIF_PLUGGING_TIMEOUT"
393+
iniset $conf DEFAULT vif_plugging_is_fatal "$VIF_PLUGGING_IS_FATAL"
394+
iniset $conf DEFAULT vif_plugging_timeout "$VIF_PLUGGING_TIMEOUT"
394395
}
395396

396397
# create_mutnauq_accounts() - Set up common required neutron accounts

lib/neutron_plugins/nuage

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,10 +8,11 @@ _XTRACE_NEUTRON_NU=$(set +o | grep xtrace)
88
set +o xtrace
99

1010
function neutron_plugin_create_nova_conf {
11+
local conf="$1"
1112
NOVA_OVS_BRIDGE=${NOVA_OVS_BRIDGE:-"br-int"}
12-
iniset $NOVA_CONF neutron ovs_bridge $NOVA_OVS_BRIDGE
13+
iniset $conf neutron ovs_bridge $NOVA_OVS_BRIDGE
1314
LIBVIRT_FIREWALL_DRIVER=nova.virt.firewall.NoopFirewallDriver
14-
iniset $NOVA_CONF DEFAULT firewall_driver $LIBVIRT_FIREWALL_DRIVER
15+
iniset $conf DEFAULT firewall_driver $LIBVIRT_FIREWALL_DRIVER
1516
}
1617

1718
function neutron_plugin_install_agent_packages {

0 commit comments

Comments
 (0)