Skip to content

Commit 2307f9d

Browse files
author
Akihiro Motoki
committed
Avoid using deprecated Q_AGENT_EXTRA_AGENT_OPTS in ML2 setup
Q_AGENT_EXTRA_AGENT_OPTS is deprecated now and stack.sh displays the warning that it will be removed early in "K" cycle. Neutron ML2 devstack uses Q_AGENT_EXTRA_AGENT_OPTS to pass tunnel configurations to Neutron plugin agents. This commit removes the usage of Q_AGENT_EXTRA_AGENT_OPTS and configures these config options directly. Change-Id: I7fc1613ff78c0a8b1e80cc041b06f5d4680c34f2 Closes-Bug: #1354670
1 parent 0824c17 commit 2307f9d

4 files changed

Lines changed: 6 additions & 3 deletions

File tree

lib/neutron_plugins/linuxbridge_agent

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,7 @@ function neutron_plugin_configure_plugin_agent {
4747
iniset /$Q_PLUGIN_CONF_FILE securitygroup firewall_driver neutron.agent.firewall.NoopFirewallDriver
4848
fi
4949
AGENT_BINARY="$NEUTRON_BIN_DIR/neutron-linuxbridge-agent"
50+
iniset /$Q_PLUGIN_CONF_FILE agent tunnel_types $Q_TUNNEL_TYPES
5051
# Define extra "AGENT" configuration options when q-agt is configured by defining
5152
# the array ``Q_AGENT_EXTRA_AGENT_OPTS``.
5253
# For Example: ``Q_AGENT_EXTRA_AGENT_OPTS=(foo=true bar=2)``

lib/neutron_plugins/ml2

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,13 +6,13 @@ ML2_XTRACE=$(set +o | grep xtrace)
66
set +o xtrace
77

88
# Enable this to simply and quickly enable tunneling with ML2.
9-
# Select either 'gre', 'vxlan', or '(gre vxlan)'
9+
# Select either 'gre', 'vxlan', or 'gre,vxlan'
1010
Q_ML2_TENANT_NETWORK_TYPE=${Q_ML2_TENANT_NETWORK_TYPE:-"vxlan"}
1111
# This has to be set here since the agent will set this in the config file
1212
if [[ "$Q_ML2_TENANT_NETWORK_TYPE" == "gre" || "$Q_ML2_TENANT_NETWORK_TYPE" == "vxlan" ]]; then
13-
Q_AGENT_EXTRA_AGENT_OPTS+=(tunnel_types=$Q_ML2_TENANT_NETWORK_TYPE)
13+
Q_TUNNEL_TYPES=$Q_ML2_TENANT_NETWORK_TYPE
1414
elif [[ "$ENABLE_TENANT_TUNNELS" == "True" ]]; then
15-
Q_AGENT_EXTRA_AGENT_OPTS+=(tunnel_types=gre)
15+
Q_TUNNEL_TYPES=gre
1616
fi
1717

1818
# Default openvswitch L2 agent

lib/neutron_plugins/ofagent_agent

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -71,6 +71,7 @@ function neutron_plugin_configure_plugin_agent {
7171
fi
7272
AGENT_BINARY="$NEUTRON_BIN_DIR/neutron-ofagent-agent"
7373

74+
iniset /$Q_PLUGIN_CONF_FILE agent tunnel_types $Q_TUNNEL_TYPES
7475
# Define extra "AGENT" configuration options when q-agt is configured by defining
7576
# defining the array ``Q_AGENT_EXTRA_AGENT_OPTS``.
7677
# For Example: ``Q_AGENT_EXTRA_AGENT_OPTS=(foo=true bar=2)``

lib/neutron_plugins/openvswitch_agent

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -102,6 +102,7 @@ function neutron_plugin_configure_plugin_agent {
102102
# Set root wrap
103103
iniset "/$Q_PLUGIN_CONF_FILE.domU" agent root_helper "$Q_RR_COMMAND"
104104
fi
105+
iniset /$Q_PLUGIN_CONF_FILE agent tunnel_types $Q_TUNNEL_TYPES
105106
# Define extra "AGENT" configuration options when q-agt is configured by defining
106107
# defining the array ``Q_AGENT_EXTRA_AGENT_OPTS``.
107108
# For Example: ``Q_AGENT_EXTRA_AGENT_OPTS=(foo=true bar=2)``

0 commit comments

Comments
 (0)