openstack/neutron
Code Issues Proposed changes

Remove "vf_management" and "vf_extended_management" checks

Browse Source 
Since [1], the SR-IOV commands are executed using Pyroute2. The
support to execute those commands is guaranteed by the requested
minimum version of this library.

[1]https://review.opendev.org/#/c/727811/

Change-Id: I53372524c9cdc75c4b24e1f3c973f8f87a73a8f9
Closes-Bug: #1888920
This commit is contained in:
Rodolfo Alonso Hernandez 2020-07-27 10:47:16 +00:00
parent 1075e79be5
commit 6da9936814
2 changed files with 1 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
doc/source/cli/neutron-sanity-check.rst
View File

@ -36,14 +36,12 @@ neutron-sanity-check usage
[--noovs_conntrack] [--noovs_geneve]
[--noovs_patch] [--noovs_vxlan] [--noovsdb_native]
[--noread_netns] [--nouse-syslog] [--nova_notify]
[--noverbose] [--novf_extended_management]
[--novf_management] [--nowatch-log-file]
[--noverbose] [--nowatch-log-file]
[--ovs_conntrack] [--ovs_geneve] [--ovs_patch]
[--ovs_vxlan] [--ovsdb_native] [--read_netns]
[--state_path STATE_PATH]
[--syslog-log-facility SYSLOG_LOG_FACILITY]
[--use-syslog] [--verbose] [--version]
[--vf_extended_management] [--vf_management]
[--watch-log-file]
neutron-sanity-check optional arguments
@ -200,12 +198,6 @@ neutron-sanity-check optional arguments
``--noverbose``
The inverse of --verbose
``--novf_extended_management``
The inverse of --vf_extended_management
``--novf_management``
The inverse of --vf_management
``--nowatch-log-file``
The inverse of --watch-log-file
@ -244,12 +236,6 @@ neutron-sanity-check optional arguments
``--version``
show program's version number and exit
``--vf_extended_management``
Check for VF extended management support
``--vf_management``
Check for VF management support
``--watch-log-file``
Uses logging handler designed to watch file system.
When log file is moved or removed this handler will open a new log

4
neutron/cmd/sanity_check.py
View File

@ -357,7 +357,6 @@ def enable_tests_from_config():
run all necessary tests, just by passing in the appropriate configs.
"""
cfg.CONF.set_default('vf_management', True)
cfg.CONF.set_default('arp_header_match', True)
cfg.CONF.set_default('icmpv6_header_match', True)
if 'vxlan' in cfg.CONF.AGENT.tunnel_types:
@ -388,9 +387,6 @@ def enable_tests_from_config():
cfg.CONF.set_default('ipset_installed', True)
if cfg.CONF.SECURITYGROUP.enable_security_group:
cfg.CONF.set_default('ip6tables_installed', True)
if ('sriovnicswitch' in cfg.CONF.ml2.mechanism_drivers and
'qos' in cfg.CONF.ml2.extension_drivers):
cfg.CONF.set_default('vf_extended_management', True)
if cfg.CONF.SECURITYGROUP.firewall_driver in (
'iptables',
'iptables_hybrid',