openstack/neutron
Code Issues Proposed changes

SR-IOV: deprecate agent_required option

Browse Source 
When SR-IOV introduce in Juno Agent supported only link state change
Some Intel cards don't support setting link state, so to
resolve it the SR-IOV mech driver supports agent and agent less mode.
From Liberty the SR-IOV agent brings more functionality like
qos and port security so we want to make the agent mandatory.
(of course I already talked with Intel Guys to get their approval)

This patch deprecates the agent_required in Liberty
and updates the agent_required default to be True.

DocImpact
Closes-bug: #1488807
Change-Id: I8799425c2825415ef05bec909e6b4085ffc1e3c5
This commit is contained in:
Moshe Levi
2015-08-18 23:10:05 +03:00
parent 82acd0652f
commit cf003af209
2 changed files with 10 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
etc/neutron/plugins/ml2/ml2_conf_sriov.ini
View File

@@ -9,7 +9,10 @@
# Example: supported_pci_vendor_devs = 15b3:1004
#
# (BoolOpt) Requires running SRIOV neutron agent for port binding
# agent_required = False
# agent_required = True
# DEPRECATED: This option is deprecated in the Liberty release
# and will be removed in the Mitaka release. From Mitaka the agent will
# always be required.
[sriov_nic]
# (ListOpt) Comma-separated list of <physical_network>:<network_device>

9
neutron/plugins/ml2/drivers/mech_sriov/mech_driver/mech_driver.py
View File

@@ -38,9 +38,12 @@ sriov_opts = [
"Repository. Default enables support for Intel "
"and Mellanox SR-IOV capable NICs")),
cfg.BoolOpt('agent_required',
default=False,
help=_("SRIOV neutron agent is required for port binding")),
default=True,
help=_("SRIOV neutron agent is required for port binding. "
'DEPRECATED: This option is deprecated in the Liberty '
'release and will be removed in the Mitaka release. '
'From Mitaka the agent will always be required.'),
deprecated_for_removal=True),
]
cfg.CONF.register_opts(sriov_opts, "ml2_sriov")